search for: b1d2068fa2bd

Displaying 6 results from an estimated 6 matches for "b1d2068fa2bd".

2020 Feb 04
5
Balloon pressuring page cache
...even though the hypervisor did not notify us to do so. Issue 1: When we unload the balloon driver in the guest in an unlucky event, we won't free the pages. We are missing something like (if I am not wrong): diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c index b1d2068fa2bd..e2b0925e1e83 100644 --- a/drivers/virtio/virtio_balloon.c +++ b/drivers/virtio/virtio_balloon.c @@ -929,6 +929,10 @@ static void remove_common(struct virtio_balloon *vb) leak_balloon(vb, vb->num_pages); update_balloon_size(vb); + /* There might be free pages that...
2020 Feb 04
5
Balloon pressuring page cache
...even though the hypervisor did not notify us to do so. Issue 1: When we unload the balloon driver in the guest in an unlucky event, we won't free the pages. We are missing something like (if I am not wrong): diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c index b1d2068fa2bd..e2b0925e1e83 100644 --- a/drivers/virtio/virtio_balloon.c +++ b/drivers/virtio/virtio_balloon.c @@ -929,6 +929,10 @@ static void remove_common(struct virtio_balloon *vb) leak_balloon(vb, vb->num_pages); update_balloon_size(vb); + /* There might be free pages that...
2020 Feb 05
0
Balloon pressuring page cache
...notify us to do so. > > > Issue 1: When we unload the balloon driver in the guest in an unlucky event, > we won't free the pages. We are missing something like (if I am not wrong): > > diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c > index b1d2068fa2bd..e2b0925e1e83 100644 > --- a/drivers/virtio/virtio_balloon.c > +++ b/drivers/virtio/virtio_balloon.c > @@ -929,6 +929,10 @@ static void remove_common(struct virtio_balloon > *vb) > leak_balloon(vb, vb->num_pages); > update_balloon_size(vb); > > +...
2020 Feb 04
0
Balloon pressuring page cache
...notify us to do so. > > > Issue 1: When we unload the balloon driver in the guest in an unlucky event, > we won't free the pages. We are missing something like (if I am not wrong): > > diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c > index b1d2068fa2bd..e2b0925e1e83 100644 > --- a/drivers/virtio/virtio_balloon.c > +++ b/drivers/virtio/virtio_balloon.c > @@ -929,6 +929,10 @@ static void remove_common(struct virtio_balloon *vb) > leak_balloon(vb, vb->num_pages); > update_balloon_size(vb); > > +...
2020 Feb 04
3
Balloon pressuring page cache
>>> I would say reverting probably makes sense. I'm not sure there is much >>> value to having a shrinker running deflation when you are actively trying >>> to increase the balloon. It would make more sense to wait until you are >>> actually about to start hitting oom. >> >> I think the shrinker makes sense for free page hinting feature >>
2020 Feb 04
3
Balloon pressuring page cache
>>> I would say reverting probably makes sense. I'm not sure there is much >>> value to having a shrinker running deflation when you are actively trying >>> to increase the balloon. It would make more sense to wait until you are >>> actually about to start hitting oom. >> >> I think the shrinker makes sense for free page hinting feature >>