RE: [PATCH v7 3/4] admin: Add group member legacy register access commands

[Parav Pandit <parav@nvidia.com>]

> From: Michael S. Tsirkin <mst@redhat.com>
> Sent: Thursday, June 29, 2023 6:08 PM

[..]
> > Ok. For v8 I will have link and keep in pci transport section. Later I can move as
> union if needed.
> 
> I can do that too.
> 
Ok.
If you can merge the first two editorial fixes, I can drop them from v8.
I didn't attach github issue in those first two patches.
 
> > SIOV for PCI and CXL will look very different, so SIOV legacy I am not expecting
> any change for these commands.
> 
> SIOV will need a notification structure within BAR, and it could reuse the same
> one.
> 
> > > Also look at virtio_pci_notify_cap and check whether any normative
> > > statements there should apply here. Alignment I guess?
> >
> > I will move below normative from pci to generic.
> >
> > The group member driver SHOULD use the notification region supplied by the
> group owner driver.
> 
> By owner device.
> 
Ok.

> --
> MST