* [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node @ 2021-06-25 4:30 Gavin Shan 2021-06-25 3:47 ` Randy Dunlap 0 siblings, 1 reply; 5+ messages in thread From: Gavin Shan @ 2021-06-25 4:30 UTC (permalink / raw) To: devicetree; +Cc: linux-kernel, rdunlap, robh+dt, drjones, shan.gavin The empty NUMA nodes, where no memory resides in, are allowed. For these empty NUMA nodes, the 'len' of 'reg' property is zero. These empty NUMA node IDs are still valid and parsed. I finds difficulty to get where it's properly documented. So lets add note to empty NUMA nodes in the NUMA binding doc. Signed-off-by: Gavin Shan <gshan@redhat.com> --- Documentation/devicetree/bindings/numa.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/numa.txt b/Documentation/devicetree/bindings/numa.txt index 21b35053ca5a..edf728cff155 100644 --- a/Documentation/devicetree/bindings/numa.txt +++ b/Documentation/devicetree/bindings/numa.txt @@ -109,6 +109,10 @@ Example: Dual socket system consists of 2 boards connected through ccn bus and each board having one socket/soc of 8 cpus, memory and pci bus. +Note that the empty NUMA nodes, which no memory resides in, are allowed. +Their NUMA node IDs are still valid so that memory can be added into these +NUMA nodes through hotplug afterwards. + memory@c00000 { device_type = "memory"; reg = <0x0 0xc00000 0x0 0x80000000>; -- 2.23.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node 2021-06-25 4:30 [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node Gavin Shan @ 2021-06-25 3:47 ` Randy Dunlap 2021-06-25 4:32 ` Gavin Shan 0 siblings, 1 reply; 5+ messages in thread From: Randy Dunlap @ 2021-06-25 3:47 UTC (permalink / raw) To: Gavin Shan, devicetree; +Cc: linux-kernel, robh+dt, drjones, shan.gavin On 6/24/21 9:30 PM, Gavin Shan wrote: > The empty NUMA nodes, where no memory resides in, are allowed. For > these empty NUMA nodes, the 'len' of 'reg' property is zero. These > empty NUMA node IDs are still valid and parsed. I finds difficulty > to get where it's properly documented. > > So lets add note to empty NUMA nodes in the NUMA binding doc. > > Signed-off-by: Gavin Shan <gshan@redhat.com> > --- > Documentation/devicetree/bindings/numa.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/numa.txt b/Documentation/devicetree/bindings/numa.txt > index 21b35053ca5a..edf728cff155 100644 > --- a/Documentation/devicetree/bindings/numa.txt > +++ b/Documentation/devicetree/bindings/numa.txt > @@ -109,6 +109,10 @@ Example: > Dual socket system consists of 2 boards connected through ccn bus and > each board having one socket/soc of 8 cpus, memory and pci bus. > > +Note that the empty NUMA nodes, which no memory resides in, are allowed. I would write that without "the": +Note that empty NUMA nodes, which no memory resides in, are allowed. BTW, AFAIK, NUMA nodes may contain memory, CPU(s), or I/O -- any one, two, or three, without the other types of resources being present. > +Their NUMA node IDs are still valid so that memory can be added into these > +NUMA nodes through hotplug afterwards. > + > memory@c00000 { > device_type = "memory"; > reg = <0x0 0xc00000 0x0 0x80000000>; OT: is your system clock off by a couple of hours? Your emails seem to be from in the future. Thanks. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node 2021-06-25 3:47 ` Randy Dunlap @ 2021-06-25 4:32 ` Gavin Shan 2021-06-25 4:49 ` Randy Dunlap 0 siblings, 1 reply; 5+ messages in thread From: Gavin Shan @ 2021-06-25 4:32 UTC (permalink / raw) To: Randy Dunlap, devicetree; +Cc: linux-kernel, robh+dt, drjones, shan.gavin On 6/25/21 1:47 PM, Randy Dunlap wrote: > On 6/24/21 9:30 PM, Gavin Shan wrote: >> The empty NUMA nodes, where no memory resides in, are allowed. For >> these empty NUMA nodes, the 'len' of 'reg' property is zero. These >> empty NUMA node IDs are still valid and parsed. I finds difficulty >> to get where it's properly documented. >> >> So lets add note to empty NUMA nodes in the NUMA binding doc. >> >> Signed-off-by: Gavin Shan <gshan@redhat.com> >> --- >> Documentation/devicetree/bindings/numa.txt | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/numa.txt b/Documentation/devicetree/bindings/numa.txt >> index 21b35053ca5a..edf728cff155 100644 >> --- a/Documentation/devicetree/bindings/numa.txt >> +++ b/Documentation/devicetree/bindings/numa.txt >> @@ -109,6 +109,10 @@ Example: >> Dual socket system consists of 2 boards connected through ccn bus and >> each board having one socket/soc of 8 cpus, memory and pci bus. >> >> +Note that the empty NUMA nodes, which no memory resides in, are allowed. > > I would write that without "the": > > +Note that empty NUMA nodes, which no memory resides in, are allowed. > > BTW, AFAIK, NUMA nodes may contain memory, CPU(s), or I/O -- any one, two, or > three, without the other types of resources being present. > Sure, I will drop "the" in v4. The NUMA nodes here are memory nodes here. Since the NUMA node usually means memory node. I'm not sure if I change the term "NUMA node" to "NUMA memory node" in v4. If you agree, I would have something like this: Note that empty memory nodes, which no memory resides in, are allowed. The NUMA node IDs in these empty memory nodes are still valid, but memory can be added into them through hotplug afterwards. > >> +Their NUMA node IDs are still valid so that memory can be added into these >> +NUMA nodes through hotplug afterwards. >> + >> memory@c00000 { >> device_type = "memory"; >> reg = <0x0 0xc00000 0x0 0x80000000>; > > > OT: is your system clock off by a couple of hours? > Your emails seem to be from in the future. > Yeah, The local clock becomes messy after my laptop reboots. I enabled NTP and kept it running. Hopefully, the clock is precise this time. Thanks for the reminder, Randy :) Thanks, Gavin ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node 2021-06-25 4:32 ` Gavin Shan @ 2021-06-25 4:49 ` Randy Dunlap 2021-06-25 5:25 ` Gavin Shan 0 siblings, 1 reply; 5+ messages in thread From: Randy Dunlap @ 2021-06-25 4:49 UTC (permalink / raw) To: Gavin Shan, devicetree; +Cc: linux-kernel, robh+dt, drjones, shan.gavin On 6/24/21 9:32 PM, Gavin Shan wrote: > On 6/25/21 1:47 PM, Randy Dunlap wrote: >> On 6/24/21 9:30 PM, Gavin Shan wrote: >>> The empty NUMA nodes, where no memory resides in, are allowed. For >>> these empty NUMA nodes, the 'len' of 'reg' property is zero. These >>> empty NUMA node IDs are still valid and parsed. I finds difficulty >>> to get where it's properly documented. >>> >>> So lets add note to empty NUMA nodes in the NUMA binding doc. >>> >>> Signed-off-by: Gavin Shan <gshan@redhat.com> >>> --- >>> Documentation/devicetree/bindings/numa.txt | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/numa.txt b/Documentation/devicetree/bindings/numa.txt >>> index 21b35053ca5a..edf728cff155 100644 >>> --- a/Documentation/devicetree/bindings/numa.txt >>> +++ b/Documentation/devicetree/bindings/numa.txt >>> @@ -109,6 +109,10 @@ Example: >>> Dual socket system consists of 2 boards connected through ccn bus and >>> each board having one socket/soc of 8 cpus, memory and pci bus. >>> +Note that the empty NUMA nodes, which no memory resides in, are allowed. >> >> I would write that without "the": >> >> +Note that empty NUMA nodes, which no memory resides in, are allowed. >> >> BTW, AFAIK, NUMA nodes may contain memory, CPU(s), or I/O -- any one, two, or >> three, without the other types of resources being present. >> > > Sure, I will drop "the" in v4. > > The NUMA nodes here are memory nodes here. Since the NUMA node usually means > memory node. I'm not sure if I change the term "NUMA node" to "NUMA memory > node" in v4. If you agree, I would have something like this: > > Note that empty memory nodes, which no memory resides in, are allowed. > The NUMA node IDs in these empty memory nodes are still valid, but memory > can be added into them through hotplug afterwards. OK, that sounds fine to me. thanks. -- ~Randy ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node 2021-06-25 4:49 ` Randy Dunlap @ 2021-06-25 5:25 ` Gavin Shan 0 siblings, 0 replies; 5+ messages in thread From: Gavin Shan @ 2021-06-25 5:25 UTC (permalink / raw) To: Randy Dunlap, devicetree; +Cc: linux-kernel, robh+dt, drjones, shan.gavin On 6/25/21 2:49 PM, Randy Dunlap wrote: > On 6/24/21 9:32 PM, Gavin Shan wrote: >> On 6/25/21 1:47 PM, Randy Dunlap wrote: >>> On 6/24/21 9:30 PM, Gavin Shan wrote: >>>> The empty NUMA nodes, where no memory resides in, are allowed. For >>>> these empty NUMA nodes, the 'len' of 'reg' property is zero. These >>>> empty NUMA node IDs are still valid and parsed. I finds difficulty >>>> to get where it's properly documented. >>>> >>>> So lets add note to empty NUMA nodes in the NUMA binding doc. >>>> >>>> Signed-off-by: Gavin Shan <gshan@redhat.com> >>>> --- >>>> Documentation/devicetree/bindings/numa.txt | 4 ++++ >>>> 1 file changed, 4 insertions(+) >>>> >>>> diff --git a/Documentation/devicetree/bindings/numa.txt b/Documentation/devicetree/bindings/numa.txt >>>> index 21b35053ca5a..edf728cff155 100644 >>>> --- a/Documentation/devicetree/bindings/numa.txt >>>> +++ b/Documentation/devicetree/bindings/numa.txt >>>> @@ -109,6 +109,10 @@ Example: >>>> Dual socket system consists of 2 boards connected through ccn bus and >>>> each board having one socket/soc of 8 cpus, memory and pci bus. >>>> +Note that the empty NUMA nodes, which no memory resides in, are allowed. >>> >>> I would write that without "the": >>> >>> +Note that empty NUMA nodes, which no memory resides in, are allowed. >>> >>> BTW, AFAIK, NUMA nodes may contain memory, CPU(s), or I/O -- any one, two, or >>> three, without the other types of resources being present. >>> >> >> Sure, I will drop "the" in v4. >> >> The NUMA nodes here are memory nodes here. Since the NUMA node usually means >> memory node. I'm not sure if I change the term "NUMA node" to "NUMA memory >> node" in v4. If you agree, I would have something like this: >> >> Note that empty memory nodes, which no memory resides in, are allowed. >> The NUMA node IDs in these empty memory nodes are still valid, but memory >> can be added into them through hotplug afterwards. > > OK, that sounds fine to me. > Thanks, Randy. I just posted v4 to include the changes. Thanks, Gavin ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-06-25 5:25 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-06-25 4:30 [PATCH v3] Documentation, dt, numa: Add note to empty NUMA node Gavin Shan 2021-06-25 3:47 ` Randy Dunlap 2021-06-25 4:32 ` Gavin Shan 2021-06-25 4:49 ` Randy Dunlap 2021-06-25 5:25 ` Gavin Shan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).