On Thu, Nov 20, 2025 at 01:27:25PM -0800, Nicolin Chen wrote:
> On Thu, Nov 20, 2025 at 01:22:00PM +0000, Shameer Kolothum wrote:
> > + /* QEMU SMMUv3 supports Range Invalidation by default */
> > + if (FIELD_EX32(info->idr[3], IDR3, RIL) !=
> > + FIELD_EX32(s->idr[3], IDR3, RIL)) {
> > + error_setg(errp, "Host SMMUv3 doesn't support Range Invalidation");
> > + return false;
>
> If host reports info->ril=1 while VM sets s->ril=0, it could have
> worked. But this would reject the case.
>
> I think it should be:
> if (FIELD_EX32(info->idr[3], IDR3, RIL) <
> FIELD_EX32(s->idr[3], IDR3, RIL)) {
> ?
Never mind. I realized that you are doing in a followup patch :)
Nicolin
