Re: [stable] [PATCH] virtio: fix oops on OOM

2010-07-26 Thread Chuck Ebbert
On Fri, 23 Jul 2010 15:48:37 +0930 Rusty Russell wrote: Note that commit 686d363786a53ed28ee875b84ef24e6d5126ef6f, which caused this problem, is already queued for -stable, so it should either be removed or this should be sent for -stable as well. > From: "Michael S. Tsirkin" > > virtio ring w

[PATCH] virtio: fix oops on OOM

2010-07-26 Thread Rusty Russell
From: "Michael S. Tsirkin" virtio ring was changed to return an error code on OOM, but one caller was missed and still checks for vq->vring.num. The fix is just to check for <0 error code. Long term it might make sense to change goto add_head to just return an error on oom instead, but let's app

[PATCH] virtio: fix oops on OOM

2010-07-22 Thread Rusty Russell
From: "Michael S. Tsirkin" virtio ring was changed to return an error code on OOM, but one caller was missed and still checks for vq->vring.num. The fix is just to check for <0 error code. Long term it might make sense to change goto add_head to just return an error on oom instead, but let's app