public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
From: Thomas Fourier <fourier.thomas@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: "Eric Dumazet" <edumazet@google.com>,
	"Andrew Lunn" <andrew+netdev@lunn.ch>,
	"David S. Miller" <davem@davemloft.net>,
	"Paolo Abeni" <pabeni@redhat.com>,
	"Uwe Kleine-König" <u.kleine-koenig@baylibre.com>,
	"Moritz Fischer" <mdf@kernel.org>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	"Baptiste Lepers" <baptiste.lepers@gmail.com>
Subject: Re: [PATCH net] net: ethernet: nixge: Add missing check after DMA map
Date: Mon, 28 Jul 2025 15:56:41 +0200	[thread overview]
Message-ID: <ab715726-77e2-4af7-bc1d-98a92a653617@gmail.com> (raw)
In-Reply-To: <20250725081045.34ac4130@kernel.org>

On 25/07/2025 17:10, Jakub Kicinski wrote:
> On Fri, 25 Jul 2025 06:53:16 -0700 Eric Dumazet wrote:
>> Not sure if this driver is actively used...
> Like most of the drivers that are missing dma_mappnig_error() :(
>
> Thomas, would it be possible for you to sort the drivers you have
> reports for in the reverse chronological order (when they were added
> or last time they had significant work done on them)? Start from
> the most recent ones?

This is already one of the most recent drivers with missing 
dma_mapping_error(),

I have patched the missing dma_mapping_error from 2017 onward that I 
could find.

Here are the drivers that are missing a dma_ampping_error() that I could 
find sorted by

the last date in their copyright notice:

2017 drivers/net/ethernet/ni/nixge.c
2017 drivers/scsi/aacraid/commctrl.c
2013 drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c
2012 drivers/net/ethernet/rdc/r6040.c
2011 drivers/scsi/isci/request.c
2010 drivers/tty/serial/amba-pl011.c
2010 drivers/net/ethernet/neterion/s2io.c
2010 drivers/net/ethernet/micrel/ksz884x.c
2010 drivers/net/ethernet/marvell/pxa168_eth.c
2009 crypto/async_tx/async_pq.c
2008 drivers/message/fusion/mptlan.c
2008 drivers/message/fusion/mptctl.c
2008 drivers/atm/solos-pci.c
2007 sound/pci/sis7019.c
2007 drivers/scsi/initio.c
2007 drivers/scsi/esp_scsi.c
2007 drivers/net/ethernet/tehuti/tehuti.c
2007 drivers/media/pci/ivtv/ivtv-udma.c
2007 drivers/crypto/hifn_795x.c
2006 crypto/async_tx/async_xor.c
2006 crypto/async_tx/async_memcpy.c
2005 drivers/scsi/megaraid.c
2005 drivers/net/ethernet/chelsio/cxgb/sge.c
2005 drivers/net/ethernet/dec/tulip/uli526x.c
2004 drivers/net/ethernet/via/via-velocity.c
2004 drivers/net/ethernet/sun/sungem.c
2004 drivers/net/ethernet/marvell/mv643xx_eth.c
2003 drivers/tty/serial/atmel_serial.c
2003 drivers/scsi/ips.c
2003 drivers/scsi/dc395x.c
2003 drivers/net/wan/wanxl.c
2003 drivers/net/ethernet/sun/cassini.c
2002 drivers/net/hippi/rrunner.c
2002 drivers/net/ethernet/ti/tlan.c
2002 drivers/net/ethernet/alteon/acenic.c
2002 crypto/async_tx/async_raid6_recov.c
2001 drivers/scsi/53c700.c
2001 drivers/parport/parport_pc.c
2001 drivers/net/ethernet/smsc/epic100.c
2001 drivers/net/ethernet/packetengines/yellowfin.c
2001 drivers/net/ethernet/natsemi/ns83820.c
2001 drivers/net/ethernet/dlink/dl2k.c
2001 drivers/net/ethernet/dec/tulip/winbond-840.c
2001 drivers/net/ethernet/dec/tulip/tulip_core.c
2001 drivers/net/ethernet/dec/tulip/de2104x.c
2001 drivers/atm/he.c
2000 drivers/net/ethernet/toshiba/tc35815.c
2000 drivers/net/ethernet/packetengines/hamachi.c
2000 drivers/net/ethernet/fealnx.c
2000 drivers/net/ethernet/amd/amd8111e.c
2000 drivers/net/ethernet/3com/typhoon.c
1999 drivers/net/fddi/skfp/skfddi.c
1999 drivers/net/ethernet/3com/3c59x.c
1999 drivers/atm/nicstar.c
1998 drivers/net/wireless/intel/ipw2x00/ipw2200.c
1998 drivers/net/ethernet/dec/tulip/dmfe.c
1997 drivers/scsi/aha1740.c


      parent reply	other threads:[~2025-07-28 13:56 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-25 13:33 [PATCH net] net: ethernet: nixge: Add missing check after DMA map Thomas Fourier
2025-07-25 13:53 ` Eric Dumazet
2025-07-25 15:10   ` Jakub Kicinski
2025-07-25 15:47     ` Eric Dumazet
2025-07-28 13:56     ` Thomas Fourier [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ab715726-77e2-4af7-bc1d-98a92a653617@gmail.com \
    --to=fourier.thomas@gmail.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=baptiste.lepers@gmail.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mdf@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=u.kleine-koenig@baylibre.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox