[PATCH 1/1] hv: Use only one txf buffer per channel and kmalloc on initialize
Jesper Juhl
jj at chaosbits.net
Mon Dec 13 21:22:11 UTC 2010
On Mon, 13 Dec 2010, Hank Janssen wrote:
> Correct issue with not checking kmalloc return value.
> This fix now only uses one receive buffer for all hv_utils
> channels, and will do only one kmalloc on init and will return
> with a -ENOMEM if kmalloc fails on initialize.
>
> And properly clean up memory on failure.
>
> Thanks to Evgeniy Polyakov <zbr at ioremap.net> for pointing this out.
> And thanks to Jesper Juhl <jj at chaosbits.net> and Ky Srinivasan
> <ksrinivasan at novell.com> for suggesting a better implementation of
> my original patch.
>
> Signed-off-by: Haiyang Zhang <haiyangz at microsoft.com>
> Signed-off-by: Hank Janssen <hjanssen at microsoft.com>
> Cc: Evgeniy Polyakov <zbr at ioremap.net>
> Cc: Jesper Juhl <jj at chaosbits.net>
> Cc: Ky Srinivasan <ksrinivasan at novell.com>
>
I can't spot any problems with these changes now, so feel free to add
Reviewed-by: Jesper Juhl <jj at chaosbits.net>
if you like.
--
Jesper Juhl <jj at chaosbits.net> http://www.chaosbits.net/
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please.
More information about the devel
mailing list