[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1178711957.7286.87.camel@localhost.localdomain>
Date: Wed, 09 May 2007 21:59:17 +1000
From: Rusty Russell <rusty@...tcorp.com.au>
To: Pekka Enberg <penberg@...helsinki.fi>
Cc: "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org, virtualization@...ts.osdl.org,
ak@...e.de, jeremy@...p.org
Subject: Re: [patch 2/9] lguest: the guest code
On Wed, 2007-05-09 at 13:16 +0300, Pekka Enberg wrote:
> Hi Rusty,
>
> On 5/9/07, akpm@...ux-foundation.org <akpm@...ux-foundation.org> wrote:
> > +static void add_lguest_device(unsigned int index)
> > +{
> > + struct lguest_device *new;
> > +
> > + lguest_devices[index].status |= LGUEST_DEVICE_S_ACKNOWLEDGE;
> > + new = kmalloc(sizeof(struct lguest_device), GFP_KERNEL);
> > + if (!new) {
> > + printk(KERN_EMERG "Cannot allocate lguest device %u\n", index);
> > + lguest_devices[index].status |= LGUEST_DEVICE_S_FAILED;
>
> [snip]
>
> Any reason why we don't return an error here?
Hi Pekka,
Hmm, where would the error go?
A little confused,
Rusty.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists