On 7 July 2010 14:00, Christoph Egger <christoph_eg...@gmx.de> wrote: > On Tuesday 06 July 2010 17:02:08 Cherry G. Mathew wrote: >> On 6 July 2010 09:19, Cherry G. Mathew <cherry.g.mat...@gmail.com> wrote: >> > jym@ and I made a few more changes to this. >> > Have a look here: >> > ftp://ftp.netbsd.org/pub/NetBSD/misc/cherry/balloon/ >> > >> > Any feedback would be much appreciated. If things go well, I'm hoping >> > to commit this marked "EXPERIMENTAL" "soon". >> >> Done. > > Thank you for doing this work. > > Though I have to report a bug: > > I have an amd64 Dom0 kernel w/o XEN_BALLOONING in the kernel config. > Boot the xen kernel with dom0_mem=512m > > After login I do: > > # sysctl kern.xen > kern.xen.balloon.current = 131072 > node_val out of range. > node_val = 0 > > I expect kern.xen.balloon.current to say 512MB and I don't expect to see > the node_val messages.
Whoops! Does the following patch fix it for you: Cheers, -- ~Cherry Index: sys/arch/xen/conf/files.xen =================================================================== RCS file: /cvsroot/src/sys/arch/xen/conf/files.xen,v retrieving revision 1.108 diff -u -r1.108 files.xen --- sys/arch/xen/conf/files.xen 6 Jul 2010 15:00:09 -0000 1.108 +++ sys/arch/xen/conf/files.xen 7 Jul 2010 07:08:47 -0000 @@ -406,7 +406,7 @@ file arch/xen/xen/xennetback_xenbus.c xvif file arch/xen/xen/xennet_checksum.c xvif | xennet file arch/xen/xen/xbdback_xenbus.c xbdback -file arch/xen/xen/balloon.c hypervisor +file arch/xen/xen/balloon.c hypervisor & xenbus & xen_balloon ifdef i386 include "arch/i386/conf/majors.i386"
