On Sun, May 15 2022, "Michael S. Tsirkin" <[email protected]> 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 <[email protected]> >> Signed-off-by: Max Gurtovoy <[email protected]> > > > 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: [email protected] For additional commands, e-mail: [email protected]
