Waskiewicz Jr, Peter P <[EMAIL PROTECTED]> wrote: > >> >@@ -3356,6 +3370,7 @@ void free_netdev(struct net_device *dev) >> > /* will free via device release */ >> > put_device(&dev->dev); >> > #else >> >+ kfree((char *)dev->egress_subqueue); >> > kfree((char *)dev - dev->padded); >> > #endif >> > } >> >> Ahem. Explain that cast. > > This can be removed if needed; however, I'm just copying what > the other kfree()'s are doing in this function. Any instance of a > typecast that I introduced in these patches are just following what > others have done in that section of the code. So the cast is just for > consistency in this particular area. If you'd like me to remove it, I > can do that.
The other cast is there for the subtraction, not the kfree... Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
