From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Hubbard Subject: Re: [PATCH v2 4/6] mm/fs: add a sync_mode param for clear_page_dirty_for_io() Date: Sun, 1 Jul 2018 21:40:22 -0700 Message-ID: References: <20180702005654.20369-5-jhubbard@nvidia.com> <201807020900.J6omiRbx%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201807020900.J6omiRbx%fengguang.wu@intel.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: kbuild test robot , john.hubbard@gmail.com Cc: kbuild-all@01.org, Matthew Wilcox , Michal Hocko , Christopher Lameter , Jason Gunthorpe , Dan Williams , Jan Kara , linux-mm@kvack.org, LKML , linux-rdma , linux-fsdevel@vger.kernel.org List-Id: linux-rdma@vger.kernel.org On 07/01/2018 07:47 PM, kbuild test robot wrote: > Hi John, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on linus/master] > [also build test ERROR on v4.18-rc3] > [cannot apply to next-20180629] > [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] > > url: https://github.com/0day-ci/linux/commits/john-hubbard-gmail-com/mm-fs-gup-don-t-unmap-or-drop-filesystem-buffers/20180702-090125 > config: i386-randconfig-x075-201826 (attached as .config) > compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 > reproduce: > # save the attached .config to linux build tree > make ARCH=i386 > > All errors (new ones prefixed by >>): > > fs/f2fs/dir.c: In function 'f2fs_delete_entry': >>> fs/f2fs/dir.c:734:33: error: 'WB_SYNC_ALL' undeclared (first use in this function); did you mean 'FS_SYNC_FL'? > clear_page_dirty_for_io(page, WB_SYNC_ALL); > ^~~~~~~~~~~ > FS_SYNC_FL Fixed locally, via: diff --git a/fs/f2fs/dir.c b/fs/f2fs/dir.c index 258f9dc117f4..ca20c1262582 100644 --- a/fs/f2fs/dir.c +++ b/fs/f2fs/dir.c @@ -16,6 +16,7 @@ #include "acl.h" #include "xattr.h" #include +#include static unsigned long dir_blocks(struct inode *inode) { thanks, -- John Hubbard NVIDIA