Re: Balloon driver.

["Cherry G. Mathew" <cherry.g.mathew%gmail.com@localhost>]

On 7 July 2010 14:00, Christoph Egger <Christoph_Egger%gmx.de@localhost> wrote:
> On Tuesday 06 July 2010 17:02:08 Cherry G. Mathew wrote:
>> On 6 July 2010 09:19, Cherry G. Mathew <cherry.g.mathew%gmail.com@localhost> 
>> 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"