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
|