mbox series

[v2,0/3] scsi: smartpqi: fix DMA overlapping mappings asymmetry

Message ID 20230316140912.1038404-1-desnesn@redhat.com
Headers show
Series scsi: smartpqi: fix DMA overlapping mappings asymmetry | expand

Message

Desnes Nunes March 16, 2023, 2:09 p.m. UTC 
In summary, this series fixes an overlapping mappings asymmetry on the
smartpqi driver due to a missing pqi_pci_unmap() call, while also adding
the cacheline on debug messages of dma-debug debugging functions.

Other minor non-functional updates are also provided.

Desnes Nunes (3):
  dma-debug: small dma_debug_entry's comment and variable name updates
  dma-debug: add cacheline to user/kernel space dump messages
  scsi: smartpqi: fix overlapping mappings asymmetry on DMA

 drivers/scsi/smartpqi/smartpqi_init.c |   2 +
 kernel/dma/debug.c                    | 133 ++++++++++++++------------
 2 files changed, 72 insertions(+), 63 deletions(-)

Comments

Christoph Hellwig March 28, 2023, 1:37 a.m. UTC | #1 
Thanks,

I've applied patches 1 and 2 to the dma mapping tree.  You might want
to resent patch 1 to the scsi maintainers and list to better get it
onto their radar.
Geert Uytterhoeven March 29, 2023, 7:15 a.m. UTC | #2 
Hi Desnes,

On Thu, Mar 16, 2023 at 3:13 PM Desnes Nunes <desnesn@redhat.com> wrote:
> In summary, this series fixes an overlapping mappings asymmetry on the
> smartpqi driver due to a missing pqi_pci_unmap() call, while also adding
> the cacheline on debug messages of dma-debug debugging functions.
>
> Other minor non-functional updates are also provided.

Thanks for your series!

Unfortunately this v2 does not fix the build issues reported by the
bots, thus breaking linux-next on plain 32-bit.

I have sent a fix
https://lore.kernel.org/r/20230329071405.1568021-1-geert+renesas@glider.be

Gr{oetje,eeting}s,

                        Geert


--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds