Linux-PM Archive on lore.kernel.org
 help / color / Atom feed
* [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
@ 2019-06-06  1:24 Rajneesh Bhardwaj
  2019-06-12  9:52 ` Andy Shevchenko
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Rajneesh Bhardwaj @ 2019-06-06  1:24 UTC (permalink / raw)
  To: linux-kernel
  Cc: x86, bp, Rajneesh Bhardwaj, Borislav Petkov, Dave Hansen,
	Andy Shevchenko, H. Peter Anvin, Ingo Molnar, Kan Liang,
	Peter Zijlstra, platform-driver-x86, Qiuxu Zhuo,
	Srinivas Pandruvada, Len Brown, Thomas Gleixner, Linux PM

Add the CPUID model number of Ice Lake Neural Network Processor for Deep
Learning Inference (ICL-NNPI) to the Intel family list. Ice Lake NNPI uses
model number 0x9D and this will be documented in a future version of Intel
Software Development Manual.

Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: platform-driver-x86@vger.kernel.org
Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: Len Brown <lenb@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: x86-ml <x86@kernel.org>
Cc: Linux PM <linux-pm@vger.kernel.org>
Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@linux.intel.com>
---

Changes in v2:
 * Renamed Icelake to Ice Lake in the commit msg

 arch/x86/include/asm/intel-family.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/intel-family.h b/arch/x86/include/asm/intel-family.h
index 9f15384c504a..087de5d3b93a 100644
--- a/arch/x86/include/asm/intel-family.h
+++ b/arch/x86/include/asm/intel-family.h
@@ -53,6 +53,7 @@
 #define INTEL_FAM6_CANNONLAKE_MOBILE	0x66
 
 #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
+#define INTEL_FAM6_ICELAKE_NNPI		0x9D
 
 /* "Small Core" Processors (Atom) */
 
-- 
2.17.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-06  1:24 [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family Rajneesh Bhardwaj
@ 2019-06-12  9:52 ` Andy Shevchenko
  2019-06-12 14:51   ` Dave Hansen
  2019-06-12 12:37 ` [tip:x86/cpu] " tip-bot for Rajneesh Bhardwaj
  2019-06-14  9:28 ` tip-bot for Rajneesh Bhardwaj
  2 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2019-06-12  9:52 UTC (permalink / raw)
  To: Rajneesh Bhardwaj
  Cc: linux-kernel, x86, bp, Borislav Petkov, Dave Hansen,
	H. Peter Anvin, Ingo Molnar, Kan Liang, Peter Zijlstra,
	platform-driver-x86, Qiuxu Zhuo, Srinivas Pandruvada, Len Brown,
	Thomas Gleixner, Linux PM

On Thu, Jun 06, 2019 at 06:54:19AM +0530, Rajneesh Bhardwaj wrote:
> Add the CPUID model number of Ice Lake Neural Network Processor for Deep
> Learning Inference (ICL-NNPI) to the Intel family list. Ice Lake NNPI uses
> model number 0x9D and this will be documented in a future version of Intel
> Software Development Manual.

>  #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
> +#define INTEL_FAM6_ICELAKE_NNPI		0x9D

What "I" stands for?

For me sounds like it's redundant here or something like NNP_DLI would be
better (because somewhere we have _NP as for Network Processor).

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* [tip:x86/cpu] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-06  1:24 [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family Rajneesh Bhardwaj
  2019-06-12  9:52 ` Andy Shevchenko
@ 2019-06-12 12:37 ` " tip-bot for Rajneesh Bhardwaj
  2019-06-14  9:28 ` tip-bot for Rajneesh Bhardwaj
  2 siblings, 0 replies; 9+ messages in thread
From: tip-bot for Rajneesh Bhardwaj @ 2019-06-12 12:37 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: bp, lenb, kan.liang, srinivas.pandruvada, tglx,
	rajneesh.bhardwaj, qiuxu.zhuo, mingo, linux-kernel, peterz, hpa,
	andriy.shevchenko, linux-pm, dave.hansen

Commit-ID:  61caa8621b9979a78b04e353ab2ee44a47ef7a62
Gitweb:     https://git.kernel.org/tip/61caa8621b9979a78b04e353ab2ee44a47ef7a62
Author:     Rajneesh Bhardwaj <rajneesh.bhardwaj@linux.intel.com>
AuthorDate: Thu, 6 Jun 2019 06:54:19 +0530
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Wed, 12 Jun 2019 11:43:42 +0200

x86/cpu: Add Ice Lake NNPI to Intel family

Add the CPUID model number of Ice Lake Neural Network Processor for Deep
Learning Inference (ICL-NNPI) to the Intel family list. Ice Lake NNPI uses
model number 0x9D and this will be documented in a future version of Intel
Software Development Manual.

Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@linux.intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: bp@suse.de
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: platform-driver-x86@vger.kernel.org
Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: Len Brown <lenb@kernel.org>
Cc: Linux PM <linux-pm@vger.kernel.org>
Link: https://lkml.kernel.org/r/20190606012419.13250-1-rajneesh.bhardwaj@linux.intel.com

---
 arch/x86/include/asm/intel-family.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/intel-family.h b/arch/x86/include/asm/intel-family.h
index 9f15384c504a..087de5d3b93a 100644
--- a/arch/x86/include/asm/intel-family.h
+++ b/arch/x86/include/asm/intel-family.h
@@ -53,6 +53,7 @@
 #define INTEL_FAM6_CANNONLAKE_MOBILE	0x66
 
 #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
+#define INTEL_FAM6_ICELAKE_NNPI		0x9D
 
 /* "Small Core" Processors (Atom) */
 

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-12  9:52 ` Andy Shevchenko
@ 2019-06-12 14:51   ` Dave Hansen
  2019-06-12 16:29     ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Dave Hansen @ 2019-06-12 14:51 UTC (permalink / raw)
  To: Andy Shevchenko, Rajneesh Bhardwaj
  Cc: linux-kernel, x86, bp, Borislav Petkov, Dave Hansen,
	H. Peter Anvin, Ingo Molnar, Kan Liang, Peter Zijlstra,
	platform-driver-x86, Qiuxu Zhuo, Srinivas Pandruvada, Len Brown,
	Thomas Gleixner, Linux PM

On 6/12/19 2:52 AM, Andy Shevchenko wrote:
>>  #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
>> +#define INTEL_FAM6_ICELAKE_NNPI		0x9D
> What "I" stands for?
> 
> For me sounds like it's redundant here or something like NNP_DLI would be
> better (because somewhere we have _NP as for Network Processor).

Let's not bikeshed this too much.  These things aren't used that widely
and mostly they're just used for figuring out the processor generation.
 It's exceedingly rare to have something like:


	if (model == INTEL_FAM6_ICELAKE_MOBILE)
		foo();
	else if (model == INTEL_FAM6_ICELAKE_NNPI)
		bar();

where what you suggest would matter.

Preserving the ability to google "ice lake nnpi" is pretty important, so
preserving the Intel name makes a lot of sense here when possible.

Do we *HAVE* an Ice Lake network processor?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-12 14:51   ` Dave Hansen
@ 2019-06-12 16:29     ` Andy Shevchenko
  2019-06-12 16:33       ` Dave Hansen
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2019-06-12 16:29 UTC (permalink / raw)
  To: Dave Hansen
  Cc: Rajneesh Bhardwaj, linux-kernel, x86, bp, Borislav Petkov,
	Dave Hansen, H. Peter Anvin, Ingo Molnar, Kan Liang,
	Peter Zijlstra, platform-driver-x86, Qiuxu Zhuo,
	Srinivas Pandruvada, Len Brown, Thomas Gleixner, Linux PM

On Wed, Jun 12, 2019 at 07:51:26AM -0700, Dave Hansen wrote:
> On 6/12/19 2:52 AM, Andy Shevchenko wrote:
> >>  #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
> >> +#define INTEL_FAM6_ICELAKE_NNPI		0x9D
> > What "I" stands for?
> > 
> > For me sounds like it's redundant here or something like NNP_DLI would be
> > better (because somewhere we have _NP as for Network Processor).
> 
> Let's not bikeshed this too much.  These things aren't used that widely
> and mostly they're just used for figuring out the processor generation.
>  It's exceedingly rare to have something like:
> 
> 
> 	if (model == INTEL_FAM6_ICELAKE_MOBILE)
> 		foo();
> 	else if (model == INTEL_FAM6_ICELAKE_NNPI)
> 		bar();
> 
> where what you suggest would matter.
> 
> Preserving the ability to google "ice lake nnpi" is pretty important, so
> preserving the Intel name makes a lot of sense here when possible.

What I'm talking is a consistency among suffixes. If there is a real
abbreviation (NNPI) which anybody can google, I'm totally for it to be used.

> Do we *HAVE* an Ice Lake network processor?

Not Ice Lake, something else.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-12 16:29     ` Andy Shevchenko
@ 2019-06-12 16:33       ` Dave Hansen
  2019-06-12 17:08         ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Dave Hansen @ 2019-06-12 16:33 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Rajneesh Bhardwaj, linux-kernel, x86, bp, Borislav Petkov,
	Dave Hansen, H. Peter Anvin, Ingo Molnar, Kan Liang,
	Peter Zijlstra, platform-driver-x86, Qiuxu Zhuo,
	Srinivas Pandruvada, Len Brown, Thomas Gleixner, Linux PM

On 6/12/19 9:29 AM, Andy Shevchenko wrote:
> What I'm talking is a consistency among suffixes. If there is a real
> abbreviation (NNPI) which anybody can google, 

There is and you can. :)

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-12 16:33       ` Dave Hansen
@ 2019-06-12 17:08         ` Andy Shevchenko
  2019-06-12 17:12           ` Bhardwaj, Rajneesh
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2019-06-12 17:08 UTC (permalink / raw)
  To: Dave Hansen
  Cc: Rajneesh Bhardwaj, linux-kernel, x86, bp, Borislav Petkov,
	Dave Hansen, H. Peter Anvin, Ingo Molnar, Kan Liang,
	Peter Zijlstra, platform-driver-x86, Qiuxu Zhuo,
	Srinivas Pandruvada, Len Brown, Thomas Gleixner, Linux PM

On Wed, Jun 12, 2019 at 09:33:30AM -0700, Dave Hansen wrote:
> On 6/12/19 9:29 AM, Andy Shevchenko wrote:
> > What I'm talking is a consistency among suffixes. If there is a real
> > abbreviation (NNPI) which anybody can google, 
> 
> There is and you can. :)

