>-----Original Message----- >From: Ruifeng Wang [mailto:[email protected]] >Sent: Monday, April 29, 2019 3:02 AM >To: Wang, Yipeng1 <[email protected]>; Gobriel, Sameh ><[email protected]>; Richardson, Bruce ><[email protected]>; De Lara Guarch, Pablo ><[email protected]> >Cc: [email protected]; [email protected]; [email protected]; >[email protected]; Ruifeng Wang <[email protected]> >Subject: [PATCH v1] hash: simplify signature compare neon process > >Replaced multiple neon instructions with single equivalent instruction. >This made simpler code and a bit higher performance. >Hash bulk lookup had 0.1% ~ 3% performance gain in tests on ARM A72 >platforms. > >Signed-off-by: Ruifeng Wang <[email protected]> >Reviewed-by: Gavin Hu <[email protected]> >--- [Wang, Yipeng] Sorry for the late review. The logic seems fine to me based on my understanding of the instructions. But I don't have an ARM machine to test. Please make sure It passes the compilation and unit tests. Or others (Jerin maybe?) could provide more feedback?
Acked-by: Yipeng Wang <[email protected]>
