Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1461625

Re: [PATCH 1/3] perf/x86/intel/uncore: remove hard code for Node ID mapping location

Show all headers | View raw

On 12 August 2016 at 16:30, Kan Liang <kan.liang@intel.com> wrote:
> From: Kan Liang <kan.liang@intel.com>
>
> The method to build PCI bus to socket mapping is similar among
> platforms. However, the PCI location where store Node ID mapping could

I think we should replace "where store" with "which store".

> vary for different platforms. For example, the Node ID mapping address
> on Skylake server is different as the previous platform. Also, to build

I think we should change "as the previous platform" to "from the
previous platforms".

> the mapping for the PCI bus without UBOX, it has to start from bus 0 on
> Skylake server.
>
> This patch removes the hard code in current implementation, and adds
> three parameters for snbep_pci2phy_map_init. So the Node ID mapping
> address and bus searching direction could be configured according to
> different platforms.
>
> Signed-off-by: Kan Liang <kan.liang@intel.com>
> ---
>  arch/x86/events/intel/uncore_snbep.c | 37 ++++++++++++++++++++++++------------
>  1 file changed, 25 insertions(+), 12 deletions(-)
>

Kan,  I have suggested a couple of changes to patch's description
above.  The main content of the patch seems good to me.


--
Nilay

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 1/3] perf/x86/intel/uncore: remove hard code for Node ID mapping location Kan Liang <kan.liang@intel.com> - 2016-08-12 23:40 +0200
  Re: [PATCH 1/3] perf/x86/intel/uncore: remove hard code for Node ID  mapping location Nilay Vaish <nilayvaish@gmail.com> - 2016-08-13 16:10 +0200

csiph-web