On 19.09.19 16:22, David Hildenbrand wrote:> Will be needed by virtio-mem to identify the node from a pxm. > > Cc: "Rafael J. Wysocki" <rjw at rjwysocki.net> > Cc: Len Brown <lenb at kernel.org> > Cc: linux-acpi at vger.kernel.org > Signed-off-by: David Hildenbrand <david at redhat.com> > --- > drivers/acpi/numa.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c > index eadbf90e65d1..d5847fa7ac69 100644 > --- a/drivers/acpi/numa.c > +++ b/drivers/acpi/numa.c > @@ -35,6 +35,7 @@ int pxm_to_node(int pxm) > return NUMA_NO_NODE; > return pxm_to_node_map[pxm]; > } > +EXPORT_SYMBOL(pxm_to_node);FWIW, this is a fairly old patch I dragged along and I think I'll convert this to EXPORT_SYMBOL_GPL now that I know better :)> > int node_to_pxm(int node) > { >-- Thanks, David / dhildenb
On Mon 23-09-19 12:13:11, David Hildenbrand wrote:> On 19.09.19 16:22, David Hildenbrand wrote: > > Will be needed by virtio-mem to identify the node from a pxm. > > > > Cc: "Rafael J. Wysocki" <rjw at rjwysocki.net> > > Cc: Len Brown <lenb at kernel.org> > > Cc: linux-acpi at vger.kernel.org > > Signed-off-by: David Hildenbrand <david at redhat.com> > > --- > > drivers/acpi/numa.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c > > index eadbf90e65d1..d5847fa7ac69 100644 > > --- a/drivers/acpi/numa.c > > +++ b/drivers/acpi/numa.c > > @@ -35,6 +35,7 @@ int pxm_to_node(int pxm) > > return NUMA_NO_NODE; > > return pxm_to_node_map[pxm]; > > } > > +EXPORT_SYMBOL(pxm_to_node); > > FWIW, this is a fairly old patch I dragged along and I think I'll > convert this to EXPORT_SYMBOL_GPL now that I know better :)All other exports in this file are EXPORT_SYMBOL. Why is this one considered special to restrict the access? -- Michal Hocko SUSE Labs
On 23.09.19 12:36, Michal Hocko wrote:> On Mon 23-09-19 12:13:11, David Hildenbrand wrote: >> On 19.09.19 16:22, David Hildenbrand wrote: >>> Will be needed by virtio-mem to identify the node from a pxm. >>> >>> Cc: "Rafael J. Wysocki" <rjw at rjwysocki.net> >>> Cc: Len Brown <lenb at kernel.org> >>> Cc: linux-acpi at vger.kernel.org >>> Signed-off-by: David Hildenbrand <david at redhat.com> >>> --- >>> drivers/acpi/numa.c | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c >>> index eadbf90e65d1..d5847fa7ac69 100644 >>> --- a/drivers/acpi/numa.c >>> +++ b/drivers/acpi/numa.c >>> @@ -35,6 +35,7 @@ int pxm_to_node(int pxm) >>> return NUMA_NO_NODE; >>> return pxm_to_node_map[pxm]; >>> } >>> +EXPORT_SYMBOL(pxm_to_node); >> >> FWIW, this is a fairly old patch I dragged along and I think I'll >> convert this to EXPORT_SYMBOL_GPL now that I know better :) > > All other exports in this file are EXPORT_SYMBOL. Why is this one > considered special to restrict the access? >Uh, so I did actually use my brain two years ago back when I was crafting this patch :) -- Thanks, David / dhildenb