* [PATCH v2 1/2] driver core: use NUMA_NO_NODE during device_initialize
@ 2021-10-03 9:13 Max Gurtovoy
2021-10-03 9:13 ` [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro Max Gurtovoy
0 siblings, 1 reply; 5+ messages in thread
From: Max Gurtovoy @ 2021-10-03 9:13 UTC (permalink / raw)
To: hch, linux-kernel, gregkh; +Cc: stefanha, oren, linux-pci, kw, Max Gurtovoy
Don't use (-1) constant for setting initial device node. Instead, use
the generic NUMA_NO_NODE definition to indicate that "no node id
specified".
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
---
changes from v1:
- added patch 2/2 (Krzysztof)
- added Reviewed-by signature (Stefan)
---
drivers/base/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/base/core.c b/drivers/base/core.c
index e65dd803a453..2b4b46f6c676 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -2838,7 +2838,7 @@ void device_initialize(struct device *dev)
spin_lock_init(&dev->devres_lock);
INIT_LIST_HEAD(&dev->devres_head);
device_pm_init(dev);
- set_dev_node(dev, -1);
+ set_dev_node(dev, NUMA_NO_NODE);
#ifdef CONFIG_GENERIC_MSI_IRQ
raw_spin_lock_init(&dev->msi_lock);
INIT_LIST_HEAD(&dev->msi_list);
--
2.18.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro
2021-10-03 9:13 [PATCH v2 1/2] driver core: use NUMA_NO_NODE during device_initialize Max Gurtovoy
@ 2021-10-03 9:13 ` Max Gurtovoy
2021-10-04 1:45 ` Krzysztof Wilczyński
2021-10-04 8:12 ` Stefan Hajnoczi
0 siblings, 2 replies; 5+ messages in thread
From: Max Gurtovoy @ 2021-10-03 9:13 UTC (permalink / raw)
To: hch, linux-kernel, gregkh; +Cc: stefanha, oren, linux-pci, kw, Max Gurtovoy
Use the proper macro instead of hard-coded (-1) value.
Suggested-by: Krzysztof Wilczyński <kw@linux.com>
Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
---
drivers/pci/pci-sysfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
index 7fb5cd17cc98..b21065222c87 100644
--- a/drivers/pci/pci-sysfs.c
+++ b/drivers/pci/pci-sysfs.c
@@ -81,8 +81,8 @@ static ssize_t pci_dev_show_local_cpu(struct device *dev, bool list,
const struct cpumask *mask;
#ifdef CONFIG_NUMA
- mask = (dev_to_node(dev) == -1) ? cpu_online_mask :
- cpumask_of_node(dev_to_node(dev));
+ mask = (dev_to_node(dev) == NUMA_NO_NODE) ? cpu_online_mask :
+ cpumask_of_node(dev_to_node(dev));
#else
mask = cpumask_of_pcibus(to_pci_dev(dev)->bus);
#endif
--
2.18.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro
2021-10-03 9:13 ` [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro Max Gurtovoy
@ 2021-10-04 1:45 ` Krzysztof Wilczyński
2021-10-04 19:07 ` Krzysztof Wilczyński
2021-10-04 8:12 ` Stefan Hajnoczi
1 sibling, 1 reply; 5+ messages in thread
From: Krzysztof Wilczyński @ 2021-10-04 1:45 UTC (permalink / raw)
To: Max Gurtovoy
Cc: Bjorn Helgaas, gregkh, hch, stefanha, oren, linux-pci, linux-kernel
[+cc Bjorn as he has strong code formatting preference in the PCI tree]
Hi Max,
> Use the proper macro instead of hard-coded (-1) value.
>
> Suggested-by: Krzysztof Wilczyński <kw@linux.com>
> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
Thank you for taking care of this! Much appreciated!
> ---
> drivers/pci/pci-sysfs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
> index 7fb5cd17cc98..b21065222c87 100644
> --- a/drivers/pci/pci-sysfs.c
> +++ b/drivers/pci/pci-sysfs.c
> @@ -81,8 +81,8 @@ static ssize_t pci_dev_show_local_cpu(struct device *dev, bool list,
> const struct cpumask *mask;
>
> #ifdef CONFIG_NUMA
> - mask = (dev_to_node(dev) == -1) ? cpu_online_mask :
> - cpumask_of_node(dev_to_node(dev));
> + mask = (dev_to_node(dev) == NUMA_NO_NODE) ? cpu_online_mask :
> + cpumask_of_node(dev_to_node(dev));
Oh this somewhat awkward indentation we have with this ternary now,
and so I wonder if either:
mask = (dev_to_node(dev) == NUMA_NO_NODE) ?
cpu_online_mask : cpumask_of_node(dev_to_node(dev));
Or, perhaps (yes, a few more lines):
if (dev_to_node(dev) == NUMA_NO_NODE)
mask = cpu_online_mask;
else
mask = cpumask_of_node(node);
Would be easier on the eyes, so to speak. What do you think (not a problem
if you don't want to change anything, thoguh)?
Thank you!
Reviewed-by: Krzysztof Wilczyński <kw@linux.com>
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro
2021-10-03 9:13 ` [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro Max Gurtovoy
2021-10-04 1:45 ` Krzysztof Wilczyński
@ 2021-10-04 8:12 ` Stefan Hajnoczi
1 sibling, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2021-10-04 8:12 UTC (permalink / raw)
To: Max Gurtovoy; +Cc: hch, linux-kernel, gregkh, oren, linux-pci, kw
[-- Attachment #1: Type: text/plain, Size: 383 bytes --]
On Sun, Oct 03, 2021 at 12:13:44PM +0300, Max Gurtovoy wrote:
> Use the proper macro instead of hard-coded (-1) value.
>
> Suggested-by: Krzysztof Wilczyński <kw@linux.com>
> Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
> ---
> drivers/pci/pci-sysfs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro
2021-10-04 1:45 ` Krzysztof Wilczyński
@ 2021-10-04 19:07 ` Krzysztof Wilczyński
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Wilczyński @ 2021-10-04 19:07 UTC (permalink / raw)
To: Max Gurtovoy
Cc: Bjorn Helgaas, gregkh, hch, stefanha, oren, linux-pci, linux-kernel
Hi Max,
[...]
> > #ifdef CONFIG_NUMA
> > - mask = (dev_to_node(dev) == -1) ? cpu_online_mask :
> > - cpumask_of_node(dev_to_node(dev));
> > + mask = (dev_to_node(dev) == NUMA_NO_NODE) ? cpu_online_mask :
> > + cpumask_of_node(dev_to_node(dev));
>
> Oh this somewhat awkward indentation we have with this ternary now,
> and so I wonder if either:
>
> mask = (dev_to_node(dev) == NUMA_NO_NODE) ?
> cpu_online_mask : cpumask_of_node(dev_to_node(dev));
>
> Or, perhaps (yes, a few more lines):
>
> if (dev_to_node(dev) == NUMA_NO_NODE)
> mask = cpu_online_mask;
> else
> mask = cpumask_of_node(node);
Doh! I should be cpumask_of_node(dev_to_node(dev)) in the above, of
course. Apologies! Albeit, v3 you sent looks great! Thank you!
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-10-04 19:07 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-03 9:13 [PATCH v2 1/2] driver core: use NUMA_NO_NODE during device_initialize Max Gurtovoy
2021-10-03 9:13 ` [PATCH v2 2/2] PCI/sysfs: use NUMA_NO_NODE macro Max Gurtovoy
2021-10-04 1:45 ` Krzysztof Wilczyński
2021-10-04 19:07 ` Krzysztof Wilczyński
2021-10-04 8:12 ` Stefan Hajnoczi
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.