On Wed, Apr 16, 2025 at 06:20:50AM +0000, Chen, Jiqian wrote: > On 2025/4/15 21:45, Roger Pau Monné wrote: > > On Wed, Apr 09, 2025 at 02:45:28PM +0800, Jiqian Chen wrote: > >> + > >> + for ( unsigned int i = 0; i < ARRAY_SIZE(vpci->msix->table); i++ ) > >> + if ( vpci->msix->table[i] ) > >> + iounmap(vpci->msix->table[i]); > > > > The MSI-X init function never maps tables, so the code here (given > > it's current usage) will also never unmap anything. However I would > > also like to use this code for vPCI deassing, at which point the logic > > will get used. > So, I still need to keep this part, right?
Yes, I guess leave it for now, we can always rip it later. I would like to see how doing the cleanup in the caller in case of failure looks. Then we can likely extend such cleanup to be done in the deassign case also. Thanks, Roger.
