On 5/27/24 11:09 PM, Nikita Shubin via B4 Relay wrote: > Started with observing leakage in patch 3, investigating revealed much > more problems in probing error path. > > Andy you are always welcome to review if you have a spare time. > > Thank you Andy and Markus for your comments. > > Signed-off-by: Nikita Shubin <n.shubin@xxxxxxxxx> > --- > Changes in v2: > - dmaengine: ioatdma: Fix error path in ioat3_dma_probe(): > Markus: > - fix typo > > - dmaengine: ioatdma: Fix kmemleak in ioat_pci_probe() > Andy: > - s/int/unsigned int/ > - fix spelling errors > - trimmed kmemleak reports > > - Link to v1: https://lore.kernel.org/r/20240524-ioatdma-fixes-v1-0-b785f1f7accc@xxxxxxxxx > > --- > Nikita Shubin (3): > dmaengine: ioatdma: Fix leaking on version mismatch > dmaengine: ioatdma: Fix error path in ioat3_dma_probe() > dmaengine: ioatdma: Fix kmemleak in ioat_pci_probe() > > drivers/dma/ioat/init.c | 54 ++++++++++++++++++++++++++----------------------- > 1 file changed, 29 insertions(+), 25 deletions(-) > --- > base-commit: 6d69b6c12fce479fde7bc06f686212451688a102 > change-id: 20240524-ioatdma-fixes-a8fccda9bd79 Thanks for the fixes. Reviewed-by: Dave Jiang <dave.jiang@xxxxxxxxx> for the series Would be nice if someone wants to move everything to the devm_* management APIs. Would make this a lot less messy. Probably not worth the effort though given how old the driver is and no more devices are being created to use this driver. > > Best regards,