Good, I have no objections.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-12 17:08         ` Andy Shevchenko
@ 2019-06-12 17:12           ` Bhardwaj, Rajneesh
  0 siblings, 0 replies; 9+ messages in thread
From: Bhardwaj, Rajneesh @ 2019-06-12 17:12 UTC (permalink / raw)
  To: Andy Shevchenko, Dave Hansen
  Cc: linux-kernel, x86, bp, Borislav Petkov, Dave Hansen,
	H. Peter Anvin, Ingo Molnar, Kan Liang, Peter Zijlstra,
	platform-driver-x86, Qiuxu Zhuo, Srinivas Pandruvada, Len Brown,
	Thomas Gleixner, Linux PM


On 12-Jun-19 10:38 PM, Andy Shevchenko wrote:
> On Wed, Jun 12, 2019 at 09:33:30AM -0700, Dave Hansen wrote:
>> On 6/12/19 9:29 AM, Andy Shevchenko wrote:
>>> What I'm talking is a consistency among suffixes. If there is a real
>>> abbreviation (NNPI) which anybody can google,
>> There is and you can. :)
> Good, I have no objections.


Great, thanks Andy and Dave for your valuable comments. I see that 
Thomas has accepted this one, thanks a lot, Thomas!

https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?id=61caa8621b9979a78b04e353ab2ee44a47ef7a62&anzwix=1 


