From: Thomas Monjalon <email@example.com> To: "Phil Yang (Arm Technology China)" <Phil.Yang@arm.com> Cc: Bruce Richardson <firstname.lastname@example.org>, "email@example.com" <firstname.lastname@example.org>, "email@example.com" <firstname.lastname@example.org>, "email@example.com" <firstname.lastname@example.org>, Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>, "Gavin Hu (Arm Technology China)" <Gavin.Hu@arm.com>, "Joyce Kong (Arm Technology China)" <Joyce.Kong@arm.com>, nd <email@example.com>, "firstname.lastname@example.org" <email@example.com> Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH 1/2] mk: fix unsupported flag error on armhf architercture Date: Tue, 12 Nov 2019 07:03:18 +0100 Message-ID: <3677391.EzyCDiKdWj@xps> (raw) In-Reply-To: <VE1PR08MB46405B2EBCE4D942936ABF11E9770@VE1PR08MB4640.eurprd08.prod.outlook.com> 12/11/2019 06:25, Phil Yang (Arm Technology China): > From: Bruce Richardson <firstname.lastname@example.org> > > On Mon, Nov 11, 2019 at 06:34:20PM +0800, Phil Yang wrote: > > > The older version (e.g. version 7.4.0 ) of GNU C compiler for the armhf > > > architecture doesn't support the flag '-Wno-address-of-packed-member', > > > so remove this flag for aarch32. > > > > > > Fixes: a385972c3675 ("mk: disable warning for packed member pointer") > > > Cc: email@example.com > > > > > > Signed-off-by: Phil Yang <firstname.lastname@example.org> > > > Reviewed-by: Gavin Hu <email@example.com> > > > Tested-by: Joyce Kong <firstname.lastname@example.org> > > > > > > --- > > > mk/toolchain/gcc/rte.vars.mk | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/mk/toolchain/gcc/rte.vars.mk b/mk/toolchain/gcc/rte.vars.mk > > > index 9fc7041..ec0cbbf 100644 > > > --- a/mk/toolchain/gcc/rte.vars.mk > > > +++ b/mk/toolchain/gcc/rte.vars.mk > > > @@ -100,7 +100,9 @@ WERROR_FLAGS += -Wno-format-truncation > > > endif > > > > > > # disable packed member unalign warnings > > > +ifneq ($(CONFIG_RTE_ARCH_ARM), y) > > > WERROR_FLAGS += -Wno-address-of-packed-member > > > +endif > > > > > You don't need to do this, as gcc will not complain about this unknown flag > > unless you have other issues in your code. I think it's better to keep the > > code clean in this case, otherwise we'll have the code littered with > > conditionals for various flags. > > > > /Bruce > > > >  https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html > > "When an unrecognized warning option is requested (e.g., > > -Wunknown-warning), GCC emits a diagnostic stating that the option is not > > recognized. However, if the -Wno- form is used, the behavior is slightly > > different: no diagnostic is produced for -Wno-unknown-warning unless other > > diagnostics are being produced. This allows the use of new -Wno- options > > with old compilers, but if something goes wrong, the compiler warns that an > > unrecognized option is present." > > Thanks, Bruce. > > There are thousands of warnings with alignment when compiling for ARMv7 (ARMv7 supports unaligned memory access). > Without this new flag, the build system works fine for ARMv7. So I add this conditional for ARMv7 only. > I agree with you we should clean up the code, but it needs a lot of effort to achieve that. > This patch is going to make the ARMv7 target build successfully during this period. I would go the opposite direction. I think we should remove all warnings disablement like "-Wno-something", and clean the code. If you want to allow building ARMv7, you can just allow warnings as not errors. So we will still see something is wrong but it won't prevent from building.
next prev parent reply other threads:[~2019-11-12 6:04 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-11-11 10:34 [dpdk-stable] " Phil Yang 2019-11-11 10:34 ` [dpdk-stable] [PATCH 2/2] doc: add aarch32 build guidance Phil Yang 2020-05-24 21:21 ` [dpdk-stable] [dpdk-dev] " Thomas Monjalon 2020-05-25 3:20 ` Phil Yang 2020-05-27 8:28 ` Ruifeng Wang 2020-07-02 5:37 ` Phil Yang 2020-07-02 8:01 ` Jerin Jacob 2019-11-11 10:47 ` [dpdk-stable] [dpdk-dev] [PATCH 1/2] mk: fix unsupported flag error on armhf architercture Bruce Richardson 2019-11-12 5:25 ` Phil Yang (Arm Technology China) 2019-11-12 6:03 ` Thomas Monjalon [this message] 2019-11-12 6:40 ` Phil Yang (Arm Technology China) 2019-11-26 17:01 ` Thomas Monjalon 2019-11-27 8:09 ` Phil Yang (Arm Technology China) 2019-11-27 9:29 ` Bruce Richardson 2019-11-27 10:00 ` Phil Yang (Arm Technology China) 2019-11-27 10:32 ` Thomas Monjalon
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=3677391.EzyCDiKdWj@xps \ --email@example.com \ --cc=Gavin.Hu@arm.com \ --cc=Honnappa.Nagarahalli@arm.com \ --cc=Joyce.Kong@arm.com \ --cc=Phil.Yang@arm.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
patches for DPDK stable branches This inbox may be cloned and mirrored by anyone: git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \ firstname.lastname@example.org public-inbox-index stable Example config snippet for mirrors. Newsgroup available over NNTP: nntp://inbox.dpdk.org/inbox.dpdk.stable AGPL code for this site: git clone https://public-inbox.org/public-inbox.git