On Tue, Mar 28, 2023 at 01:09:49PM -0600, Alex Williamson wrote: > "Just trigger it" is the same in either case. It seems bold to play > the complexity argument when we already have a function that does 90% > the correct thing where we can share much of the implementation and > userspace code without duplicating, but still relying on a legacy > interface for debugging. It just doesn't seem like we are sharing a lot, this patch is a whole new function and you are saying the unique implementation needs to be more complex still. Maybe the next version will be able to share more?? Like can we just patch the existing code that sets the group_id/dev_id or somethiing??? Still, I'm not sure that qemu will share really share much here, because if it runs in group mode then it has to parse the result in a totally different way than if it runs in dev_id mode. The ioctl call is only a line or two. I imagine qemu will end up with two different functions that both return some kind of internal list of qemu devices in the reset group. Jason
