From: Greg Kroah-Hartman Date: Mon, 18 Sep 2017 14:22:54 +0000 (+0200) Subject: 4.13-stable patches X-Git-Tag: v4.9.51~4 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=dd41ca269b00d3cc3abc1e0b756f1a797ee4dd30;p=thirdparty%2Fkernel%2Fstable-queue.git 4.13-stable patches added patches: xfs-fix-compiler-warnings.patch --- diff --git a/queue-4.13/series b/queue-4.13/series index ac2cc1eeffc..388ddb47baf 100644 --- a/queue-4.13/series +++ b/queue-4.13/series @@ -50,3 +50,4 @@ xfs-open-code-end_buffer_async_write-in-xfs_finish_page_writeback.patch xfs-use-kmem_free-to-free-return-value-of-kmem_zalloc.patch md-raid1-10-reset-bio-allocated-from-mempool.patch md-raid5-release-flush-io-in-raid5_do_work.patch +xfs-fix-compiler-warnings.patch diff --git a/queue-4.13/xfs-fix-compiler-warnings.patch b/queue-4.13/xfs-fix-compiler-warnings.patch new file mode 100644 index 00000000000..2e2979336f5 --- /dev/null +++ b/queue-4.13/xfs-fix-compiler-warnings.patch @@ -0,0 +1,97 @@ +From 7bf7a193a90cadccaad21c5970435c665c40fe27 Mon Sep 17 00:00:00 2001 +From: "Darrick J. Wong" +Date: Thu, 31 Aug 2017 15:11:06 -0700 +Subject: xfs: fix compiler warnings + +From: Darrick J. Wong + +commit 7bf7a193a90cadccaad21c5970435c665c40fe27 upstream. + +Fix up all the compiler warnings that have crept in. + +Signed-off-by: Darrick J. Wong +Reviewed-by: Christoph Hellwig +Cc: Arnd Bergmann +Signed-off-by: Greg Kroah-Hartman + +--- + fs/xfs/libxfs/xfs_bmap.c | 2 +- + fs/xfs/libxfs/xfs_inode_fork.c | 9 +++------ + fs/xfs/xfs_buf_item.c | 2 ++ + fs/xfs/xfs_iops.c | 2 +- + fs/xfs/xfs_log_recover.c | 4 ++++ + 5 files changed, 11 insertions(+), 8 deletions(-) + +--- a/fs/xfs/libxfs/xfs_bmap.c ++++ b/fs/xfs/libxfs/xfs_bmap.c +@@ -579,7 +579,7 @@ xfs_bmap_validate_ret( + + #else + #define xfs_bmap_check_leaf_extents(cur, ip, whichfork) do { } while (0) +-#define xfs_bmap_validate_ret(bno,len,flags,mval,onmap,nmap) ++#define xfs_bmap_validate_ret(bno,len,flags,mval,onmap,nmap) do { } while (0) + #endif /* DEBUG */ + + /* +--- a/fs/xfs/libxfs/xfs_inode_fork.c ++++ b/fs/xfs/libxfs/xfs_inode_fork.c +@@ -1499,14 +1499,11 @@ xfs_iext_realloc_indirect( + xfs_ifork_t *ifp, /* inode fork pointer */ + int new_size) /* new indirection array size */ + { +- int nlists; /* number of irec's (ex lists) */ +- int size; /* current indirection array size */ +- + ASSERT(ifp->if_flags & XFS_IFEXTIREC); +- nlists = ifp->if_real_bytes / XFS_IEXT_BUFSZ; +- size = nlists * sizeof(xfs_ext_irec_t); + ASSERT(ifp->if_real_bytes); +- ASSERT((new_size >= 0) && (new_size != size)); ++ ASSERT((new_size >= 0) && ++ (new_size != ((ifp->if_real_bytes / XFS_IEXT_BUFSZ) * ++ sizeof(xfs_ext_irec_t)))); + if (new_size == 0) { + xfs_iext_destroy(ifp); + } else { +--- a/fs/xfs/xfs_buf_item.c ++++ b/fs/xfs/xfs_buf_item.c +@@ -570,7 +570,9 @@ xfs_buf_item_unlock( + bool aborted = !!(lip->li_flags & XFS_LI_ABORTED); + bool hold = !!(bip->bli_flags & XFS_BLI_HOLD); + bool dirty = !!(bip->bli_flags & XFS_BLI_DIRTY); ++#if defined(DEBUG) || defined(XFS_WARN) + bool ordered = !!(bip->bli_flags & XFS_BLI_ORDERED); ++#endif + + /* Clear the buffer's association with this transaction. */ + bp->b_transp = NULL; +--- a/fs/xfs/xfs_iops.c ++++ b/fs/xfs/xfs_iops.c +@@ -817,7 +817,7 @@ xfs_vn_setattr_nonsize( + * Caution: The caller of this function is responsible for calling + * setattr_prepare() or otherwise verifying the change is fine. + */ +-int ++STATIC int + xfs_setattr_size( + struct xfs_inode *ip, + struct iattr *iattr) +--- a/fs/xfs/xfs_log_recover.c ++++ b/fs/xfs/xfs_log_recover.c +@@ -4827,12 +4827,16 @@ xlog_recover_process_intents( + int error = 0; + struct xfs_ail_cursor cur; + struct xfs_ail *ailp; ++#if defined(DEBUG) || defined(XFS_WARN) + xfs_lsn_t last_lsn; ++#endif + + ailp = log->l_ailp; + spin_lock(&ailp->xa_lock); + lip = xfs_trans_ail_cursor_first(ailp, &cur, 0); ++#if defined(DEBUG) || defined(XFS_WARN) + last_lsn = xlog_assign_lsn(log->l_curr_cycle, log->l_curr_block); ++#endif + while (lip != NULL) { + /* + * We're done when we see something other than an intent.