| Message ID | 20191212171137.13872-2-david@redhat.com (mailing list archive) |
|---|---|
| State | RFC, archived |
| Headers | show |
| Series | [RFC,v4,01/13] ACPI: NUMA: export pxm_to_node | expand |
On Thursday, December 12, 2019 6:11:25 PM CET David Hildenbrand wrote: > Will be needed by virtio-mem to identify the node from a pxm. > > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Cc: Len Brown <lenb@kernel.org> > Cc: linux-acpi@vger.kernel.org > Signed-off-by: David Hildenbrand <david@redhat.com> > --- > drivers/acpi/numa/srat.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c > index eadbf90e65d1..d5847fa7ac69 100644 > --- a/drivers/acpi/numa/srat.c > +++ b/drivers/acpi/numa/srat.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); > > int node_to_pxm(int node) > { > This is fine by me FWIW.
On 12.12.19 22:43, Rafael J. Wysocki wrote: > On Thursday, December 12, 2019 6:11:25 PM CET David Hildenbrand wrote: >> Will be needed by virtio-mem to identify the node from a pxm. >> >> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> >> Cc: Len Brown <lenb@kernel.org> >> Cc: linux-acpi@vger.kernel.org >> Signed-off-by: David Hildenbrand <david@redhat.com> >> --- >> drivers/acpi/numa/srat.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c >> index eadbf90e65d1..d5847fa7ac69 100644 >> --- a/drivers/acpi/numa/srat.c >> +++ b/drivers/acpi/numa/srat.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); >> >> int node_to_pxm(int node) >> { >> > > This is fine by me FWIW. Can I count that as an Acked-by and carry it along? Thanks!
On Fri, Dec 13, 2019 at 10:41 AM David Hildenbrand <david@redhat.com> wrote: > > On 12.12.19 22:43, Rafael J. Wysocki wrote: > > On Thursday, December 12, 2019 6:11:25 PM CET David Hildenbrand wrote: > >> Will be needed by virtio-mem to identify the node from a pxm. > >> > >> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > >> Cc: Len Brown <lenb@kernel.org> > >> Cc: linux-acpi@vger.kernel.org > >> Signed-off-by: David Hildenbrand <david@redhat.com> > >> --- > >> drivers/acpi/numa/srat.c | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c > >> index eadbf90e65d1..d5847fa7ac69 100644 > >> --- a/drivers/acpi/numa/srat.c > >> +++ b/drivers/acpi/numa/srat.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); > >> > >> int node_to_pxm(int node) > >> { > >> > > > > This is fine by me FWIW. > > Can I count that as an Acked-by and carry it along? Thanks! Yes, please.
diff --git a/drivers/acpi/numa/srat.c b/drivers/acpi/numa/srat.c index eadbf90e65d1..d5847fa7ac69 100644 --- a/drivers/acpi/numa/srat.c +++ b/drivers/acpi/numa/srat.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); int node_to_pxm(int node) {
Will be needed by virtio-mem to identify the node from a pxm. Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> Cc: Len Brown <lenb@kernel.org> Cc: linux-acpi@vger.kernel.org Signed-off-by: David Hildenbrand <david@redhat.com> --- drivers/acpi/numa/srat.c | 1 + 1 file changed, 1 insertion(+)