On Sun, May 15 2022, "Michael S. Tsirkin" <m...@redhat.com> wrote:
> On Wed, Apr 27, 2022 at 01:58:22AM +0300, Max Gurtovoy wrote: >> This new structure will be used for adding new miscellaneous registers >> for a virtio device configuration layout. >> >> For now, only admin_queue_index register is added. Admin virtqueue index >> does not depend on the device type. Hence, add a PCI capability to read >> the admin virtqueue index. >> >> Reviewed-by: Parav Pandit <pa...@nvidia.com> >> Signed-off-by: Max Gurtovoy <mgurto...@nvidia.com> > > > I guess we discussed this but I forgot. Why do we have this new > structure as opposed to just adding the value at the end of config > structure? I was kind of hoping that the structure can be > reused for CCW/MMIO and then we can add more use-cases with > new transport and device independent structures. > > If we keep it transport specific I don't really understand why > is it useful ... Nod, just define a common misc_configuration struct and have the individual transports access it in the way it works best for them. --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org