Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-28 Thread Tao Xu
On 10/26/2019 3:44 AM, Markus Armbruster wrote: Igor Mammedov writes: [...] 1st: above is applicable to both bw and lat values and should be documented as such 2nd: 'max NUM is 65534' when different suffixes is fleeting target, spec says that entry with 0x is unreachable,

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-27 Thread Markus Armbruster
Tao Xu writes: > Got it. I will use bytes per second for bandwidth here. Usually we use > nanosecond for memory latency, so if we use second for latency, it may > lose precision. So can I use nanosecond here, because we now use > nanosecond as smallest time unit. Sounds fair, go ahead.

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-27 Thread Tao Xu
On 10/26/2019 4:51 AM, Eduardo Habkost wrote: On Fri, Oct 25, 2019 at 09:44:50PM +0200, Markus Armbruster wrote: Igor Mammedov writes: On Fri, 25 Oct 2019 14:33:53 +0800 Tao Xu wrote: On 10/23/2019 11:28 PM, Igor Mammedov wrote: On Sun, 20 Oct 2019 19:11:19 +0800 Tao Xu wrote: [...]

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-25 Thread Eduardo Habkost
On Fri, Oct 25, 2019 at 09:44:50PM +0200, Markus Armbruster wrote: > Igor Mammedov writes: > > > On Fri, 25 Oct 2019 14:33:53 +0800 > > Tao Xu wrote: > > > >> On 10/23/2019 11:28 PM, Igor Mammedov wrote: > >> > On Sun, 20 Oct 2019 19:11:19 +0800 > >> > Tao Xu wrote: > >> [...] > >> >> +# >

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-25 Thread Markus Armbruster
Igor Mammedov writes: > On Fri, 25 Oct 2019 14:33:53 +0800 > Tao Xu wrote: > >> On 10/23/2019 11:28 PM, Igor Mammedov wrote: >> > On Sun, 20 Oct 2019 19:11:19 +0800 >> > Tao Xu wrote: >> [...] >> >> +# >> >> +# @access-bandwidth: access bandwidth (MB/s) >> >> +# >> >> +# @read-bandwidth:

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-25 Thread Igor Mammedov
On Fri, 25 Oct 2019 14:33:53 +0800 Tao Xu wrote: > On 10/23/2019 11:28 PM, Igor Mammedov wrote: > > On Sun, 20 Oct 2019 19:11:19 +0800 > > Tao Xu wrote: > [...] > >> +# > >> +# @access-bandwidth: access bandwidth (MB/s) > >> +# > >> +# @read-bandwidth: read bandwidth (MB/s) > >> +# > >> +#

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-25 Thread Tao Xu
On 10/23/2019 11:28 PM, Igor Mammedov wrote: On Sun, 20 Oct 2019 19:11:19 +0800 Tao Xu wrote: [...] +# +# @access-bandwidth: access bandwidth (MB/s) +# +# @read-bandwidth: read bandwidth (MB/s) +# +# @write-bandwidth: write bandwidth (MB/s) I think units here are not appropriate, values

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-23 Thread Igor Mammedov
On Sun, 20 Oct 2019 19:11:19 +0800 Tao Xu wrote: > From: Liu Jingqi > > Add -numa hmat-lb option to provide System Locality Latency and > Bandwidth Information. These memory attributes help to build > System Locality Latency and Bandwidth Information Structure(s) > in ACPI Heterogeneous Memory

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-22 Thread Tao Xu
On 10/22/2019 3:08 PM, Igor Mammedov wrote: On Sun, 20 Oct 2019 19:11:19 +0800 Tao Xu wrote: From: Liu Jingqi Add -numa hmat-lb option to provide System Locality Latency and Bandwidth Information. These memory attributes help to build System Locality Latency and Bandwidth Information

Re: [PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-22 Thread Igor Mammedov
On Sun, 20 Oct 2019 19:11:19 +0800 Tao Xu wrote: > From: Liu Jingqi > > Add -numa hmat-lb option to provide System Locality Latency and > Bandwidth Information. These memory attributes help to build > System Locality Latency and Bandwidth Information Structure(s) > in ACPI Heterogeneous Memory

[PATCH v13 06/12] numa: Extend CLI to provide memory latency and bandwidth information

2019-10-20 Thread Tao Xu
From: Liu Jingqi Add -numa hmat-lb option to provide System Locality Latency and Bandwidth Information. These memory attributes help to build System Locality Latency and Bandwidth Information Structure(s) in ACPI Heterogeneous Memory Attribute Table (HMAT). Signed-off-by: Liu Jingqi