On Thu, 26 Feb 2026 17:56:07 -0700, Keith Busch wrote:

> > diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h
> > index ac2329f24141..bff2f5f7e38d 100644
> > --- a/include/uapi/linux/vfio.h
> > +++ b/include/uapi/linux/vfio.h
> > @@ -1501,6 +1501,8 @@ struct vfio_region_dma_range {
> > struct vfio_device_feature_dma_buf {
> >             __u32   region_index;
> >             __u32   open_flags;
> > +           __u16   steering_tag;
> > +           __u8    ph;
> >             __u32   flags;
> >             __u32   nr_ranges;
> >             struct vfio_region_dma_range dma_ranges[] 
> > __counted_by(nr_ranges);
>
> I don't think you can add fields to a uapi struct like this since it
> breaks comptibility. Instead, I think you may be able to carve it out of
> the "flags" field since it's currently reserved to be 0, so I guess it's
> potentially available to define a new feature.

Sure, thanks for the suggestion, yes I can also use the 'flags' field
for this.

Zhiping


Reply via email to