Hi Cyril, Thanks for the review.
On Mon, Oct 16, 2017 at 02:32:58PM +1100, Cyril Bur wrote: > On Fri, 2017-10-13 at 12:30 +0800, wei.guo.si...@gmail.com wrote: > > From: Simon Guo <wei.guo.si...@gmail.com> > > > > This patch adjust selftest memcmp_64 so that memcmp selftest can be > > compiled successfully. > > > > Do they not compile at the moment? > Since I renamed enter/exit_vmx_copy() to enter/exit_vmx_ops() in patch 2nd, this patch is necessary to make selftest compiled based on patch 2nd. > > It also adds testcases for: > > - memcmp over 4K bytes size. > > - s1/s2 with different/random offset on 16 bytes boundary. > > - enter/exit_vmx_ops pairness. > > > > This is a great idea, just a thought though - perhaps it might make > more sense to have each condition be tested for in a separate binary > rather than a single binary that tests everything. > I am not sure whether it is better to seperate those test cases on the same memcmp func. Currently the time to execute this single test bin is within 35 secs, and keeping them in one exec will make people test more comprehensively. So Personally I prefer to keep them in one at present. Thanks, - Simon
