From: Arnd Bergmann <arnd@arndb.de> To: Nicholas Piggin <npiggin@gmail.com> Cc: linux-kbuild@vger.kernel.org, linux-arch@vger.kernel.org, Michal Marek <mmarek@suse.com>, Sam Ravnborg <sam@ravnborg.org>, Stephen Rothwell <sfr@canb.auug.org.au>, Nicolas Pitre <nico@linaro.org>, Segher Boessenkool <segher@kernel.crashing.org>, Alan Modra <amodra@gmail.com>, Boris Ostrovsky <boris.ostrovsky@oracle.com>, David Vrabel <david.vrabel@citrix.com>, Juergen Gross <jgross@suse.com>, xen-devel@lists.xenproject.org Subject: [PATCH] Xen: remove -fshort-wchar gcc flag Date: Thu, 11 Aug 2016 14:39:50 +0200 [thread overview] Message-ID: <16541778.WMH6K24aDt@wuerfel> (raw) In-Reply-To: <1470910580-18458-1-git-send-email-npiggin@gmail.com> A previous patch added the --no-wchar-size-warning to the Makefile to avoid this harmless warning: arm-linux-gnueabi-ld: warning: drivers/xen/efi.o uses 2-byte wchar_t yet the output is to use 4-byte wchar_t; use of wchar_t values across objects may fail Changing kbuild to use thin archives instead of recursive linking unfortunately brings the same warning back during the final link. This time, we remove the -fshort-wchar flag that originally caused the warning, hopefully fixing the problem for good. I don't see any reason for having the flag in the first place, as the Xen code does not use wchar_t at all. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Fixes: 971a69db7dc0 ("Xen: don't warn about 2-byte wchar_t in efi") --- On Thursday, August 11, 2016 8:16:14 PM CEST Nicholas Piggin wrote: > Hi, > > I would like to submit the kbuild changes in patches 1-3 for > consideration. > > I've taken on the feedback, so thanks everybody for that. The > biggest change since last time is a more general way for > architectures to do a post-link pass in patch 3. > > On the question of whether to enable thin archives unconditionally, > I prefer to have architectures enable them as they are tested. But > I would like to see everybody moved as soon as possible and the > incremental linking removed. It would be nice to get this patch merged along with the thin archive conversion, either by merging it through the xen Tree, or by making it part of Nick's series with an Ack from the xen maintainers. diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile index 8feab810aed9..7f188b8d0c67 100644 --- a/drivers/xen/Makefile +++ b/drivers/xen/Makefile @@ -7,9 +7,6 @@ obj-y += xenbus/ nostackp := $(call cc-option, -fno-stack-protector) CFLAGS_features.o := $(nostackp) -CFLAGS_efi.o += -fshort-wchar -LDFLAGS += $(call ld-option, --no-wchar-size-warning) - dom0-$(CONFIG_ARM64) += arm-device.o dom0-$(CONFIG_PCI) += pci.o dom0-$(CONFIG_USB_SUPPORT) += dbgp.o
next prev parent reply other threads:[~2016-08-11 12:40 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-08-11 10:16 [PATCH 0/6 v2] kbuild changes, thin archives, --gc-sections Nicholas Piggin 2016-08-11 10:16 ` [PATCH 1/6] kbuild: allow architectures to use thin archives instead of ld -r Nicholas Piggin 2016-08-11 10:16 ` [PATCH 2/6] kbuild: allow archs to select link dead code/data elimination Nicholas Piggin 2016-08-11 10:16 ` [PATCH 3/6] kbuild: add arch specific post-link pass Nicholas Piggin 2016-08-16 9:23 ` Michal Marek 2016-08-17 4:43 ` Nicholas Piggin 2016-08-17 9:05 ` Michal Marek 2016-08-17 9:56 ` Sam Ravnborg 2016-08-18 0:17 ` Nicholas Piggin 2016-08-11 10:16 ` [PATCH 4/6] powerpc: switch to using thin archives Nicholas Piggin 2016-08-11 10:16 ` [PATCH 5/6] powerpc/64: use linker dead code elimination Nicholas Piggin 2016-08-11 10:16 ` [PATCH 6/6] powerpc: use the new post-link pass to check relocations Nicholas Piggin 2016-08-11 12:31 ` [PATCH 0/6 v2] kbuild changes, thin archives, --gc-sections Stephen Rothwell 2016-08-11 12:39 ` Arnd Bergmann [this message] 2016-08-11 12:51 ` [PATCH] Xen: remove -fshort-wchar gcc flag Jan Beulich 2016-08-11 12:51 ` [Xen-devel] " Jan Beulich 2016-08-11 14:01 ` Arnd Bergmann 2016-08-11 14:06 ` Jan Beulich 2016-08-11 14:06 ` Jan Beulich 2016-08-11 14:01 ` Arnd Bergmann 2016-08-24 17:18 ` [Xen-devel] " David Vrabel 2016-08-24 17:18 ` David Vrabel 2016-08-11 12:39 ` Arnd Bergmann 2016-08-11 13:55 ` [EXPERIMENTAL] enable thin archives and --gc-sections on ARM Arnd Bergmann 2016-08-11 20:01 ` Nicolas Pitre 2016-08-11 20:37 ` Arnd Bergmann 2016-08-11 21:05 ` Nicolas Pitre 2016-08-23 6:17 ` [PATCH 0/6 v2] kbuild changes, thin archives, --gc-sections Nicholas Piggin
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=16541778.WMH6K24aDt@wuerfel \ --to=arnd@arndb.de \ --cc=amodra@gmail.com \ --cc=boris.ostrovsky@oracle.com \ --cc=david.vrabel@citrix.com \ --cc=jgross@suse.com \ --cc=linux-arch@vger.kernel.org \ --cc=linux-kbuild@vger.kernel.org \ --cc=mmarek@suse.com \ --cc=nico@linaro.org \ --cc=npiggin@gmail.com \ --cc=sam@ravnborg.org \ --cc=segher@kernel.crashing.org \ --cc=sfr@canb.auug.org.au \ --cc=xen-devel@lists.xenproject.org \ --subject='Re: [PATCH] Xen: remove -fshort-wchar gcc flag' \ /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
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.