Re: [PATCH 1/1] hash: add SVE support for bulk key lookup

[David Marchand <david.marchand@redhat.com>]

On Thu, Aug 17, 2023 at 11:24 PM Harjot Singh <Harjot.Singh@arm.com> wrote:
>
> From: Harjot Singh <harjot.singh@arm.com>
>
> - Implemented Vector Length Agnostic SVE code for comparing signatures
> in bulk lookup.
> - Added Defines in code for SVE code support.
> - New Optimised SVE code is 1-2 CPU cycle slower than NEON for N2
> processor.
>
> Performance Numbers from hash_perf_autotest :
>
> Elements in Primary or Secondary Location
>
> Results (in CPU cycles/operation)
> -----------------------------------
>  Operations without data
>
> Without pre-computed hash values
>
> Keysize     Add/Lookup/Lookup_bulk
>             Neon         SVE
> 4           93/71/26     93/71/27
> 8           93/70/26     93/70/27
> 9           94/74/27     94/74/28
> 13          100/80/31    100/79/32
> 16          100/78/30    100/78/31
> 32          109/110/38   108/110/39
>
> With pre-computed hash values
>
> Keysize     Add/Lookup/Lookup_bulk
>             Neon         SVE
> 4           83/58/27     83/58/29
> 8           83/57/27     83/57/28
> 9           83/60/28     83/60/29
> 13          84/60/28     83/60/29
> 16          83/58/27     83/58/29
> 32          84/68/31     84/68/32
>
> Signed-off-by: Harjot Singh <harjot.singh@arm.com>
> Reviewed-by: Nathan Brown <nathan.brown@arm.com>
> Reviewed-by: Feifei Wang <feifei.wang2@arm.com>
> Reviewed-by: Jieqiang Wang <jieqiang.wang@arm.com>
> Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>

Thanks for the patch, please update the release notes.


-- 
David Marchand