public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: uri@jdland.co.il (Uri Simchoni)
To: linux-arm-kernel@lists.infradead.org
Subject: Bug#552270: Marvell CESA driver and Kirkwood
Date: Sat, 24 Apr 2010 21:43:35 +0300	[thread overview]
Message-ID: <4BD33BD7.80400@jdland.co.il> (raw)
In-Reply-To: <20100424151207.GB9057@Chamillionaire.breakpoint.cc>

On 4/24/2010 6:12 PM, Sebastian Andrzej Siewior wrote:
> * Uri Simchoni | 2010-04-22 06:23:12 [+0300]:
> 
> For IPSec I use this[0] shell script which sets up a connection. Good for
> testing :) 
Thanks, That'll save time setting it up...
> I enabled list and sg debugging and a flood ping triggered a couple of
> warning. Could you please look at this?
Sure.
> 
> IPsec requests authenc(hmac(sha1),cbc(aes)) so right now it reqeusts two
> cesa provided algorithms. A single ping results in around 30ms RTT.
Since the CESA does each operation faster than sw (at least when the packet size exceeds some threshold), I see no reason for it to slow the process down. The slowness probably is somehow caused by the same thing that causes the oops, or by debug warning prints.

> Disabling hmac(sha1) gives me less than 1ms.
> Implementing authenc() for IPsec should speed things up. Right I'm stuck
> with hacking DMA support.
Well, so far I wasn't able to figure out how it all fits together - sure, the CESA can do AES-CBC+HMAC-SHA1 in one run, but I'm not sure it's suitable for IPSec, or that the crypto infrastructure supports a HW driver for combined operation. (the CESA is probably not suitable for SSL because of alignment problems, IPSec is better in that respect).
> 
> For now I think lowering priority of hmac() should fix the problem. A
> direct request "mv-hmac-sha1" should still returned the mv driver. What
> do you thing?
> 
I think there's a bug here I should find and fix. Till then perhaps the mv-hmac-sha1 driver should not be registered at all.
> Need to run now....
> 
>> Thanks,
>> Uri.
> 
> Sebastian

  parent reply	other threads:[~2010-04-24 18:43 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4B12F9E7.1020207@gmail.com>
     [not found] ` <20091130211531.GK18101@deprecation.cyrius.com>
     [not found]   ` <4B144627.7060901@gmail.com>
     [not found]     ` <20091204212847.GC28480@deprecation.cyrius.com>
2010-04-18 17:23       ` Bug#552270: Marvell CESA driver and Kirkwood L.C.
2010-04-18 19:17         ` Sebastian Andrzej Siewior
2010-04-19 23:11           ` L.C.
2010-04-20 20:45           ` L.C.
2010-04-21  8:13             ` Sebastian Andrzej Siewior
2010-04-22  3:23               ` Uri Simchoni
2010-04-24 15:12                 ` Sebastian Andrzej Siewior
2010-04-24 15:13                   ` Sebastian Andrzej Siewior
2010-04-24 18:43                   ` Uri Simchoni [this message]
2010-04-30  9:20                     ` Sebastian Andrzej Siewior
2010-04-24 14:42             ` [PATCH] net/ipsec: don't treat EINPROGRESS as a regular error Sebastian Andrzej Siewior
2010-04-25  1:18               ` Herbert Xu
2010-04-25 15:29                 ` Sebastian Andrzej Siewior
2010-04-26  1:17                   ` Herbert Xu
2010-04-26 18:11                     ` Sebastian Andrzej Siewior
2010-04-27  1:35                       ` Herbert Xu

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=4BD33BD7.80400@jdland.co.il \
    --to=uri@jdland.co.il \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox