[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Jonathan Larmour
Subject: [lwip-devel] [patch #6253] Added csum to struct pbuf.
Date: Thu, 01 Nov 2007 14:38:54 +0000
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv: Gecko/20070530 Fedora/ Firefox/

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:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]