Re: [PATCH] forcedeth: fix random memory scribbling bug

[Jeff Garzik <jgarzik@xxxxxxxxx>]

Linus Torvalds wrote:
> However, that
>
>         "skb->end - skb->data"
>
> calculation is a bit strange. It correctly maps the whole skb, but 
nod


> wouldn't it make more sense to use the length we actually tell the card to 
> use? 
>
> In other words, wouldn't it be a whole lot more sensible and logical to 
> use
>         np->rx_buf_sz
>
> instead? That's the value we use for allocation and that's the size we 
> tell the card we have.
That's the sort of thing I prefer.


> Of course, on the alloc path, it seems to add an additional 
> "NV_RX_ALLOC_PAD" thing, so maybe the "end-data" thing makes sense.
None of the other ethernet drivers do 'end - data', which is why I 
hesitated quite a bit on this change.
        Jeff