Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1630389
| From | Or Gerlitz <gerlitz.or@gmail.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] IB/IPoIB: Check the headroom size |
| Date | 2017-04-25 12:40 +0200 |
| Message-ID | <tA630-3ve-15@gated-at.bofh.it> (permalink) |
| References | <tA5qi-31F-17@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI <honli@redhat.com> wrote:
> From: Honggang Li <honli@redhat.com>
>
> Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which
> is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the
> size of headroom to avoid skb_under_panic.
sounds terrible, ipoib bonding is supported since ~2007, thanks for
reporting on that.
> [ 122.871493] ipoib_hard_header: skb->head= ffff8808179d9400, skb->data= ffff8808179d9420, skb_headroom= 0x20
> [ 123.055400] bond0: Releasing backup interface mthca_ib1
> [ 123.560529] bond_compute_features:1112 bond0 bond_dev->hard_header_len = 14
> [ 123.568822] CPU: 0 PID: 12336 Comm: ifdown-ib Not tainted 4.9.0-debug #1
did you generate this trace by calling dump_stack or this is existing
kernel code.
> Fixes: fc791b633515 ('IB/ipoib: move back IB LL address into the hard header')
this is more of WA to avoid some crash or failure but not fixing the
actual problem
Erez, can you comment?
Or.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH] IB/IPoIB: Check the headroom size Honggang LI <honli@redhat.com> - 2017-04-25 12:00 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Yuval Shaia <yuval.shaia@oracle.com> - 2017-04-25 12:20 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Or Gerlitz <gerlitz.or@gmail.com> - 2017-04-25 12:40 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Honggang LI <honli@redhat.com> - 2017-04-25 13:00 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Erez Shitrit <erezsh@dev.mellanox.co.il> - 2017-04-25 13:20 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Or Gerlitz <gerlitz.or@gmail.com> - 2017-04-25 13:20 +0200
Re: [PATCH] IB/IPoIB: Check the headroom size Erez Shitrit <erezsh@dev.mellanox.co.il> - 2017-04-25 13:50 +0200
csiph-web