[lwip-devel] [patch #6253] Added csum to struct pbuf.

[Jonathan Larmour]

Follow-up Comment #10, patch #6253 (project lwip):

Oh, I hate it when it turns out we've been fervently agreeing all along :-).

> I don't think I understand. What do you mean with 'this'? DMA
> alignment? I wouldn't add fields to struct pbuf until we have
> code that takes care for DMA alignment or else my code doesn't
> work any more, either! 

Yes, I meant alignment for DMA/cache/whatever. My changes to the core code
were modest. The syntax of enforcing alignment is non-portable (unless you
start to let things get wasteful), so I thought the best thing would be to
allow the driver or port to override how the pbuf pools are declared. I then
also added a hook that could be called on freeing.

But the patch is not against CVS, so I'd need to port it to the current code
base, after updating my own port to CVS.


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?6253>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/