Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-17 Thread Richard Henderson
On 8/17/20 10:09 AM, Daniel P. Berrangé wrote: > What were you measuring performance with ? Did you use the > benchmark-crypto-cipher program or something else ? Perf of a boot of an aarch64 kernel, which * debug enabled for regression testing, * the v8.3-pauth instructions enabled, * a l

Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-17 Thread Daniel P . Berrangé
On Wed, Aug 12, 2020 at 08:25:20PM -0700, Richard Henderson wrote: > Mostly this is intended to cleanup the class hierarchy > used for the ciphers. We currently have multiple levels > of dispatch, and multiple separate allocations. The final > patches rearrange this to one level of indirect call,

Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-17 Thread Daniel P . Berrangé
On Thu, Aug 13, 2020 at 04:11:40AM -0700, no-re...@patchew.org wrote: > Patchew URL: > https://patchew.org/QEMU/20200813032537.2888593-1-richard.hender...@linaro.org/ > > > > Hi, > > This series failed the docker-quick@centos7 build test. Please find the > testing commands and > their output

Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200813032537.2888593-1-richard.hender...@linaro.org/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRI

Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-13 Thread no-reply
] crypto/cipher: Class hierarchy cleanups === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT

Re: [PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-13 Thread Philippe Mathieu-Daudé
On 8/13/20 5:25 AM, Richard Henderson wrote: > Mostly this is intended to cleanup the class hierarchy > used for the ciphers. We currently have multiple levels > of dispatch, and multiple separate allocations. The final > patches rearrange this to one level of indirect call, and > all memory allo

[PATCH 00/17] crypto/cipher: Class hierarchy cleanups

2020-08-12 Thread Richard Henderson
Mostly this is intended to cleanup the class hierarchy used for the ciphers. We currently have multiple levels of dispatch, and multiple separate allocations. The final patches rearrange this to one level of indirect call, and all memory allocated contiguously. But on the way there are a number