Messages in this thread | | | Date | Wed, 17 Dec 2008 13:50:52 -0800 | From | Jeremy Fitzhardinge <> | Subject | Re: [PATCH UPDATED] xen: add xenfs to allow usermode <-> Xen interaction |
| |
Andrew Morton wrote: > I'm bored of your filesystem, so I think I'll just merge it, unless > someone else wants to ;) >
Yay! Boring code Wins!
> Was this noticed and considered? > > WARNING: consider using strict_strtoul in preference to simple_strtoul > #543: FILE: drivers/xen/xenfs/xenbus.c:315: > + trans->handle.id = simple_strtoul(reply, NULL, 0); >
Noticed that, but wasn't sure how to handle an error there anyway. Something pretty horrible must have happened at the other end.
J
| |