RE: [PATCH v2] bus/pci: fix legacy device IO port map in secondary process

2023-08-21 Thread Ma, WenwuX
Re: [PATCH v2] bus/pci: fix legacy device IO port map in secondary > process > > On Mon, 21 Aug 2023 09:27:07 +0800 > Wenwu Ma wrote: > > > + struct vfio_device_info device_info = { .argsz = sizeof(device_info) }; > > + char pci_addr[PATH_MAX] = {0} > > Not

Re: [PATCH v2] bus/pci: fix legacy device IO port map in secondary process

2023-08-20 Thread Stephen Hemminger
On Mon, 21 Aug 2023 09:27:07 +0800 Wenwu Ma wrote: > + struct vfio_device_info device_info = { .argsz = sizeof(device_info) }; > + char pci_addr[PATH_MAX] = {0} Not sure if some tools will complain about initializing chars as zero. Anyway, why bother since you are using it with snprintf.

[PATCH v2] bus/pci: fix legacy device IO port map in secondary process

2023-08-20 Thread Wenwu Ma
When doing IO port mapping for legacy device in secondary process, the region information is missing, so, we need to refill it. Fixes: 4b741542ecde ("bus/pci: avoid depending on private kernel value") Cc: sta...@dpdk.org Signed-off-by: Wenwu Ma --- v2: add release of device in pci_vfio_ioport_u