On 6/21/21 10:58 PM, Matthew Wilcox wrote: > On Mon, Jun 21, 2021 at 11:07:16PM +0200, Veronika Kabatova wrote: >> On Mon, Jun 21, 2021 at 11:00 PM Jens Axboe <axboe@xxxxxxxxx> wrote: >>> >>> On 6/21/21 2:57 PM, Veronika Kabatova wrote: >>>> On Mon, Jun 21, 2021 at 9:20 PM Veronika Kabatova <vkabatov@xxxxxxxxxx> wrote: >>>>> >>>>> On Mon, Jun 21, 2021 at 9:17 PM CKI Project <cki-project@xxxxxxxxxx> wrote: >>>>>> >>>>>> >>>>>> Hello, >>>>>> >>>>>> We ran automated tests on a recent commit from this kernel tree: >>>>>> >>>>>> Kernel repo: https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git >>>>>> Commit: b0740de3330a - Merge branch 'for-5.14/drivers-late' into for-next >>>>>> >>>>>> The results of these automated tests are provided below. >>>>>> >>>>>> Overall result: FAILED (see details below) >>>>>> Merge: OK >>>>>> Compile: FAILED >>>>>> >>>>> >>>>> Hi, >>>>> >>>>> the failure is introduced between this commit and d142f908ebab64955eb48e. >>>>> Currently seeing if I can bisect it closer but maybe someone already has an >>>>> idea what went wrong. >>>>> >>>> >>>> First commit failing the compilation is 7a2b0ef2a3b83733d7. >>> >>> Where's the log? Adding Willy... >>> >> >> Logs and kernel configs for each arch are linked in the original email at >> https://arr-cki-prod-datawarehouse-public.s3.amazonaws.com/index.html?prefix=datawarehouse-public/2021/06/21/324657779 > > Which aren't there by the time they get to the original commit author. > You need to do better than this; the Intel build-bot bisects to the > commit which actually causes the error. Matthew, I've just followed the link out of curiosity: ... 00:04:22 In file included from fs/orangefs/inode.c:14: 00:04:22 ./include/linux/fileattr.h:36:60: warning: ‘struct fsxattr’ declared inside parameter list will not be visible outside of this definition or declaration 00:04:22 36 | int copy_fsxattr_to_user(const struct fileattr *fa, struct fsxattr __user *ufa); 00:04:22 | ^~~~~~~ 00:04:22 ./include/linux/fileattr.h: In function ‘fileattr_has_fsx’: 00:04:22 ./include/linux/fileattr.h:13:10: error: ‘FS_XFLAG_SYNC’ undeclared (first use in this function) 00:04:22 13 | (FS_XFLAG_SYNC | FS_XFLAG_IMMUTABLE | FS_XFLAG_APPEND | \ 00:04:22 | ^~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:13:10: note: each undeclared identifier is reported only once for each function it appears in 00:04:22 13 | (FS_XFLAG_SYNC | FS_XFLAG_IMMUTABLE | FS_XFLAG_APPEND | \ 00:04:22 | ^~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:13:26: error: ‘FS_XFLAG_IMMUTABLE’ undeclared (first use in this function) 00:04:22 13 | (FS_XFLAG_SYNC | FS_XFLAG_IMMUTABLE | FS_XFLAG_APPEND | \ 00:04:22 | ^~~~~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:13:47: error: ‘FS_XFLAG_APPEND’ undeclared (first use in this function) 00:04:22 13 | (FS_XFLAG_SYNC | FS_XFLAG_IMMUTABLE | FS_XFLAG_APPEND | \ 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:14:10: error: ‘FS_XFLAG_NODUMP’ undeclared (first use in this function); did you mean ‘FS_XFLAG_COMMON’? 00:04:22 14 | FS_XFLAG_NODUMP | FS_XFLAG_NOATIME | FS_XFLAG_DAX | \ 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:14:28: error: ‘FS_XFLAG_NOATIME’ undeclared (first use in this function) 00:04:22 14 | FS_XFLAG_NODUMP | FS_XFLAG_NOATIME | FS_XFLAG_DAX | \ 00:04:22 | ^~~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:14:47: error: ‘FS_XFLAG_DAX’ undeclared (first use in this function) 00:04:22 14 | FS_XFLAG_NODUMP | FS_XFLAG_NOATIME | FS_XFLAG_DAX | \ 00:04:22 | ^~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:15:10: error: ‘FS_XFLAG_PROJINHERIT’ undeclared (first use in this function) 00:04:22 15 | FS_XFLAG_PROJINHERIT) 00:04:22 | ^~~~~~~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h:51:37: note: in expansion of macro ‘FS_XFLAG_COMMON’ 00:04:22 51 | ((fa->fsx_xflags & ~FS_XFLAG_COMMON) || fa->fsx_extsize != 0 || 00:04:22 | ^~~~~~~~~~~~~~~ 00:04:22 ./include/linux/fileattr.h: At top level: 00:04:22 ./include/linux/fileattr.h:55:29: warning: ‘struct dentry’ declared inside parameter list will not be visible outside of this definition or declaration 00:04:22 55 | int vfs_fileattr_get(struct dentry *dentry, struct fileattr *fa); 00:04:22 | ^~~~~~ 00:04:22 ./include/linux/fileattr.h:56:64: warning: ‘struct dentry’ declared inside parameter list will not be visible outside of this definition or declaration 00:04:22 56 | int vfs_fileattr_set(struct user_namespace *mnt_userns, struct dentry *dentry, 00:04:22 | ^~~~~~ 00:04:22 ./include/linux/fileattr.h: In function ‘fileattr_has_fsx’: 00:04:22 ./include/linux/fileattr.h:53:1: error: control reaches end of non-void function [-Werror=return-type] 00:04:22 53 | } 00:04:22 | ^ 00:04:22 cc1: some warnings being treated as errors ... -- Pavel Begunkov
