All of lore.kernel.org
 help / color / mirror / Atom feed
From: Karl Hiramoto <karl@hiramoto.org>
To: Christian Hohnstaedt <chohnstaedt@innominate.com>
Cc: linux-crypto@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>
Subject: Re: ixp4xx_crypto panic with fragmented packets in scatterlist
Date: Wed, 25 Feb 2009 15:35:48 +0100	[thread overview]
Message-ID: <49A55744.3010702@hiramoto.org> (raw)
In-Reply-To: <20090225115407.GB6283@elara.bln.innominate.local>

Christian Hohnstaedt wrote:
>
>
> looks like there are different, incompatible sg chaining implementations:
>
> include/crypto/scatterwalk.h:scatterwalk_sg_chain()  uses
> sg->lenght == 0 as indicator for a chained sg
>
> include/linux/scatterlist.h:sg_chain() uses
> bit 0 of sg->page_link to indicate chaining
>
> Maybe the matters for b2ab4a57b018aafbba35bff088218f5cc3d2142e
> are obsolete now...
>
> However the scatterlist iteration in the arm implementation of
> dma_map_sg() uses neither of them, but simply sg++
>
> Please try the attached compile-tested patch.
>
>   
I had to adapt the change in dma_map_sg() a bit for 2.6.28.7.   The
patch  fixed the BUG() and  kernel panic.    But a ping with a
fragmented packet does not get a proper response.

Note: below, the 2nd half of the fragmented packet has the data 0x0000



What i see on   the PC host 192.168.10.51   where the IXP434
(192.168.10.54)  is doing  "ping -s 2000 -c1  192.168.10.51"

 tcpdump -i eth0 -vnXX esp
tcpdump: listening on eth0, link-type EN10MB (Ethernet), capture size 96
bytes
15:28:36.373270 IP (tos 0x0, ttl 64, id 26818, offset 0, flags [+],
proto ESP (50), length 1500) 192.168.10.54 > 192.168.10.51:
ESP(spi=0x00000101,seq=0xd), length 1480
        0x0000:  0015 c509 9b4a 9a39 d3c6 3651 0800 4500  .....J.9..6Q..E.
        0x0010:  05dc 68c2 2000 4032 5674 c0a8 0a36 c0a8  ..h...@2Vt...6..
        0x0020:  0a33 0000 0101 0000 000d 0f22 c952 606e  .3.........".R`n
        0x0030:  3d01 2214 d019 90ae ce7d 7bef 8067 4371  =."......}{..gCq
        0x0040:  6404 e937 0054 9835 a09a 080e ea31 b599  d..7.T.5.....1..
        0x0050:  353a d9a2 1074 3cb3 d856 0da7 13a9 a6c6  5:...t<..V......
15:28:36.373662 IP (tos 0x0, ttl 64, id 26818, offset 1480, flags
[none], proto ESP (50), length 580) 192.168.10.54 > 192.168.10.51: esp
        0x0000:  0015 c509 9b4a 9a39 d3c6 3651 0800 4500  .....J.9..6Q..E.
        0x0010:  0244 68c2 00b9 4032 7953 c0a8 0a36 c0a8  .Dh...@2yS...6..
        0x0020:  0a33 0000 0000 0000 0000 0000 0000 0000  .3..............
        0x0030:  0000 0000 0000 0000 0000 0000 0000 0000  ................
        0x0040:  0000 0000 0000 0000 0000 0000 0000 0000  ................
        0x0050:  0000 0000 0000 0000 0000 0000 0000 0000  ................
15:28:36.373952 IP (tos 0xc0, ttl 64, id 48218, offset 0, flags [none],
proto ESP (50), length 604) 192.168.10.51 > 192.168.10.54:
ESP(spi=0x00000201,seq=0xf), length 584
        0x0000:  9a39 d3c6 3651 0015 c509 9b4a 0800 45c0  .9..6Q.....J..E.
        0x0010:  025c bc5a 0000 4032 259c c0a8 0a33 c0a8  .\.Z..@2%....3..
        0x0020:  0a36 0000 0201 0000 000f a250 69a6 bf1f  .6.........Pi...
        0x0030:  bdc7 9873 1f25 cc84 b3f9 f1cf 2339 36c6  ...s.%......#96.
        0x0040:  fd2a 5097 bcef e915 437a 7c8f bc0b 4905  .*P.....Cz|...I.
        0x0050:  a73b ddb9 1bf9 b54b 6ade c59c ed80 9047  .;.....Kj......G



Thanks

--

Karl

  reply	other threads:[~2009-02-25 14:36 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-23 18:00 ixp4xx_crypto panic Karl Hiramoto
2009-02-24 13:19 ` ixp4xx_crypto panic with fragmented packets in scatterlist Karl Hiramoto
2009-02-25  9:07   ` Christian Hohnstaedt
2009-02-25  9:36     ` Karl Hiramoto
2009-02-25 11:54       ` Christian Hohnstaedt
2009-02-25 14:35         ` Karl Hiramoto [this message]
2009-02-25 15:35           ` Karl Hiramoto
2009-02-26  6:42             ` Herbert Xu
2009-02-26  9:55             ` [PATCH] " Russell King - ARM Linux
2009-02-26 12:10               ` Herbert Xu
2009-02-26 20:27                 ` Karl Hiramoto
2009-02-27 10:26                   ` Christian Hohnstaedt
2009-02-27 10:26                     ` Christian Hohnstaedt
2009-02-26 23:20                 ` Russell King - ARM Linux
2009-02-27  0:50                   ` Herbert Xu
2009-03-02 11:45                     ` [PATCH] crypto: fix handling of sg buffers in ixp4xx driver Christian Hohnstaedt
2009-03-02 20:42                       ` Russell King - ARM Linux
2009-03-02 20:42                         ` Russell King - ARM Linux
2009-03-03  3:02                         ` Herbert Xu
2009-03-27  7:09                       ` Herbert Xu
2009-03-27  7:09                         ` Herbert Xu
2009-03-03  3:02                   ` [PATCH] ixp4xx_crypto panic with fragmented packets in scatterlist Herbert Xu
2009-02-26  6:41         ` 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=49A55744.3010702@hiramoto.org \
    --to=karl@hiramoto.org \
    --cc=chohnstaedt@innominate.com \
    --cc=linux-crypto@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.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 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.