From: Tianjia Zhang <tianjia.zhang@linux.alibaba.com>
To: Borislav Petkov <bp@alien8.de>
Cc: Herbert Xu <herbert@gondor.apana.org.au>,
"David S. Miller" <davem@davemloft.net>,
Vitaly Chikunov <vt@altlinux.org>,
Eric Biggers <ebiggers@google.com>,
Eric Biggers <ebiggers@kernel.org>,
Gilad Ben-Yossef <gilad@benyossef.com>,
Ard Biesheuvel <ardb@kernel.org>,
Jussi Kivilinna <jussi.kivilinna@iki.fi>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>,
Dave Hansen <dave.hansen@linux.intel.com>,
"H. Peter Anvin" <hpa@zytor.com>,
linux-crypto@vger.kernel.org, x86@kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/6] Introduce x86 assembly accelerated implementation for SM3 algorithm
Date: Fri, 24 Dec 2021 18:01:44 +0800 [thread overview]
Message-ID: <ed4ec4f9-9ede-8718-811c-8eae92c220bb@linux.alibaba.com> (raw)
In-Reply-To: <YcRPYHg3SMezrfiX@zn.tnic>
Hi Borislav,
On 12/23/21 6:28 PM, Borislav Petkov wrote:
> On Thu, Dec 23, 2021 at 12:35:41PM +0800, Tianjia Zhang wrote:
>> This series of patches creates an stand-alone library for SM3 hash
>> algorithm in the lib/crypto directory, and makes the implementations
>> that originally depended on sm3-generic depend on the stand-alone SM3
>> library, which also includes sm3-generic itself.
>>
>> On this basis, the AVX assembly acceleration implementation of SM3
>> algorithm is introduced, the main algorithm implementation based on
>> SM3 AES/BMI2 accelerated work by libgcrypt at:
>> https://gnupg.org/software/libgcrypt/index.html
>>
>> From the performance benchmark data, the performance improvement of
>> SM3 algorithm after AVX optimization can reach up to 38%.
>>
>> ---
>> v3 changes:
>> - update git commit message for patch 01
>>
>> v2 changes:
>> - x86/sm3: Change K macros to signed decimal and use LEA and 32-bit offset
>
> So you sent v2 yesterday. One day later, you're spamming again.
>
> So we have those process rules for a reason - use the time to read them
> before sending again a day later please.
>
> Documentation/process/submitting-patches.rst:
>
> "Don't get discouraged - or impatient
> ------------------------------------
>
> After you have submitted your change, be patient and wait. Reviewers are
> busy people and may not get to your patch right away.
>
> Once upon a time, patches used to disappear into the void without comment,
> but the development process works more smoothly than that now. You should
> receive comments within a week or so; if that does not happen, make sure
> that you have sent your patches to the right place. Wait for a minimum of
> one week before resubmitting or pinging reviewers - possibly longer during
> busy times like merge windows."
>
I'm sorry for the trouble to the community and reviews. Thanks for your
reminder. I will control the time interval when sending patches in the
future.
Merry Christmas Eve.
Tianjia
next prev parent reply other threads:[~2021-12-24 10:01 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-23 4:35 [PATCH v3 0/6] Introduce x86 assembly accelerated implementation for SM3 algorithm Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 1/6] crypto: sm3 - create SM3 stand-alone library Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 2/6] crypto: arm64/sm3-ce - make dependent on sm3 library Tianjia Zhang
2022-01-07 3:28 ` Herbert Xu
2022-01-07 8:57 ` Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 3/6] crypto: sm2 " Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 4/6] crypto: sm3 " Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 5/6] crypto: x86/sm3 - add AVX assembly implementation Tianjia Zhang
2021-12-23 4:35 ` [PATCH v3 6/6] crypto: tcrypt - add asynchronous speed test for SM3 Tianjia Zhang
2021-12-23 10:28 ` [PATCH v3 0/6] Introduce x86 assembly accelerated implementation for SM3 algorithm Borislav Petkov
2021-12-24 10:01 ` Tianjia Zhang [this message]
2021-12-24 13:00 ` Borislav Petkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ed4ec4f9-9ede-8718-811c-8eae92c220bb@linux.alibaba.com \
--to=tianjia.zhang@linux.alibaba.com \
--cc=ardb@kernel.org \
--cc=bp@alien8.de \
--cc=catalin.marinas@arm.com \
--cc=dave.hansen@linux.intel.com \
--cc=davem@davemloft.net \
--cc=ebiggers@google.com \
--cc=ebiggers@kernel.org \
--cc=gilad@benyossef.com \
--cc=herbert@gondor.apana.org.au \
--cc=hpa@zytor.com \
--cc=jussi.kivilinna@iki.fi \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=tglx@linutronix.de \
--cc=vt@altlinux.org \
--cc=will@kernel.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox