Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-08 Thread Dave Young
On Fri, Oct 8, 2010 at 11:53 PM, Balbir Singh wrote: > * Dave Young [2010-10-08 21:33:02]: > >> On Fri, Oct 8, 2010 at 9:09 PM, Balbir Singh >> wrote: >> > * Dave Young [2010-10-05 20:45:21]: >> > >> >> Balloon could cause guest memory oom killing and panic. >> >> >> >> Add oom notify to leak

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-08 Thread Balbir Singh
* Dave Young [2010-10-08 21:33:02]: > On Fri, Oct 8, 2010 at 9:09 PM, Balbir Singh > wrote: > > * Dave Young [2010-10-05 20:45:21]: > > > >> Balloon could cause guest memory oom killing and panic. > >> > >> Add oom notify to leak some memory and retry fill balloon after 5 minutes. > >> > >> At

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-08 Thread Dave Young
On Fri, Oct 8, 2010 at 9:09 PM, Balbir Singh wrote: > * Dave Young [2010-10-05 20:45:21]: > >> Balloon could cause guest memory oom killing and panic. >> >> Add oom notify to leak some memory and retry fill balloon after 5 minutes. >> >> At the same time add a mutex to protect balloon operations

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-08 Thread Balbir Singh
* Dave Young [2010-10-05 20:45:21]: > Balloon could cause guest memory oom killing and panic. > > Add oom notify to leak some memory and retry fill balloon after 5 minutes. > > At the same time add a mutex to protect balloon operations > because we need leak balloon in oom notifier and give bac

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-08 Thread Dave Young
On Wed, Oct 6, 2010 at 9:50 PM, Dave Young wrote: > On Wed, Oct 6, 2010 at 5:05 PM, Rusty Russell wrote: >> On Tue, 5 Oct 2010 11:15:21 pm Dave Young wrote: >>> Balloon could cause guest memory oom killing and panic. >>> >>> Add oom notify to leak some memory and retry fill balloon after 5 minute

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-06 Thread Dave Young
On Wed, Oct 6, 2010 at 5:05 PM, Rusty Russell wrote: > On Tue, 5 Oct 2010 11:15:21 pm Dave Young wrote: >> Balloon could cause guest memory oom killing and panic. >> >> Add oom notify to leak some memory and retry fill balloon after 5 minutes. > > Have you tried registering a shrinker?  See mm.h.

Re: [PATCH] kvm: add oom notifier for virtio balloon

2010-10-06 Thread Rusty Russell
On Tue, 5 Oct 2010 11:15:21 pm Dave Young wrote: > Balloon could cause guest memory oom killing and panic. > > Add oom notify to leak some memory and retry fill balloon after 5 minutes. Have you tried registering a shrinker? See mm.h. Thanks, Rusty. -- To unsubscribe from this list: send the li

[PATCH] kvm: add oom notifier for virtio balloon

2010-10-05 Thread Dave Young
Balloon could cause guest memory oom killing and panic. Add oom notify to leak some memory and retry fill balloon after 5 minutes. At the same time add a mutex to protect balloon operations because we need leak balloon in oom notifier and give back freed value. Thanks Anthony Liguori for his su