On Friday 10 October 2008, Dave wrote:Thanks for identifying the problem. The Agere case looks good - a few suggestions for the Symbol case though:
Updated patch attached
But you absolutely have to kfree(pda) here.
Yes, I forgot; sorry.
Dan Williams wrote:maybe you should not use priv->pda_size butUsing fw->pda_size should be fine. The value comes from a const static, set to 0x100 for Symbol.
#define SYMBOL_PDA_SIZE 256 and use that for the hermes_read_pda()
length just to ensure the patched code is functionally the same as
before the patch.
Actually it allocated 512 byte before. Should it probably be set to 0x200?