All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Brian J. Murrell" <brian@interlinx.bc.ca>
To: dm-crypt@saout.de
Subject: Re: [dm-crypt] dm-crypt multithreaded yet?
Date: Sun, 16 Dec 2012 16:43:50 -0500	[thread overview]
Message-ID: <50CE4096.5030508@interlinx.bc.ca> (raw)
In-Reply-To: <50CE0FAE.6030707@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1530 bytes --]

On 12-12-16 01:15 PM, Milan Broz wrote:
> 
> No idea why it turned world non-readable, fixed.

Thanks.

> But I am afraid I did not find any real situation where these patches
> really helps

If successive block I/Os from a sequential I/O can all get queued up by
different CPU (cores) I can't imagine why that wouldn't help.

> See also discussion on dm-devel list, I still think generic
> kernel workqueues should be used to provide such parallelization and
> we should not duplicate code in dm-crypt...

Well, definitely, if the parallelization work is duplicating existing
kernel functionality, that needs to be factored out.  But regardless...

> My suggestion is that using AES-NI extension helps much more with
> the current upstream code than anything else (for AES, obviously).

But of course.  I am sure anyone that has a CPU with AES-NI instructions
wouldn't care so much (at all even) about multi-core dm-crypt.  But
unfortunately not everyone has AES-NI capable processors, yet they might
have 4 cores (8 if HT enabled) available, with 3 (7, again if HT is
enabled) all sitting around doing nothing while one is grunting away
with all of the workload.

I wish I could just trade in my laptop for one with AES-NI, but other
than the lack of AES-NI it's far better than what I would get in return
for trading it in.  Kind of like trading in one's Ferrari just because
it doesn't have a leather steering-wheel for a Fiat that does have a
leather steering wheel.  :-/

Cheers,
b.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 261 bytes --]

      parent reply	other threads:[~2012-12-16 21:44 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-08 14:29 [dm-crypt] dm-crypt multithreaded yet? Brian J. Murrell
2012-08-08 15:31 ` Thomas Bächler
2012-08-08 15:45   ` Milan Broz
2012-08-09  3:17   ` Marc MERLIN
2012-08-09 10:09     ` Brian J. Murrell
2012-08-08 15:39 ` Milan Broz
2012-12-16 12:10   ` Brian J. Murrell
2012-12-16 18:15     ` Milan Broz
2012-12-16 21:29       ` Heinz Diehl
2012-12-16 21:43       ` Brian J. Murrell [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50CE4096.5030508@interlinx.bc.ca \
    --to=brian@interlinx.bc.ca \
    --cc=dm-crypt@saout.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.