From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 249F3C77B60 for ; Mon, 24 Apr 2023 03:13:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229517AbjDXDNe (ORCPT ); Sun, 23 Apr 2023 23:13:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57092 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229476AbjDXDNd (ORCPT ); Sun, 23 Apr 2023 23:13:33 -0400 Received: from out30-131.freemail.mail.aliyun.com (out30-131.freemail.mail.aliyun.com [115.124.30.131]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4A53B1701 for ; Sun, 23 Apr 2023 20:13:31 -0700 (PDT) X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R351e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=ay29a033018045170;MF=hsiangkao@linux.alibaba.com;NM=1;PH=DS;RN=4;SR=0;TI=SMTPD_---0Vgm9ltO_1682306005; Received: from 30.97.48.250(mailfrom:hsiangkao@linux.alibaba.com fp:SMTPD_---0Vgm9ltO_1682306005) by smtp.aliyun-inc.com; Mon, 24 Apr 2023 11:13:27 +0800 Message-ID: Date: Mon, 24 Apr 2023 11:13:25 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH v2] src: fix detached_mounts_propagation compile errors To: Zorro Lang Cc: fstests@vger.kernel.org, Christian Brauner , "Yang Xu (Fujitsu)" References: <20230419032955.114278-1-hsiangkao@linux.alibaba.com> <20230420021106.41970-1-hsiangkao@linux.alibaba.com> <20230422135427.vbrnj6vh46omo34e@zlang-mailbox> From: Gao Xiang In-Reply-To: <20230422135427.vbrnj6vh46omo34e@zlang-mailbox> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org Hi Zorro, On 2023/4/22 21:54, Zorro Lang wrote: > On Thu, Apr 20, 2023 at 10:11:06AM +0800, Gao Xiang wrote: >> Newer glibc such as glibc 2.36 also defines 'struct mount_attr' >> in addition to . It will report as below when >> compiling with old linux kernel headers (without idmapped mounts, >> such as kernel-headers 5.10.134) but with newer glibc (here checked >> with glibc 2.36.6): >> >> [CC] detached_mounts_propagation >> In file included from detached_mounts_propagation.c:29: >> vfs/missing.h:115:8: error: redefinition of 'struct mount_attr' >> 115 | struct mount_attr { >> | ^~~~~~~~~~ >> In file included from detached_mounts_propagation.c:23: >> /usr/include/sys/mount.h:210:8: note: originally defined here >> 210 | struct mount_attr >> | ^~~~~~~~~~ >> gmake[3]: *** [Makefile:102: detached_mounts_propagation] Error 1 >> gmake[2]: *** [include/buildrules:31: src] Error 2 >> make[1]: *** [Makefile:51: default] Error 2 >> make: *** [Makefile:49: default] Error 2 >> >> Let's get rid of as Christian suggested to avoid >> potential incompatibility between these two headers. >> >> Cc: Christian Brauner >> Cc: "Yang Xu (Fujitsu)" >> Signed-off-by: Gao Xiang >> --- > > This patch works on latest fedora, but it cause fstests can't be built on > RHEL-8 [1] and RHEL-9 [2], please check this incompatible problem. I don't have RHEL-8 and RHEL-9 enviroment at hand, it seems that #include in vfs/missing.h and xfs/xfs.h uapi header file causes more conflicts now. So could you help check if _updating configure.ac only_ resolves the problem? I will recheck this as well. Thanks, Gao Xiang > > Thanks, > Zorro > > [1] > ... > Building include > Building lib > Building ltp > Building src > [CC] feature > In file included from /usr/include/linux/fs.h:18, > from /usr/include/xfs/linux.h:30, > from /usr/include/xfs/xfs.h:9, > from global.h:13, > from feature.c:29: > /usr/include/sys/mount.h:35:3: error: expected identifier before numeric constant > MS_RDONLY = 1, /* Mount read-only. */ > ^~~~~~~~~ > gmake[2]: *** [Makefile:112: feature] Error 1 > gmake[1]: *** [include/buildrules:31: src] Error 2 > make: *** [Makefile:51: default] Error 2 > > [2] > ... > Building include > Building lib > Building ltp > Building src > [CC] feature > In file included from /usr/include/linux/fs.h:19, > from /usr/include/xfs/linux.h:36, > from /usr/include/xfs/xfs.h:9, > from global.h:13, > from feature.c:29: > /usr/include/sys/mount.h:35:3: error: expected identifier before numeric constant > 35 | MS_RDONLY = 1, /* Mount read-only. */ > | ^~~~~~~~~ > In file included from feature.c:49: > vfs/missing.h:116:8: error: redefinition of 'struct mount_attr' > 116 | struct mount_attr { > | ^~~~~~~~~~ > In file included from /usr/include/linux/fs.h:19, > from /usr/include/xfs/linux.h:36, > from /usr/include/xfs/xfs.h:9, > from global.h:13, > from feature.c:29: > /usr/include/linux/mount.h:128:8: note: originally defined here > 128 | struct mount_attr { > | ^~~~~~~~~~ > gmake[2]: *** [Makefile:112: feature] Error 1 > gmake[1]: *** [include/buildrules:31: src] Error 2 > make: *** [Makefile:51: default] Error 2 > > >> configure.ac | 2 +- >> src/detached_mounts_propagation.c | 1 - >> src/vfs/missing.h | 1 + >> 3 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index 4687d8a3..bb29f37e 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -74,7 +74,7 @@ AC_HAVE_FIEXCHANGE >> >> AC_CHECK_FUNCS([renameat2]) >> AC_CHECK_FUNCS([reallocarray]) >> -AC_CHECK_TYPES([struct mount_attr], [], [], [[#include ]]) >> +AC_CHECK_TYPES([struct mount_attr], [], [], [[#include ]]) >> AC_CHECK_TYPES([struct btrfs_qgroup_limit], [], [], [[ >> #include >> #include >> diff --git a/src/detached_mounts_propagation.c b/src/detached_mounts_propagation.c >> index 17db2c02..4041c75f 100644 >> --- a/src/detached_mounts_propagation.c >> +++ b/src/detached_mounts_propagation.c >> @@ -20,7 +20,6 @@ >> #include >> #include >> #include >> -#include >> #include >> #include >> #include >> diff --git a/src/vfs/missing.h b/src/vfs/missing.h >> index 059e742d..04ab33d1 100644 >> --- a/src/vfs/missing.h >> +++ b/src/vfs/missing.h >> @@ -18,6 +18,7 @@ >> #include >> #include >> #include >> +#include >> >> #ifndef __NR_mount_setattr >> #if defined __alpha__ >> -- >> 2.24.4 >>