[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v10 4/5] libxl: add support for vscsi

Olaf Hering writes ("Re: [PATCH v10 4/5] libxl: add support for vscsi"):
> On Fri, Apr 01, Ian Jackson wrote:
> > I'm not sure that this sysfs parsing ought to be in xl rather than
> > libxl.  Also, this is Linux-specific code.  So it needs to be made
> > conditional somehow.
> The whole libxlu file is inside #ifdef __linux__?

Oh, so it is.  Sorry for not spotting that.  TBH I think it might be
better to do this the way that other kernel-specific things are done,
eg libxl_linux.c etc.

But I think changing this ought to wait until we settle whether this
code ought to be in xl or libxl.

> I will address the other comments in another mail.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.