Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751456AbdDMIrB (ORCPT ); Thu, 13 Apr 2017 04:47:01 -0400 Received: from mail-vk0-f42.google.com ([209.85.213.42]:35650 "EHLO mail-vk0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750746AbdDMIq7 (ORCPT ); Thu, 13 Apr 2017 04:46:59 -0400 MIME-Version: 1.0 In-Reply-To: <774a9713-05ef-0162-0203-461254a04f6e@gmail.com> References: <1491562064-23591-1-git-send-email-binoy.jayan@linaro.org> <774a9713-05ef-0162-0203-461254a04f6e@gmail.com> From: Binoy Jayan Date: Thu, 13 Apr 2017 14:16:43 +0530 Message-ID: Subject: Re: [RFC PATCH v5] IV Generation algorithms for dm-crypt To: Milan Broz Cc: Oded , Ofir , Herbert Xu , "David S. Miller" , linux-crypto@vger.kernel.org, Mark Brown , Arnd Bergmann , Linux kernel mailing list , Alasdair Kergon , Mike Snitzer , dm-devel@redhat.com, Shaohua Li , linux-raid@vger.kernel.org, Rajendra , Gilad Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1069 Lines: 29 Hi Milan, On 10 April 2017 at 19:30, Milan Broz wrote: Thank you for the reply. > Well, it is good that there is no performance degradation but it > would be nice to have some user of it that proves it is really > working for your hw. I have been able to get access to a hardware with IV generation support a few days back. The hardware I was having before did not have IV generation support. Will be able to come up with numbers after making it work with the new one. > FYI - with patch that increases dmcrypt sector size to 4k > I can see improvement in speed usually in 5-15% with sync AES-NI > (depends on access pattern), with dmcrypt mapped to memory > it is even close to 20% speed up (but such a configuration is > completely artificial). > > I wonder why increased dmcrypt sector size does not work for your hw, > it should help as well (and can be combiuned later with this IV approach). > (For native 4k drives this should be used in future anyway...) I think it should work well too with backward incompatibility. Thanks, Binoy