> -----Original Message-----
> From: Jonathan Cameron <[email protected]>
> Sent: 29 September 2025 17:26
> To: Shameer Kolothum <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; Jason Gunthorpe
> <[email protected]>; Nicolin Chen <[email protected]>; [email protected];
> [email protected]; Nathan Chen <[email protected]>; Matt Ochs
> <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]
> Subject: Re: [PATCH v4 08/27] hw/arm/smmuv3-accel: Add
> set/unset_iommu_device callback
> 
>  #include CONFIG_DEVICES
> >
> > +typedef struct SMMUViommu {
> > +    IOMMUFDBackend *iommufd;
> > +    IOMMUFDViommu core;
> > +    uint32_t bypass_hwpt_id;
> > +    uint32_t abort_hwpt_id;
> > +    QLIST_HEAD(, SMMUv3AccelDevice) device_list;
> 
> Does that need a forwards def of SMMUv3AccelDevice ?

I haven't seen any compiler warnings or errors and it looks like,
QLIST_HEAD expands to something like below,

struct {
    struct SMMUv3AccelDevice *lh_first;
}

So should be fine, I guess.

Thanks,
Shameer

Reply via email to