Hi,

I have updated the upstream merge request and posted performance numbers.

https://github.com/openzfs/zfs/pull/12549#issuecomment-968449709

The small benchmarks that I ran show that using sha-ni is 4x faster than using 
the current x86_64 flavor :-)

-Jan

On Sun, Nov 14, 2021, at 06:21, Nils Privat wrote:
> Since the main PR is not opened against the upstream repo it maybe doesnt get 
> the full attention - so a quick reply to bring this nice PR up again.
> As far as i can see some progress has been made but any feedback and 
> especially help solving open issues are welcome.
> 
> Much thanks in advance.
> 
> Am Mo., 13. Sept. 2021 um 07:03 Uhr schrieb Jan Kasiak <z...@cybojanek.net>:
>> Hi all!
>> 
>> I recently noticed that SHA checksums do not use SHA/AVX/SSSE extensions.
>> 
>> I have a draft of the code here: https://github.com/cybojanek/zfs/pull/1 and 
>> Rich Ercolani (rincebrain) was gracious enough to give me some feedback.
>> 
>> I submitted a pull request for a portion of it here: 
>> https://github.com/openzfs/zfs/pull/12549 but have not yet received any 
>> feedback on it.
>> 
>> Is there anything I should be doing on my side to move this forward?
>> 
>> Does anyone here have some general feedback on my code/commit structure 
>> etc...?
>> 
>> Thank you,
>> -Jan
> *openzfs <https://openzfs.topicbox.com/latest>* / openzfs-developer / see 
> discussions <https://openzfs.topicbox.com/groups/developer> + participants 
> <https://openzfs.topicbox.com/groups/developer/members> + delivery options 
> <https://openzfs.topicbox.com/groups/developer/subscription> Permalink 
> <https://openzfs.topicbox.com/groups/developer/T1b590d6f875fd4f2-M7ca241248040e731868b9a32>

------------------------------------------
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/T1b590d6f875fd4f2-M08d80dd0ff4b1305f2e8d1ff
Delivery options: https://openzfs.topicbox.com/groups/developer/subscription

Reply via email to