| Message ID | 20240123022852.2475-3-TonyWWang-oc@zhaoxin.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Herbert Xu |
| Headers | show |
| Series | Add Zhaoxin hardware engine driver support for SHA | expand |
On Tue, Jan 23, 2024 at 10:28:51AM +0800, Tony W Wang-oc wrote: > Zhaoxin CPUs have implemented the SHA(Secure Hash Algorithm) as its > instrucions. > Add two CPU feature flags indicated by CPUID.(EAX=C0000001,ECX=0):EDX > bit 25/26 which will be used by Zhaoxin SHA driver. > > Signed-off-by: Tony W Wang-oc <TonyWWang-oc@zhaoxin.com> > --- > arch/x86/include/asm/cpufeatures.h | 4 +++- > tools/arch/x86/include/asm/cpufeatures.h | 4 +++- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/arch/x86/include/asm/cpufeatures.h b/arch/x86/include/asm/cpufeatures.h > index 29cb275a219d..28b0e62dbdf5 100644 > --- a/arch/x86/include/asm/cpufeatures.h > +++ b/arch/x86/include/asm/cpufeatures.h > @@ -145,7 +145,7 @@ > #define X86_FEATURE_RDRAND ( 4*32+30) /* RDRAND instruction */ > #define X86_FEATURE_HYPERVISOR ( 4*32+31) /* Running on a hypervisor */ > > -/* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */ > +/* VIA/Cyrix/Centaur/Zhaoxin-defined CPU features, CPUID level 0xC0000001, word 5 */ Does that mean that all those companies agree on the contents of this CPUID leaf? > #define X86_FEATURE_XSTORE ( 5*32+ 2) /* "rng" RNG present (xstore) */ > #define X86_FEATURE_XSTORE_EN ( 5*32+ 3) /* "rng_en" RNG enabled */ > #define X86_FEATURE_XCRYPT ( 5*32+ 6) /* "ace" on-CPU crypto (xcrypt) */ > @@ -156,6 +156,8 @@ > #define X86_FEATURE_PHE_EN ( 5*32+11) /* PHE enabled */ > #define X86_FEATURE_PMM ( 5*32+12) /* PadLock Montgomery Multiplier */ > #define X86_FEATURE_PMM_EN ( 5*32+13) /* PMM enabled */ > +#define X86_FEATURE_PHE2 ( 5*32+25) /* "phe2" Zhaoxin Hash Engine */ > +#define X86_FEATURE_PHE2_EN ( 5*32+26) /* "phe2_en" PHE2 enabled */ ^^^^^^^^^ From: Documentation/arch/x86/cpuinfo.rst "a: Feature flags can be derived from the contents of CPUID leaves. ------------------------------------------------------------------ These feature definitions are organized mirroring the layout of CPUID leaves and grouped in words with offsets as mapped in enum cpuid_leafs in cpufeatures.h (see arch/x86/include/asm/cpufeatures.h for details). If a feature is defined with a X86_FEATURE_<name> definition in cpufeatures.h, and if it is detected at run time, the flags will be displayed accordingly in /proc/cpuinfo. For example, the flag "avx2" comes from X86_FEATURE_AVX2 in cpufeatures.h." Is your grep broken?
On January 23, 2024 1:44:27 AM PST, Borislav Petkov <bp@alien8.de> wrote: >On Tue, Jan 23, 2024 at 10:28:51AM +0800, Tony W Wang-oc wrote: >> Zhaoxin CPUs have implemented the SHA(Secure Hash Algorithm) as its >> instrucions. >> Add two CPU feature flags indicated by CPUID.(EAX=C0000001,ECX=0):EDX >> bit 25/26 which will be used by Zhaoxin SHA driver. >> >> Signed-off-by: Tony W Wang-oc <TonyWWang-oc@zhaoxin.com> >> --- >> arch/x86/include/asm/cpufeatures.h | 4 +++- >> tools/arch/x86/include/asm/cpufeatures.h | 4 +++- >> 2 files changed, 6 insertions(+), 2 deletions(-) >> >> diff --git a/arch/x86/include/asm/cpufeatures.h b/arch/x86/include/asm/cpufeatures.h >> index 29cb275a219d..28b0e62dbdf5 100644 >> --- a/arch/x86/include/asm/cpufeatures.h >> +++ b/arch/x86/include/asm/cpufeatures.h >> @@ -145,7 +145,7 @@ >> #define X86_FEATURE_RDRAND ( 4*32+30) /* RDRAND instruction */ >> #define X86_FEATURE_HYPERVISOR ( 4*32+31) /* Running on a hypervisor */ >> >> -/* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */ >> +/* VIA/Cyrix/Centaur/Zhaoxin-defined CPU features, CPUID level 0xC0000001, word 5 */ > >Does that mean that all those companies agree on the contents of this >CPUID leaf? > >> #define X86_FEATURE_XSTORE ( 5*32+ 2) /* "rng" RNG present (xstore) */ >> #define X86_FEATURE_XSTORE_EN ( 5*32+ 3) /* "rng_en" RNG enabled */ >> #define X86_FEATURE_XCRYPT ( 5*32+ 6) /* "ace" on-CPU crypto (xcrypt) */ >> @@ -156,6 +156,8 @@ >> #define X86_FEATURE_PHE_EN ( 5*32+11) /* PHE enabled */ >> #define X86_FEATURE_PMM ( 5*32+12) /* PadLock Montgomery Multiplier */ >> #define X86_FEATURE_PMM_EN ( 5*32+13) /* PMM enabled */ >> +#define X86_FEATURE_PHE2 ( 5*32+25) /* "phe2" Zhaoxin Hash Engine */ >> +#define X86_FEATURE_PHE2_EN ( 5*32+26) /* "phe2_en" PHE2 enabled */ > ^^^^^^^^^ > >From: Documentation/arch/x86/cpuinfo.rst > >"a: Feature flags can be derived from the contents of CPUID leaves. >------------------------------------------------------------------ >These feature definitions are organized mirroring the layout of CPUID >leaves and grouped in words with offsets as mapped in enum cpuid_leafs >in cpufeatures.h (see arch/x86/include/asm/cpufeatures.h for details). >If a feature is defined with a X86_FEATURE_<name> definition in >cpufeatures.h, and if it is detected at run time, the flags will be >displayed accordingly in /proc/cpuinfo. For example, the flag "avx2" >comes from X86_FEATURE_AVX2 in cpufeatures.h." > >Is your grep broken? > Well, Centaur bought Cyrix, and then VIA bought Centaur. I think Zhaoxin is a joint venture between VIA and the City of Shanghai, or something like that?
On Tue, Jan 23, 2024 at 07:42:00AM -0800, H. Peter Anvin wrote: > Well, Centaur bought Cyrix, and then VIA bought Centaur. I suspected something like that. > I think Zhaoxin is a joint venture between VIA and the City of > Shanghai, or something like that? Aha. Btw, lemme know if your reply bounces too. I got <TonyWWang-oc@zhaoxin.com>: host mx2.zhaoxin.com[203.110.167.99] said: 550 Sender IP reverse lookup rejected (in reply to RCPT TO command) earlier. Thx.
On 2024/1/23 23:42, H. Peter Anvin wrote: > > [这封邮件来自外部发件人 谨防风险] > > On January 23, 2024 1:44:27 AM PST, Borislav Petkov <bp@alien8.de> wrote: >> On Tue, Jan 23, 2024 at 10:28:51AM +0800, Tony W Wang-oc wrote: >>> Zhaoxin CPUs have implemented the SHA(Secure Hash Algorithm) as its >>> instrucions. >>> Add two CPU feature flags indicated by CPUID.(EAX=C0000001,ECX=0):EDX >>> bit 25/26 which will be used by Zhaoxin SHA driver. >>> >>> Signed-off-by: Tony W Wang-oc <TonyWWang-oc@zhaoxin.com> >>> --- >>> arch/x86/include/asm/cpufeatures.h | 4 +++- >>> tools/arch/x86/include/asm/cpufeatures.h | 4 +++- >>> 2 files changed, 6 insertions(+), 2 deletions(-) >>> >>> diff --git a/arch/x86/include/asm/cpufeatures.h b/arch/x86/include/asm/cpufeatures.h >>> index 29cb275a219d..28b0e62dbdf5 100644 >>> --- a/arch/x86/include/asm/cpufeatures.h >>> +++ b/arch/x86/include/asm/cpufeatures.h >>> @@ -145,7 +145,7 @@ >>> #define X86_FEATURE_RDRAND ( 4*32+30) /* RDRAND instruction */ >>> #define X86_FEATURE_HYPERVISOR ( 4*32+31) /* Running on a hypervisor */ >>> >>> -/* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */ >>> +/* VIA/Cyrix/Centaur/Zhaoxin-defined CPU features, CPUID level 0xC0000001, word 5 */ >> Does that mean that all those companies agree on the contents of this >> CPUID leaf? >> >>> #define X86_FEATURE_XSTORE ( 5*32+ 2) /* "rng" RNG present (xstore) */ >>> #define X86_FEATURE_XSTORE_EN ( 5*32+ 3) /* "rng_en" RNG enabled */ >>> #define X86_FEATURE_XCRYPT ( 5*32+ 6) /* "ace" on-CPU crypto (xcrypt) */ >>> @@ -156,6 +156,8 @@ >>> #define X86_FEATURE_PHE_EN ( 5*32+11) /* PHE enabled */ >>> #define X86_FEATURE_PMM ( 5*32+12) /* PadLock Montgomery Multiplier */ >>> #define X86_FEATURE_PMM_EN ( 5*32+13) /* PMM enabled */ >>> +#define X86_FEATURE_PHE2 ( 5*32+25) /* "phe2" Zhaoxin Hash Engine */ >>> +#define X86_FEATURE_PHE2_EN ( 5*32+26) /* "phe2_en" PHE2 enabled */ >> ^^^^^^^^^ >> >> From: Documentation/arch/x86/cpuinfo.rst >> >> "a: Feature flags can be derived from the contents of CPUID leaves. >> ------------------------------------------------------------------ >> These feature definitions are organized mirroring the layout of CPUID >> leaves and grouped in words with offsets as mapped in enum cpuid_leafs >> in cpufeatures.h (see arch/x86/include/asm/cpufeatures.h for details). >> If a feature is defined with a X86_FEATURE_<name> definition in >> cpufeatures.h, and if it is detected at run time, the flags will be >> displayed accordingly in /proc/cpuinfo. For example, the flag "avx2" >> comes from X86_FEATURE_AVX2 in cpufeatures.h." >> >> Is your grep broken? >> > Well, Centaur bought Cyrix, and then VIA bought Centaur. I think Zhaoxin is a joint venture between VIA and the City of Shanghai, or something like that? Yes, Zhaoxin is a joint venture including VIA and Shanghai Alliance Investment Ltd. VIA has not designed new CPU products for a long time, nor maintained the previous products. Zhaoxin is currently designing and releasing new CPU products, and VIA understands and agrees that Zhaoxin uses the contents of this CPUID leaf. Sorry for late!
diff --git a/arch/x86/include/asm/cpufeatures.h b/arch/x86/include/asm/cpufeatures.h index 29cb275a219d..28b0e62dbdf5 100644 --- a/arch/x86/include/asm/cpufeatures.h +++ b/arch/x86/include/asm/cpufeatures.h @@ -145,7 +145,7 @@ #define X86_FEATURE_RDRAND ( 4*32+30) /* RDRAND instruction */ #define X86_FEATURE_HYPERVISOR ( 4*32+31) /* Running on a hypervisor */ -/* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */ +/* VIA/Cyrix/Centaur/Zhaoxin-defined CPU features, CPUID level 0xC0000001, word 5 */ #define X86_FEATURE_XSTORE ( 5*32+ 2) /* "rng" RNG present (xstore) */ #define X86_FEATURE_XSTORE_EN ( 5*32+ 3) /* "rng_en" RNG enabled */ #define X86_FEATURE_XCRYPT ( 5*32+ 6) /* "ace" on-CPU crypto (xcrypt) */ @@ -156,6 +156,8 @@ #define X86_FEATURE_PHE_EN ( 5*32+11) /* PHE enabled */ #define X86_FEATURE_PMM ( 5*32+12) /* PadLock Montgomery Multiplier */ #define X86_FEATURE_PMM_EN ( 5*32+13) /* PMM enabled */ +#define X86_FEATURE_PHE2 ( 5*32+25) /* "phe2" Zhaoxin Hash Engine */ +#define X86_FEATURE_PHE2_EN ( 5*32+26) /* "phe2_en" PHE2 enabled */ /* More extended AMD flags: CPUID level 0x80000001, ECX, word 6 */ #define X86_FEATURE_LAHF_LM ( 6*32+ 0) /* LAHF/SAHF in long mode */ diff --git a/tools/arch/x86/include/asm/cpufeatures.h b/tools/arch/x86/include/asm/cpufeatures.h index f4542d2718f4..21caba9d070b 100644 --- a/tools/arch/x86/include/asm/cpufeatures.h +++ b/tools/arch/x86/include/asm/cpufeatures.h @@ -145,7 +145,7 @@ #define X86_FEATURE_RDRAND ( 4*32+30) /* RDRAND instruction */ #define X86_FEATURE_HYPERVISOR ( 4*32+31) /* Running on a hypervisor */ -/* VIA/Cyrix/Centaur-defined CPU features, CPUID level 0xC0000001, word 5 */ +/* VIA/Cyrix/Centaur/Zhaoxin-defined CPU features, CPUID level 0xC0000001, word 5 */ #define X86_FEATURE_XSTORE ( 5*32+ 2) /* "rng" RNG present (xstore) */ #define X86_FEATURE_XSTORE_EN ( 5*32+ 3) /* "rng_en" RNG enabled */ #define X86_FEATURE_XCRYPT ( 5*32+ 6) /* "ace" on-CPU crypto (xcrypt) */ @@ -156,6 +156,8 @@ #define X86_FEATURE_PHE_EN ( 5*32+11) /* PHE enabled */ #define X86_FEATURE_PMM ( 5*32+12) /* PadLock Montgomery Multiplier */ #define X86_FEATURE_PMM_EN ( 5*32+13) /* PMM enabled */ +#define X86_FEATURE_PHE2 ( 5*32+25) /* "phe2" Zhaoxin Hash Engine */ +#define X86_FEATURE_PHE2_EN ( 5*32+26) /* "phe2_en" PHE2 enabled */ /* More extended AMD flags: CPUID level 0x80000001, ECX, word 6 */ #define X86_FEATURE_LAHF_LM ( 6*32+ 0) /* LAHF/SAHF in long mode */
Zhaoxin CPUs have implemented the SHA(Secure Hash Algorithm) as its instrucions. Add two CPU feature flags indicated by CPUID.(EAX=C0000001,ECX=0):EDX bit 25/26 which will be used by Zhaoxin SHA driver. Signed-off-by: Tony W Wang-oc <TonyWWang-oc@zhaoxin.com> --- arch/x86/include/asm/cpufeatures.h | 4 +++- tools/arch/x86/include/asm/cpufeatures.h | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-)