<div dir="ltr"><div>Hi,<br><br>On 8 March 2017 at 13:49, Binoy Jayan <<a href="mailto:binoy.jayan@linaro.org" target="_blank">binoy.jayan@linaro.org</a>> wrote:<br>> Hi Gilad,<br>><br>>> I gave it a spin on a x86_64 with 8 CPUs with AES-NI using cryptd and<br>>> on Arm  using CryptoCell hardware accelerator.<br>>><br>>> There was no difference in performance between 512 and 4096 bytes<br>>> cluster size on the x86_64 (800 MB loop file system)<br>>><br>>> There was an improvement in latency of 3.2% between 512 and 4096 bytes<br>>> cluster size on the Arm. I expect the performance benefits for this<br>>> test for Binoy's patch to be the same.<br>>><br>>> In both cases the very naive test was a simple dd with block size of<br>>> 4096 bytes or the raw block device.<br>>><br>>> I do not know what effect having a bigger cluster size would have on<br>>> have on other more complex file system operations.<br>>> Is there any specific benchmark worth testing with?<br><br>The multiple instances issue in /proc/crypto is fixed. It was because of<br>the IV code itself modifying the algorithm name inadvertently in the<br>global crypto algorithm lookup table when it was splitting up<br>"plain(cbc(aes))" into "plain" and "cbc(aes)" so as to invoke the child<br>algorithm.<br><br>I ran a few tests with dd, bonnie and FIO under Qemu - x86 using the<br>automated script [1] that I wrote to make the testing easy.<br>The tests were done on software implementations of the algorithms<br>as the real hardware was not available with me. According to the test,<br>I found that the sequential reads and writes have a good improvement<br>(5.7 %) in the data rate with the proposed solution while the random<br>reads shows a very little improvement. When tested with FIO, the<br>random writes also shows a small improvement (2.2%) but the random<br>reads show a little deterioration in performance (4 %).<br> <br></div><div>When tested in arm hardware, only the sequential writes with bonnie<br>shows improvement (5.6%). All other tests shows degraded performance<br></div><div>in the absence of crypto hardware.<br></div><div><br>[1] <a href="https://github.com/binoyjayan/utilities/blob/master/utils/dmtest" target="_blank">https://github.com/binoyjayan/<wbr>utilities/blob/master/utils/dm<wbr>test</a><br></div>Dependencies: dd [Full version], bonnie, fio<br><div><br>Thanks,<br>Binoy<br><br></div></div>