Re: [Xen-devel] [PATCH] libxc: lzma build fix

[Christoph Egger <Christoph.Egger@xxxxxxx>]

On Wednesday 26 January 2011 17:16:11 Ian Jackson wrote:
> Christoph Egger writes ("[Xen-devel] [PATCH] libxc: lzma build fix"):
> > Attached patch lets libxc build with lzma support on NetBSD.
> > NetBSD doesn't have sysconf(_SC_PHYS_PAGES).
>
> Can you please break this out into separate os-dependent code in an
> appropriate different file ?  These kind of #ifdefs break up the code
> and make it hard to see what's going on.

Yes. Attached patch factors physmem() out into os-dependent files
and renamed it to xc_get_physmem() to not pollute the namespace.

Signed-off-by: Christoph Egger <Christoph.Egger@xxxxxxx>


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

xen_libxc_lzma.diff
Description: xen_libxc_lzma.diff

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel