From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: [PATCH 0/2] Introduce the request handling for dm-crypt Date: Thu, 12 Nov 2015 08:26:26 -0700 Message-ID: <5644AFA2.6040201@kernel.dk> References: <20151111181813.GD12236@redhat.com> <20151112100422.GM12392@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151112100422.GM12392@sirena.org.uk> Sender: linux-kernel-owner@vger.kernel.org To: Mark Brown , Baolin Wang Cc: Mike Snitzer , Alasdair G Kergon , dm-devel@redhat.com, neilb@suse.com, linux-raid@vger.kernel.org, jack@suse.cz, Arnd Bergmann , LKML , keith.busch@intel.com, jmoyer@redhat.com, tj@kernel.org, bart.vanassche@sandisk.com, "Garg, Dinesh" List-Id: linux-raid.ids On 11/12/2015 03:04 AM, Mark Brown wrote: > On Thu, Nov 12, 2015 at 04:20:41PM +0800, Baolin Wang wrote: > >> 3. perforamence data >> It is just a simple dd test result, and will provide the formal report >> in future. But from the simple test, we can see the improvement. > > It's probably also worth pointing out that Qualcomm have been shipping > an out of tree implementation of this as a separate module in their BSP > (originally written by Danesh Garg who's on this thread): > > https://android.googlesource.com/kernel/msm/+/android-msm-dory-3.10-kitkat-wear/drivers/md/dm-req-crypt.c > > Android now wants to encrypt phones and tablets by default and have been > seeing substantial performance hits as a result, we can try to get > people to share performance data from productionish systems but it might > be difficult. Well, shame on them for developing out-of-tree, looks like they are reaping all the benefits of that. Guys, we need some numbers, enough with the hand waving. There's no point discussing this further until we know how much of a difference it makes to handle X MB chunks instead of Y MB chunks. As was previously stated, unless there's a _substantial_ performance benefit, this patchset isn't going anywhere. If there is a huge benefit, we can look into ways of making it actually work. That may not even be a request interface, it could just be proper utilization of plugging for in-dm bio merging. -- Jens Axboe