All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid()
@ 2019-05-28 16:14 Tony Luck
  2019-05-28 16:20 ` Linus Torvalds
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Tony Luck @ 2019-05-28 16:14 UTC (permalink / raw)
  To: linux-ia64

From: Randy Dunlap <rdunlap@infradead.org>

Fix build errors on ia64 when DISCONTIGMEM=y and NUMA=y by
exporting paddr_to_nid().

Fixes these build errors:

ERROR: "paddr_to_nid" [sound/core/snd-pcm.ko] undefined!
ERROR: "paddr_to_nid" [net/sunrpc/sunrpc.ko] undefined!
ERROR: "paddr_to_nid" [fs/cifs/cifs.ko] undefined!
ERROR: "paddr_to_nid" [drivers/video/fbdev/core/fb.ko] undefined!
ERROR: "paddr_to_nid" [drivers/usb/mon/usbmon.ko] undefined!
ERROR: "paddr_to_nid" [drivers/usb/core/usbcore.ko] undefined!
ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!
ERROR: "paddr_to_nid" [drivers/md/dm-mod.ko] undefined!
ERROR: "paddr_to_nid" [drivers/md/dm-crypt.ko] undefined!
ERROR: "paddr_to_nid" [drivers/md/dm-bufio.ko] undefined!
ERROR: "paddr_to_nid" [drivers/ide/ide-core.ko] undefined!
ERROR: "paddr_to_nid" [drivers/ide/ide-cd_mod.ko] undefined!
ERROR: "paddr_to_nid" [drivers/gpu/drm/drm.ko] undefined!
ERROR: "paddr_to_nid" [drivers/char/agp/agpgart.ko] undefined!
ERROR: "paddr_to_nid" [drivers/block/nbd.ko] undefined!
ERROR: "paddr_to_nid" [drivers/block/loop.ko] undefined!
ERROR: "paddr_to_nid" [drivers/block/brd.ko] undefined!
ERROR: "paddr_to_nid" [crypto/ccm.ko] undefined!

Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: linux-ia64@vger.kernel.org
Signed-off-by: Tony Luck <tony.luck@intel.com>
---
 arch/ia64/mm/numa.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/ia64/mm/numa.c b/arch/ia64/mm/numa.c
index a03803506b0c..5e1015eb6d0d 100644
--- a/arch/ia64/mm/numa.c
+++ b/arch/ia64/mm/numa.c
@@ -55,6 +55,7 @@ paddr_to_nid(unsigned long paddr)
 
 	return (i < num_node_memblks) ? node_memblk[i].nid : (num_node_memblks ? -1 : 0);
 }
+EXPORT_SYMBOL(paddr_to_nid);
 
 #if defined(CONFIG_SPARSEMEM) && defined(CONFIG_NUMA)
 /*
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid()
  2019-05-28 16:14 [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid() Tony Luck
@ 2019-05-28 16:20 ` Linus Torvalds
  2019-05-28 16:39 ` Luck, Tony
  2019-05-28 16:56 ` Randy Dunlap
  2 siblings, 0 replies; 4+ messages in thread
From: Linus Torvalds @ 2019-05-28 16:20 UTC (permalink / raw)
  To: linux-ia64

On Tue, May 28, 2019 at 9:14 AM Tony Luck <tony.luck@intel.com> wrote:
>
> Fix build errors on ia64 when DISCONTIGMEM=y and NUMA=y by
> exporting paddr_to_nid().

Applied.

Just out of curiosity, what caused this error? It doesn't _look_ new,
just judging from the patch itself. Is this just a configuration that
nobody has noticed before, or something else that changed that caused
it to happen now?

                Linus

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid()
  2019-05-28 16:14 [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid() Tony Luck
  2019-05-28 16:20 ` Linus Torvalds
@ 2019-05-28 16:39 ` Luck, Tony
  2019-05-28 16:56 ` Randy Dunlap
  2 siblings, 0 replies; 4+ messages in thread
From: Luck, Tony @ 2019-05-28 16:39 UTC (permalink / raw)
  To: linux-ia64

PiBKdXN0IG91dCBvZiBjdXJpb3NpdHksIHdoYXQgY2F1c2VkIHRoaXMgZXJyb3I/IEl0IGRvZXNu
J3QgX2xvb2tfIG5ldywNCj4ganVzdCBqdWRnaW5nIGZyb20gdGhlIHBhdGNoIGl0c2VsZi4gSXMg
dGhpcyBqdXN0IGEgY29uZmlndXJhdGlvbiB0aGF0DQo+IG5vYm9keSBoYXMgbm90aWNlZCBiZWZv
cmUsIG9yIHNvbWV0aGluZyBlbHNlIHRoYXQgY2hhbmdlZCB0aGF0IGNhdXNlZA0KPiBpdCB0byBo
YXBwZW4gbm93Pw0KDQpJJ20gY29uZnVzZWQgYnkgdGhlIGVycm9yIGJlY2F1c2UgSSBkb24ndCBz
ZWUgaXQgdXNpbmcgYW4gb2xkZXIgKDQuNi40KSB2ZXJzaW9uDQpvZiBnY2MuIEJ1dCBSYW5keSBp
cyBidWlsZGluZyB3aXRoIHNvbWV0aGluZyBtb2Rlcm4gYW5kIHJ1bnMgaW50byBpdC4NCg0KU28g
c29tZSBhZ2dyZXNzaXZlIG9wdGltaXphdGlvbi9pbmxpbmU/DQoNCi1Ub255DQo

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid()
  2019-05-28 16:14 [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid() Tony Luck
  2019-05-28 16:20 ` Linus Torvalds
  2019-05-28 16:39 ` Luck, Tony
@ 2019-05-28 16:56 ` Randy Dunlap
  2 siblings, 0 replies; 4+ messages in thread
From: Randy Dunlap @ 2019-05-28 16:56 UTC (permalink / raw)
  To: linux-ia64

On 5/28/19 9:39 AM, Luck, Tony wrote:
>> Just out of curiosity, what caused this error? It doesn't _look_ new,
>> just judging from the patch itself. Is this just a configuration that
>> nobody has noticed before, or something else that changed that caused
>> it to happen now?
> 
> I'm confused by the error because I don't see it using an older (4.6.4) version
> of gcc. But Randy is building with something modern and runs into it.
> 
> So some aggressive optimization/inline?
> 
> -Tony

kbuild test robot reported the build error.  It uses gcc 7.x.
I am using gcc 8.1 from kernel.org crosstools.

-- 
~Randy

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-05-28 16:56 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-28 16:14 [PATCH - Linus please apply] ia64: fix build errors by exporting paddr_to_nid() Tony Luck
2019-05-28 16:20 ` Linus Torvalds
2019-05-28 16:39 ` Luck, Tony
2019-05-28 16:56 ` Randy Dunlap

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.