Re: [PATCH v4 12/14] vtd: use a bit field for root_entry

2020-08-18 Thread Jan Beulich
On 12.08.2020 15:13, Durrant, Paul wrote: >> -Original Message- >> From: Jan Beulich >> Sent: 06 August 2020 13:34 >> >> On 04.08.2020 15:42, Paul Durrant wrote: >>> --- a/xen/drivers/passthrough/vtd/iommu.h >>> +++ b/xen/drivers/passthrough/vtd/iommu.h >>> @@ -184,21 +184,28 @@ >>>

RE: [PATCH v4 12/14] vtd: use a bit field for root_entry

2020-08-14 Thread Tian, Kevin
> From: Paul Durrant > Sent: Tuesday, August 4, 2020 9:42 PM > > From: Paul Durrant > > This makes the code a little easier to read and also makes it more consistent > with iremap_entry. I feel the original readability is slightly better, as ctp is less obvious than set_root_value,

RE: [PATCH v4 12/14] vtd: use a bit field for root_entry

2020-08-12 Thread Durrant, Paul
> -Original Message- > From: Jan Beulich > Sent: 06 August 2020 13:34 > To: Paul Durrant > Cc: xen-devel@lists.xenproject.org; Durrant, Paul ; > Kevin Tian > > Subject: RE: [EXTERNAL] [PATCH v4 12/14] vtd: use a bit field for root_entry > > CAUTION: This email originated from outside

Re: [PATCH v4 12/14] vtd: use a bit field for root_entry

2020-08-06 Thread Jan Beulich
On 04.08.2020 15:42, Paul Durrant wrote: > --- a/xen/drivers/passthrough/vtd/iommu.h > +++ b/xen/drivers/passthrough/vtd/iommu.h > @@ -184,21 +184,28 @@ > #define dma_frcd_source_id(c) (c & 0x) > #define dma_frcd_page_addr(d) (d & (((u64)-1) << 12)) /* low 64 bit */ > > -/* > - * 0: