[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 41/42] Disable mlock around incoming postcopy
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH v7 41/42] Disable mlock around incoming postcopy |
Date: |
Tue, 28 Jul 2015 13:32:40 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Amit Shah <address@hidden> wrote:
> On (Tue) 14 Jul 2015 [17:22:13], Juan Quintela wrote:
>> "Dr. David Alan Gilbert (git)" <address@hidden> wrote:
>
>> > + if (enable_mlock) {
>> > + if (os_mlock() < 0) {
>> > + error_report("mlock: %s", strerror(errno));
>> > + /*
>> > + * It doesn't feel right to fail at this point, we have a
>> > valid
>> > + * VM state.
>> > + */
>>
>> realtime_init() exit in case of os_mlock() fails, so current code is:
>
> Yea, I was wondering the same - but then I thought: would the realtime
> case want a migration to happen at all?
Then disable migration with realtime looks like saner. But that
decission don't belong to this series.
>
>> - we start qemu with mlock requset
>> - we mlock memory
>> - we start postcopy
>> - we munlock memory
>> - we mlock memory
>>
>> I wmill really, really preffer having a check if memory is mlocked, and
>> it that case, just abort migration altogether. Or better still, wait to
>> enable mlock *until* we have finished postcopy, no?
>
> Amit