Re: [pve-devel] [PATCH common 4/4] sysfs: use new PVE::RS::VFIO::Nvidia module to retrieve vGPU info

["Christoph Heiss" <c.heiss@proxmox.com>]

On Wed Jan 21, 2026 at 9:08 AM CET, Dominik Csapak wrote:
> On 1/20/26 4:00 PM, Thomas Lamprecht wrote:
>> Am 20.01.26 um 14:12 schrieb Christoph Heiss:
[..]
>>> +use PVE::RS::VFIO::Nvidia;
>>
>> we do not depend on libpve-rs-perl in libpve-common-perl, and if easily possible
>> I'd strongly favor keeping it that way,

Yeah, I'm not happy with that either - was also going to say (seems my
patch notes got lost somewhere) that using PVE::RS::VFIO means
get_mdev_types() doesn't only deal with sysfs stuff anymore - so might
make sense to move the mdev stuff anyway.

>> bootstrapping and handling bigger package
>> bumps with newer versioned dependencies or break/depends bumps is already quite
>> a bit of work as is, such stuff does not make it easier, especially if adding
>> a new dependencies is not recorded in d/control's packaging definition.

Right, I knew I forgot something - sorry.

>>
>> Can this move to some non-leaf package instead? Where are the users of this, only
>> qemu-server or other too (pve-manager?)?
[..]
>
> In summary I think simply moving the mdev/vgpu stuff to qemu-server
> makes the most sense for now.

A separate package for vfio/mdev/vgpu type stuff could make sense, as
that code will probably grow some more.
OTOH, if (most) new stuff is implemented through
proxmox-ve-rs/proxmox-perl-rs in the future, having it live in
qemu-server definitely makes a lot of sense too, I think.

I'm open to suggestions in any case, happy to rework that as needed.


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel