qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael R. Hines" <mrhines@linux.vnet.ibm.com>
To: Chegu Vinod <chegu_vinod@hp.com>
Cc: Juan Jose Quintela Carreira <quintela@redhat.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Bulent Abali <abali@us.ibm.com>,
	"mrhines@us.ibm.com" <mrhines@us.ibm.com>,
	Anthony Liguori <anthony@codemonkey.ws>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v7 00/12] rdma: migration support
Date: Thu, 13 Jun 2013 10:55:24 -0400	[thread overview]
Message-ID: <51B9DD5C.1030409@linux.vnet.ibm.com> (raw)
In-Reply-To: <51B9D6A8.9070007@hp.com>

On 06/13/2013 10:26 AM, Chegu Vinod wrote:
>>
>> 1. start QEMU with the lock option *first*
>> 2. Then enable x-rdma-pin-all
>> 3. Then perform the migration
>>
>> What happens here? Does pinning "in advance" help you?
>
> Yes it does help by avoiding the freeze time at the start of the 
> pin-all migration.
>
> I already mentioned about this in in my earlier responses as an option 
> to consider for larger guests 
> (https://lists.gnu.org/archive/html/qemu-devel/2013-05/msg00435.html).
>
> But pinning all of guest memory has a few drawbacks...as you may 
> already know.
>
> Just to be sure I just double checked it again with your v7 bits . 
> Started a 64GB/10VCPU guest  (started qemu with the"-realtime 
> mlock=on" option) and as expected the guest startup took about 20 
> seconds longer (i.e. time taken to mlock the 64GB of guest RAM) but 
> the pin-all migration started fine...i.e. didn't observe any freezes 
> at the start of the migration
>
>
(CC-ing qemu-devel).

OK, that's good to know. This means that we need to bringup the mlock() 
problem as a "larger" issue in the linux community instead of the QEMU 
community.

In the meantime, how about I make update to the RDMA patch which does 
the following:

1. Solution #1:
        If user requests "x-rdma-pin-all", then
             If QEMU has enabled "-realtime mlock=on"
                    Then, allow the capability
             Else
                   Disallow the capability

2. Solution #2:  Create NEW qemu monitor command which locks memory *in 
advance*
                           before the migrate command occurs, to clearly 
indicate to the user
                           that the cost of locking memory must be paid 
before the migration starts.

Which solution do you prefer? Or do you have alternative idea?


> https://lists.gnu.org/archive/html/qemu-devel/2013-04/msg04161.html
>
> Again this is a generic linux mlock/clearpage related issue and not 
> directly related to your changes.
>

Do you have any ideas on how linux can be improved to solve this?
Is there any ongoing work that you know of on mlock() performance?

Is there, perhaps, some way for linux to "parallelize" the 
mlock()/clearpage operation?

- Michael

  parent reply	other threads:[~2013-06-13 17:33 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-10 16:03 [Qemu-devel] [PATCH v7 00/12] rdma: migration support mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 01/12] rdma: add documentation mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 02/12] rdma: introduce qemu_update_position() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 03/12] rdma: export yield_until_fd_readable() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 04/12] rdma: export throughput w/ MigrationStats QMP mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 05/12] rdma: introduce qemu_file_mode_is_not_valid() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 06/12] rdma: export qemu_fflush() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 07/12] rdma: introduce ram_handle_compressed() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 08/12] rdma: introduce qemu_ram_foreach_block() mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 09/12] rdma: new QEMUFileOps hooks mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 10/12] rdma: introduce capability x-rdma-pin-all mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 11/12] rdma: core logic mrhines
2013-06-10 16:03 ` [Qemu-devel] [PATCH v7 12/12] rdma: send pc.ram mrhines
     [not found] ` <4168C988EBDF2141B4E0B6475B6A73D10CE2AAC1@G6W2488.americas.hpqcorp.net>
     [not found]   ` <51B60ABA.2070401@linux.vnet.ibm.com>
     [not found]     ` <4168C988EBDF2141B4E0B6475B6A73D10CE2BAE7@G6W2488.americas.hpqcorp.net>
     [not found]       ` <51B7B652.3070905@linux.vnet.ibm.com>
     [not found]         ` <51B85EE5.1050702@hp.com>
     [not found]           ` <51B868B3.9090607@linux.vnet.ibm.com>
     [not found]             ` <51B9A614.2050101@hp.com>
     [not found]               ` <51B9BFCA.4050008@linux.vnet.ibm.com>
     [not found]                 ` <51B9CE2E.5080504@hp.com>
2013-06-13 14:45                   ` [Qemu-devel] [PATCH v7 00/12] rdma: migration support Michael R. Hines
     [not found]               ` <51B9C2D6.30000@linux.vnet.ibm.com>
     [not found]                 ` <51B9D6A8.9070007@hp.com>
2013-06-13 14:55                   ` Michael R. Hines [this message]
2013-06-13 20:06                     ` Paolo Bonzini
2013-06-13 21:17                       ` Michael R. Hines
2013-06-13 21:40                         ` Paolo Bonzini
2013-06-14  6:25                           ` Michael R. Hines
2013-06-14  6:30                         ` [Qemu-devel] RDMA: please pull and re-test freezing fixes Michael R. Hines
2013-06-14 20:38                           ` Michael R. Hines
2013-06-15 22:50                             ` Chegu Vinod
2013-06-16  4:13                             ` Michael R. Hines

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51B9DD5C.1030409@linux.vnet.ibm.com \
    --to=mrhines@linux.vnet.ibm.com \
    --cc=abali@us.ibm.com \
    --cc=anthony@codemonkey.ws \
    --cc=chegu_vinod@hp.com \
    --cc=mrhines@us.ibm.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).