[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH RFC 04/10] intel_iommu: Second Stage Access Dirty bit support
From: |
Peter Xu |
Subject: |
Re: [PATCH RFC 04/10] intel_iommu: Second Stage Access Dirty bit support |
Date: |
Fri, 29 Apr 2022 14:21:11 -0400 |
On Fri, Apr 29, 2022 at 10:12:01AM +0100, Joao Martins wrote:
> On 4/29/22 03:26, Jason Wang wrote:
> > On Fri, Apr 29, 2022 at 5:14 AM Joao Martins <joao.m.martins@oracle.com>
> > wrote:
> >> @@ -3693,7 +3759,8 @@ static void vtd_init(IntelIOMMUState *s)
> >>
> >> /* TODO: read cap/ecap from host to decide which cap to be exposed. */
> >> if (s->scalable_mode) {
> >> - s->ecap |= VTD_ECAP_SMTS | VTD_ECAP_SRS | VTD_ECAP_SLTS;
> >> + s->ecap |= VTD_ECAP_SMTS | VTD_ECAP_SRS | VTD_ECAP_SLTS |
> >> + VTD_ECAP_SLADS;
> >> }
> >
> > We probably need a dedicated command line parameter and make it compat
> > for pre 7.1 machines.
> >
> > Otherwise we may break migration.
>
> I can gate over an 'x-ssads' option (default disabled). Which reminds me that
> I probably
> should rename to the most recent mnemonic (as SLADS no longer exists in
> manuals).
>
> If we all want by default enabled I can add a separate patch to do so.
The new option sounds good.
Jason, per our previous discussion, shall we not worry about the
compatibility issues per machine-type until the whole feature reaches a
mostly-complete stage?
There seems to have a bunch of sub-features for scalable mode and it's a
large project as a whole. I'm worried trying to maintain compatibilities
for all the small sub-features could be an unnessary burden to the code
base.
Thanks,
--
Peter Xu
- [PATCH RFC 00/10] hw/vfio, x86/iommu: IOMMUFD Dirty Tracking, Joao Martins, 2022/04/28
- [PATCH RFC 02/10] amd-iommu: Access/Dirty bit support, Joao Martins, 2022/04/28
- [PATCH RFC 03/10] intel-iommu: Cache PASID entry flags, Joao Martins, 2022/04/28
- [PATCH RFC 05/10] linux-headers: import iommufd.h hwpt extensions, Joao Martins, 2022/04/28
- [PATCH RFC 07/10] vfio/iommufd: Add HWPT_GET_DIRTY_IOVA support, Joao Martins, 2022/04/28
- [PATCH RFC 08/10] vfio/iommufd: Add IOAS_UNMAP_DIRTY support, Joao Martins, 2022/04/28
- [PATCH RFC 06/10] vfio/iommufd: Add HWPT_SET_DIRTY support, Joao Martins, 2022/04/28
- [PATCH RFC 09/10] migration/dirtyrate: Expand dirty_bitmap to be tracked separately for devices, Joao Martins, 2022/04/28
- [PATCH RFC 10/10] hw/vfio: Add nr of dirty pages to tracepoints, Joao Martins, 2022/04/28