>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [tip:x86/cpu] x86/cpu: Add Ice Lake NNPI to Intel family
  2019-06-06  1:24 [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family Rajneesh Bhardwaj
  2019-06-12  9:52 ` Andy Shevchenko
  2019-06-12 12:37 ` [tip:x86/cpu] " tip-bot for Rajneesh Bhardwaj
@ 2019-06-14  9:28 ` tip-bot for Rajneesh Bhardwaj
  2 siblings, 0 replies; 9+ messages in thread
From: tip-bot for Rajneesh Bhardwaj @ 2019-06-14  9:28 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: peterz, rajneesh.bhardwaj, linux-kernel, dave.hansen, linux-pm,
	mingo, kan.liang, qiuxu.zhuo, andriy.shevchenko, bp,
	srinivas.pandruvada, tglx, hpa, lenb

Commit-ID:  e32d045cd4ba06b59878323e434bad010e78e658
Gitweb:     https://git.kernel.org/tip/e32d045cd4ba06b59878323e434bad010e78e658
Author:     Rajneesh Bhardwaj <rajneesh.bhardwaj@linux.intel.com>
AuthorDate: Thu, 6 Jun 2019 06:54:19 +0530
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Thu, 13 Jun 2019 19:37:42 +0200

x86/cpu: Add Ice Lake NNPI to Intel family

Add the CPUID model number of Ice Lake Neural Network Processor for Deep
Learning Inference (ICL-NNPI) to the Intel family list. Ice Lake NNPI uses
model number 0x9D and this will be documented in a future version of Intel
Software Development Manual.

Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@linux.intel.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: bp@suse.de
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Kan Liang <kan.liang@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: platform-driver-x86@vger.kernel.org
Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: Len Brown <lenb@kernel.org>
Cc: Linux PM <linux-pm@vger.kernel.org>
Link: https://lkml.kernel.org/r/20190606012419.13250-1-rajneesh.bhardwaj@linux.intel.com

---
 arch/x86/include/asm/intel-family.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/x86/include/asm/intel-family.h b/arch/x86/include/asm/intel-family.h
index 9f15384c504a..087de5d3b93a 100644
--- a/arch/x86/include/asm/intel-family.h
+++ b/arch/x86/include/asm/intel-family.h
@@ -53,6 +53,7 @@
 #define INTEL_FAM6_CANNONLAKE_MOBILE	0x66
 
 #define INTEL_FAM6_ICELAKE_MOBILE	0x7E
+#define INTEL_FAM6_ICELAKE_NNPI		0x9D
 
 /* "Small Core" Processors (Atom) */
 

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, back to index

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-06  1:24 [Patch v2] x86/cpu: Add Ice Lake NNPI to Intel family Rajneesh Bhardwaj
2019-06-12  9:52 ` Andy Shevchenko
2019-06-12 14:51   ` Dave Hansen
2019-06-12 16:29     ` Andy Shevchenko
2019-06-12 16:33       ` Dave Hansen
2019-06-12 17:08         ` Andy Shevchenko
2019-06-12 17:12           ` Bhardwaj, Rajneesh
2019-06-12 12:37 ` [tip:x86/cpu] " tip-bot for Rajneesh Bhardwaj
2019-06-14  9:28 ` tip-bot for Rajneesh Bhardwaj

Linux-PM Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-pm/0 linux-pm/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-pm linux-pm/ https://lore.kernel.org/linux-pm \
		linux-pm@vger.kernel.org linux-pm@archiver.kernel.org
	public-inbox-index linux-pm


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-pm


AGPL code for this site: git clone https://public-inbox.org/ public-inbox