Re: [PATCH kernel 1/2] mm: add the related functions to build the free page bitmap

From	"Dr. David Alan Gilbert" <dgilbert@redhat.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH kernel 1/2] mm: add the related functions to build the free page bitmap
Date	2016-04-22 11:50 +0200
Message-ID	<rqFSQ-8hs-37@gated-at.bofh.it> (permalink)
References	<rpF8u-8oB-17@gated-at.bofh.it> <rpF8u-8oB-19@gated-at.bofh.it> <rpFia-8s9-15@gated-at.bofh.it> <rpFrQ-ov-13@gated-at.bofh.it> <rpGxz-1bm-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

* Michael S. Tsirkin (mst@redhat.com) wrote:
> On Tue, Apr 19, 2016 at 03:02:09PM +0000, Li, Liang Z wrote:
> > > On Tue, 2016-04-19 at 22:34 +0800, Liang Li wrote:
> > > > The free page bitmap will be sent to QEMU through virtio interface and
> > > > used for live migration optimization.
> > > > Drop the cache before building the free page bitmap can get more free
> > > > pages. Whether dropping the cache is decided by user.
> > > >
> > > 
> > > How do you prevent the guest from using those recently-freed pages for
> > > something else, between when you build the bitmap and the live migration
> > > completes?
> > 
> > Because the dirty page logging is enabled before building the bitmap, there is no need
> > to prevent the guest from using the recently-freed pages ...
> > 
> > Liang
> 
> Well one point of telling host that page is free is so that
> it can mark it clean even if it was dirty previously.
> So I think you must pass the pages to guest under the lock.
> This will allow host optimizations such as marking these
> pages MADV_DONTNEED or MADV_FREE.
> Otherwise it's all too tied up to a specific usecase -
> you aren't telling host that a page is free, you are telling it
> that a page was free in the past.

But doing it under lock sounds pretty expensive, especially given
how long the userspace side is going to take to work through the bitmap
and device what to do.

Dave

> 
> -- 
> MST
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

Thread

[PATCH kernel 1/2] mm: add the related functions to build the free page bitmap Liang Li <liang.z.li@intel.com> - 2016-04-19 16:50 +0200
  Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap Rik van Riel <riel@redhat.com> - 2016-04-19 17:00 +0200
    RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-19 17:10 +0200
      Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap Rik van Riel <riel@redhat.com> - 2016-04-19 17:30 +0200
        RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-20 03:00 +0200
      Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Michael S. Tsirkin" <mst@redhat.com> - 2016-04-19 18:20 +0200
        RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-20 03:50 +0200
          Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Michael S. Tsirkin" <mst@redhat.com> - 2016-04-21 15:50 +0200
            RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-22 03:40 +0200
        Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Dr. David Alan Gilbert" <dgilbert@redhat.com> - 2016-04-22 11:50 +0200
          Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Michael S. Tsirkin" <mst@redhat.com> - 2016-04-22 16:00 +0200
            RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-25 05:20 +0200
              Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Michael S. Tsirkin" <mst@redhat.com> - 2016-04-25 12:50 +0200
                RE: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap "Li, Liang Z" <liang.z.li@intel.com> - 2016-04-26 05:30 +0200
  Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap kbuild test robot <lkp@intel.com> - 2016-04-19 17:00 +0200
  Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap kbuild test robot <lkp@intel.com> - 2016-04-19 18:10 +0200
  Re: [PATCH kernel 1/2] mm: add the related functions to build the  free page bitmap kbuild test robot <lkp@intel.com> - 2016-04-19 18:20 +0200

csiph-web