Re: [PATCH kvm-unit-tests v2 03/17] x86: intel-iommu: add vt-d init test

[Peter Xu <peterx@redhat.com>]

On Thu, Nov 10, 2016 at 08:09:36PM +0100, Andrew Jones wrote:

[...]

> > +static uint64_t vtd_root_table(void)
> > +{
> > +       /* No extend root table support yet */
> > +       return vtd_readq(DMAR_RTADDR_REG) & VTD_RTA_MASK;
> 
> Should use tabs. You can run the kernel's scripts/checkpatch.pl,
> since we're using the same coding standard (pretty much...)

Yes, the indentation is wrong, which is to be fixed. Will try the
script later and see what else I can get.

> 
> > +}
> > +
> > +static uint64_t vtd_ir_table(void)
> > +{
> > +	return vtd_readq(DMAR_IRTA_REG) & VTD_IRTA_MASK;
> > +}
> 
> The above two functions are both only used once each. Do we need them?

They are both used in further patches of this series. Also they might
be used in the future as well. So I'll keep them if you would not
mind.

Thanks,

-- peterx