* [Buildroot] [PATCH] package/erofs-utils: add patch to resolve PAGE_SIZE redefinition [not found] <20200326021215.23749-1-hsiangkao.ref@aol.com> @ 2020-03-26 2:12 ` Gao Xiang 2020-03-28 2:43 ` Gao Xiang 0 siblings, 1 reply; 3+ messages in thread From: Gao Xiang @ 2020-03-26 2:12 UTC (permalink / raw) To: buildroot This patch resolves PAGE_SIZE redefinition reported by autobuild [1] with some configrations on i586 platform. As what I said in the patch, I didn't get such error report before. [1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 Signed-off-by: Gao Xiang <hsiangkao@aol.com> --- Hi all, I have no simple way to verify the whole compilation with such configuration due to poor Internet quality to download all necessary packages in China mainland. I've just verified with my own old configuration and it has no impact. Hopefully it's the last patch for all configration to work. Thanks, Gao Xiang ...s-utils-avoid-PAGE_SIZE-redefinition.patch | 57 +++++++++++++++++++ 1 file changed, 57 insertions(+) create mode 100644 package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch diff --git a/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch new file mode 100644 index 0000000000..2441a841ba --- /dev/null +++ b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch @@ -0,0 +1,57 @@ +From 6c087a91c968197b5a95578fc645617947c943ad Mon Sep 17 00:00:00 2001 +From: Gao Xiang <hsiangkao@aol.com> +Date: Wed, 25 Mar 2020 15:42:22 +0800 +Subject: [PATCH] erofs-utils: avoid PAGE_SIZE redefinition + +Buildroot autobuild reported a PAGE_SIZE redefinition with some +configrations on i586 toolchain [1] (I didn't notice such report +from erofs-utils travis CI or distribution builds before.) + +In file included from config.c:11: +../include/erofs/internal.h:27: error: "PAGE_SIZE" redefined [-Werror] + #define PAGE_SIZE (1U << PAGE_SHIFT) + +In file included from ../include/erofs/defs.h:17, + from ../include/erofs/config.h:12, + from ../include/erofs/print.h:12, + from config.c:10: +.../sysroot/usr/include/limits.h:89: note: this is the location of the previous definition + #define PAGE_SIZE PAGESIZE + +cc1: all warnings being treated as errors + +Fix it now. + +[1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 +Link: https://lore.kernel.org/r/20200325082930.2025-1-hsiangkao at aol.com +Signed-off-by: Gao Xiang <hsiangkao@aol.com> +--- + include/erofs/internal.h | 10 ++++++++++ + 1 file changed, 10 insertions(+) + +diff --git a/include/erofs/internal.h b/include/erofs/internal.h +index e7d5a64..41da189 100644 +--- a/include/erofs/internal.h ++++ b/include/erofs/internal.h +@@ -23,8 +23,18 @@ typedef unsigned short umode_t; + #define PATH_MAX 4096 /* # chars in a path name including nul */ + #endif + ++#ifndef PAGE_SHIFT + #define PAGE_SHIFT (12) ++#endif ++ ++#ifndef PAGE_SIZE + #define PAGE_SIZE (1U << PAGE_SHIFT) ++#endif ++ ++/* no obvious reason to support explicit PAGE_SIZE != 4096 for now */ ++#if PAGE_SIZE != 4096 ++#error incompatible PAGE_SIZE is already defined ++#endif + + #define LOG_BLOCK_SIZE (12) + #define EROFS_BLKSIZ (1U << LOG_BLOCK_SIZE) +-- +2.20.1 + -- 2.20.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] package/erofs-utils: add patch to resolve PAGE_SIZE redefinition 2020-03-26 2:12 ` [Buildroot] [PATCH] package/erofs-utils: add patch to resolve PAGE_SIZE redefinition Gao Xiang @ 2020-03-28 2:43 ` Gao Xiang 2020-03-28 8:45 ` Yann E. MORIN 0 siblings, 1 reply; 3+ messages in thread From: Gao Xiang @ 2020-03-28 2:43 UTC (permalink / raw) To: buildroot Hi all, On Thu, Mar 26, 2020 at 10:12:15AM +0800, Gao Xiang wrote: > This patch resolves PAGE_SIZE redefinition reported by autobuild [1] > with some configrations on i586 platform. > > As what I said in the patch, I didn't get such error report before. > > [1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 > Signed-off-by: Gao Xiang <hsiangkao@aol.com> > --- > Hi all, I received 2 more autobuild reports due to the same cause. I have no clue which configration impacts, but this patch addresses these reports. x86_64 | erofs-utils-1.0 | http://autobuild.buildroot.net/results/42cd24535ab38cb9b416b730a034a1dbe3293bf5 x86_64 | erofs-utils-1.0 | http://autobuild.buildroot.net/results/260cdb3203e9141e674f38a2acd127d10320f8fa Please kindly consider this. I will keep track of these all. Thanks, Gao Xiang > > I have no simple way to verify the whole compilation with such > configuration due to poor Internet quality to download all > necessary packages in China mainland. > > I've just verified with my own old configuration and it has no > impact. Hopefully it's the last patch for all configration to > work. > > Thanks, > Gao Xiang > > ...s-utils-avoid-PAGE_SIZE-redefinition.patch | 57 +++++++++++++++++++ > 1 file changed, 57 insertions(+) > create mode 100644 package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > > diff --git a/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > new file mode 100644 > index 0000000000..2441a841ba > --- /dev/null > +++ b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > @@ -0,0 +1,57 @@ > +From 6c087a91c968197b5a95578fc645617947c943ad Mon Sep 17 00:00:00 2001 > +From: Gao Xiang <hsiangkao@aol.com> > +Date: Wed, 25 Mar 2020 15:42:22 +0800 > +Subject: [PATCH] erofs-utils: avoid PAGE_SIZE redefinition > + > +Buildroot autobuild reported a PAGE_SIZE redefinition with some > +configrations on i586 toolchain [1] (I didn't notice such report > +from erofs-utils travis CI or distribution builds before.) > + > +In file included from config.c:11: > +../include/erofs/internal.h:27: error: "PAGE_SIZE" redefined [-Werror] > + #define PAGE_SIZE (1U << PAGE_SHIFT) > + > +In file included from ../include/erofs/defs.h:17, > + from ../include/erofs/config.h:12, > + from ../include/erofs/print.h:12, > + from config.c:10: > +.../sysroot/usr/include/limits.h:89: note: this is the location of the previous definition > + #define PAGE_SIZE PAGESIZE > + > +cc1: all warnings being treated as errors > + > +Fix it now. > + > +[1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 > +Link: https://lore.kernel.org/r/20200325082930.2025-1-hsiangkao at aol.com > +Signed-off-by: Gao Xiang <hsiangkao@aol.com> > +--- > + include/erofs/internal.h | 10 ++++++++++ > + 1 file changed, 10 insertions(+) > + > +diff --git a/include/erofs/internal.h b/include/erofs/internal.h > +index e7d5a64..41da189 100644 > +--- a/include/erofs/internal.h > ++++ b/include/erofs/internal.h > +@@ -23,8 +23,18 @@ typedef unsigned short umode_t; > + #define PATH_MAX 4096 /* # chars in a path name including nul */ > + #endif > + > ++#ifndef PAGE_SHIFT > + #define PAGE_SHIFT (12) > ++#endif > ++ > ++#ifndef PAGE_SIZE > + #define PAGE_SIZE (1U << PAGE_SHIFT) > ++#endif > ++ > ++/* no obvious reason to support explicit PAGE_SIZE != 4096 for now */ > ++#if PAGE_SIZE != 4096 > ++#error incompatible PAGE_SIZE is already defined > ++#endif > + > + #define LOG_BLOCK_SIZE (12) > + #define EROFS_BLKSIZ (1U << LOG_BLOCK_SIZE) > +-- > +2.20.1 > + > -- > 2.20.1 > ^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] package/erofs-utils: add patch to resolve PAGE_SIZE redefinition 2020-03-28 2:43 ` Gao Xiang @ 2020-03-28 8:45 ` Yann E. MORIN 0 siblings, 0 replies; 3+ messages in thread From: Yann E. MORIN @ 2020-03-28 8:45 UTC (permalink / raw) To: buildroot Gao, All, On 2020-03-28 10:43 +0800, Gao Xiang spake thusly: > On Thu, Mar 26, 2020 at 10:12:15AM +0800, Gao Xiang wrote: > > This patch resolves PAGE_SIZE redefinition reported by autobuild [1] > > with some configrations on i586 platform. > > > > As what I said in the patch, I didn't get such error report before. > > > > [1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 > > Signed-off-by: Gao Xiang <hsiangkao@aol.com> > > --- > > Hi all, > > I received 2 more autobuild reports due to the same cause. > > I have no clue which configration impacts, but this patch > addresses these reports. > > x86_64 | erofs-utils-1.0 | http://autobuild.buildroot.net/results/42cd24535ab38cb9b416b730a034a1dbe3293bf5 > x86_64 | erofs-utils-1.0 | http://autobuild.buildroot.net/results/260cdb3203e9141e674f38a2acd127d10320f8fa The comon denominators for those three builds, is that they all are x86 (32 or 64), and that the toolchain is based on the musl libc. > Please kindly consider this. I will keep track of these all. Applied to master, thanks. Regards, Yann E. MORIN. > > Thanks, > Gao Xiang > > > > > I have no simple way to verify the whole compilation with such > > configuration due to poor Internet quality to download all > > necessary packages in China mainland. > > > > I've just verified with my own old configuration and it has no > > impact. Hopefully it's the last patch for all configration to > > work. > > > > Thanks, > > Gao Xiang > > > > ...s-utils-avoid-PAGE_SIZE-redefinition.patch | 57 +++++++++++++++++++ > > 1 file changed, 57 insertions(+) > > create mode 100644 package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > > > > diff --git a/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > > new file mode 100644 > > index 0000000000..2441a841ba > > --- /dev/null > > +++ b/package/erofs-utils/0004-erofs-utils-avoid-PAGE_SIZE-redefinition.patch > > @@ -0,0 +1,57 @@ > > +From 6c087a91c968197b5a95578fc645617947c943ad Mon Sep 17 00:00:00 2001 > > +From: Gao Xiang <hsiangkao@aol.com> > > +Date: Wed, 25 Mar 2020 15:42:22 +0800 > > +Subject: [PATCH] erofs-utils: avoid PAGE_SIZE redefinition > > + > > +Buildroot autobuild reported a PAGE_SIZE redefinition with some > > +configrations on i586 toolchain [1] (I didn't notice such report > > +from erofs-utils travis CI or distribution builds before.) > > + > > +In file included from config.c:11: > > +../include/erofs/internal.h:27: error: "PAGE_SIZE" redefined [-Werror] > > + #define PAGE_SIZE (1U << PAGE_SHIFT) > > + > > +In file included from ../include/erofs/defs.h:17, > > + from ../include/erofs/config.h:12, > > + from ../include/erofs/print.h:12, > > + from config.c:10: > > +.../sysroot/usr/include/limits.h:89: note: this is the location of the previous definition > > + #define PAGE_SIZE PAGESIZE > > + > > +cc1: all warnings being treated as errors > > + > > +Fix it now. > > + > > +[1] http://autobuild.buildroot.net/results/340b98caa45bafd43f109002be9da59ba7f6d971 > > +Link: https://lore.kernel.org/r/20200325082930.2025-1-hsiangkao at aol.com > > +Signed-off-by: Gao Xiang <hsiangkao@aol.com> > > +--- > > + include/erofs/internal.h | 10 ++++++++++ > > + 1 file changed, 10 insertions(+) > > + > > +diff --git a/include/erofs/internal.h b/include/erofs/internal.h > > +index e7d5a64..41da189 100644 > > +--- a/include/erofs/internal.h > > ++++ b/include/erofs/internal.h > > +@@ -23,8 +23,18 @@ typedef unsigned short umode_t; > > + #define PATH_MAX 4096 /* # chars in a path name including nul */ > > + #endif > > + > > ++#ifndef PAGE_SHIFT > > + #define PAGE_SHIFT (12) > > ++#endif > > ++ > > ++#ifndef PAGE_SIZE > > + #define PAGE_SIZE (1U << PAGE_SHIFT) > > ++#endif > > ++ > > ++/* no obvious reason to support explicit PAGE_SIZE != 4096 for now */ > > ++#if PAGE_SIZE != 4096 > > ++#error incompatible PAGE_SIZE is already defined > > ++#endif > > + > > + #define LOG_BLOCK_SIZE (12) > > + #define EROFS_BLKSIZ (1U << LOG_BLOCK_SIZE) > > +-- > > +2.20.1 > > + > > -- > > 2.20.1 > > -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-28 8:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20200326021215.23749-1-hsiangkao.ref@aol.com>
2020-03-26 2:12 ` [Buildroot] [PATCH] package/erofs-utils: add patch to resolve PAGE_SIZE redefinition Gao Xiang
2020-03-28 2:43 ` Gao Xiang
2020-03-28 8:45 ` Yann E. MORIN
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox