Re: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-06-17 Thread Jan Beulich
On 17.06.2021 13:40, Roger Pau Monné wrote: > On Thu, Jun 17, 2021 at 09:31:28AM +, Tian, Kevin wrote: >>> From: Roger Pau Monne >>> Sent: Saturday, May 29, 2021 1:40 AM >>> >>> Force WB type for grants and foreign pages. Those are usually mapped >>> over unpopulated physical ranges in the p2m

Re: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-06-17 Thread Roger Pau Monné
On Thu, Jun 17, 2021 at 09:31:28AM +, Tian, Kevin wrote: > > From: Roger Pau Monne > > Sent: Saturday, May 29, 2021 1:40 AM > > > > Force WB type for grants and foreign pages. Those are usually mapped > > over unpopulated physical ranges in the p2m, and those ranges would > > usually be UC in

RE: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-06-17 Thread Tian, Kevin
> From: Roger Pau Monne > Sent: Saturday, May 29, 2021 1:40 AM > > Force WB type for grants and foreign pages. Those are usually mapped > over unpopulated physical ranges in the p2m, and those ranges would > usually be UC in the MTRR state, which is unlikely to be the correct > cache attribute. I

Re: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-06-02 Thread Roger Pau Monné
On Mon, May 31, 2021 at 09:21:25AM +0200, Jan Beulich wrote: > On 28.05.2021 19:39, Roger Pau Monne wrote: > > --- a/xen/arch/x86/mm/p2m-ept.c > > +++ b/xen/arch/x86/mm/p2m-ept.c > > @@ -487,11 +487,12 @@ static int ept_invalidate_emt_range(struct p2m_domain > > *p2m, > > } > > > > int epte_ge

Re: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-05-31 Thread Jan Beulich
On 28.05.2021 19:39, Roger Pau Monne wrote: > --- a/xen/arch/x86/mm/p2m-ept.c > +++ b/xen/arch/x86/mm/p2m-ept.c > @@ -487,11 +487,12 @@ static int ept_invalidate_emt_range(struct p2m_domain > *p2m, > } > > int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn, > -

[PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps

2021-05-28 Thread Roger Pau Monne
Force WB type for grants and foreign pages. Those are usually mapped over unpopulated physical ranges in the p2m, and those ranges would usually be UC in the MTRR state, which is unlikely to be the correct cache attribute. It's also cumbersome (or even impossible) for the guest to be setting the MT