From: Junaid Shahid <junaids@google.com>
To: Ben Gardon <bgardon@google.com>,
"Zhoujian (jay)" <jianjay.zhou@huawei.com>
Cc: Peter Xu <peterx@redhat.com>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"pbonzini@redhat.com" <pbonzini@redhat.com>,
"dgilbert@redhat.com" <dgilbert@redhat.com>,
"quintela@redhat.com" <quintela@redhat.com>,
"Liujinsong (Paul)" <liu.jinsong@huawei.com>,
"linfeng (M)" <linfeng23@huawei.com>,
"wangxin (U)" <wangxinxin.wang@huawei.com>,
"Huangweidong (C)" <weidong.huang@huawei.com>,
pfeiner@google.com
Subject: Re: RFC: Split EPT huge pages in advance of dirty logging
Date: Fri, 21 Feb 2020 14:08:14 -0800 [thread overview]
Message-ID: <cd4626a1-44b5-1a62-cf4b-716950a6db1b@google.com> (raw)
In-Reply-To: <CANgfPd-P_=GqcMiwLSSkUhZDt42aMLUsCJt+CPdUN5yR3RLHmQ@mail.gmail.com>
On 2/20/20 9:34 AM, Ben Gardon wrote:
>
> FWIW, we currently do this eager splitting at Google for live
> migration. When the log-dirty-memory flag is set on a memslot we
> eagerly split all pages in the slot down to 4k granularity.
> As Jay said, this does not cause crippling lock contention because the
> vCPU page faults generated by write protection / splitting can be
> resolved in the fast page fault path without acquiring the MMU lock.
> I believe +Junaid Shahid tried to upstream this approach at some point
> in the past, but the patch set didn't make it in. (This was before my
> time, so I'm hoping he has a link.)
> I haven't done the analysis to know if eager splitting is more or less
> efficient with parallel slow-path page faults, but it's definitely
> faster under the MMU lock.
>
I am not sure if we ever posted those patches upstream. Peter Feiner would know for sure. One notable difference in what we do compared to the approach outlined by Jay is that we don't rely on tdp_page_fault() to do the splitting. So we don't have to create a dummy VCPU and the specialized split function is also much faster.
Thanks,
Junaid
next prev parent reply other threads:[~2020-02-21 22:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-18 13:13 RFC: Split EPT huge pages in advance of dirty logging Zhoujian (jay)
2020-02-18 17:43 ` Peter Xu
2020-02-19 13:19 ` Zhoujian (jay)
2020-02-19 17:19 ` Peter Xu
2020-02-20 13:52 ` Zhoujian (jay)
2020-02-20 17:32 ` Ben Gardon
2020-02-20 17:34 ` Ben Gardon
2020-02-20 18:17 ` Peter Xu
2020-02-21 6:51 ` Zhoujian (jay)
2020-02-21 22:08 ` Junaid Shahid [this message]
2020-02-22 0:19 ` Peter Feiner
2020-02-24 1:07 ` Zhoujian (jay)
2020-03-02 13:38 ` Zhoujian (jay)
2020-03-02 16:28 ` Peter Feiner
2020-03-03 4:29 ` Zhoujian (jay)
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=cd4626a1-44b5-1a62-cf4b-716950a6db1b@google.com \
--to=junaids@google.com \
--cc=bgardon@google.com \
--cc=dgilbert@redhat.com \
--cc=jianjay.zhou@huawei.com \
--cc=kvm@vger.kernel.org \
--cc=linfeng23@huawei.com \
--cc=liu.jinsong@huawei.com \
--cc=pbonzini@redhat.com \
--cc=peterx@redhat.com \
--cc=pfeiner@google.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=wangxinxin.wang@huawei.com \
--cc=weidong.huang@huawei.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).