netdev - RE: [PATCH] NET_DMA: remove unused dma_memcpy_to_kernel

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <D5C1322C3E673F459512FB59E0DDC32903539FE1@orsmsx414.amr.corp.intel.com>
Date:	Mon, 30 Jul 2007 13:47:03 -0700
From:	"Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@...el.com>
To:	"pravin" <pravin.shelar@...il.com>,
	"David Miller" <davem@...emloft.net>
Cc:	"Nelson, Shannon" <shannon.nelson@...el.com>,
	<netdev@...r.kernel.org>, <viro@....linux.org.uk>,
	"Leech, Christopher" <christopher.leech@...el.com>,
	<andy.grover@...il.com>
Subject: RE: [PATCH] NET_DMA: remove unused dma_memcpy_to_kernel_iovec

> On 7/26/07, David Miller <davem@...emloft.net> wrote:
> > From: Shannon Nelson <shannon.nelson@...el.com>
> > Date: Tue, 24 Jul 2007 17:36:06 -0700
> >
> > > (repost - original eaten by vger?)
> > >
> > > Al Viro pointed out that dma_memcpy_to_kernel_iovec() really was 
> > > unreachable and thus unused.  The code originally was there to 
> > > support in-kernel dma needs, but since it remains unused, 
> we'll pull it out.
> > >
> > > Signed-off-by: Shannon Nelson <shannon.nelson@...el.com>
> >
> > Applied, thanks Shannon.
> 
> NET_DMA on kernel buffer is pretty useful in ndb, iSCSI 
> target and initiators which uses kernel buffer to receive 
> data. Are there any other issues with dma-memcpy on kernel 
> buffers, if not then following patch makes 
> dma_memcpy_to_kernel_iovec() reachable from tcp_recvmsg.
> I tested this patch and it work fine with unh iSCSI target.
> 
> 
> comments?
> 
> --pravin.

Pravin,
	Currently, NET_DMA only has one provider in the kernel, namely
TCP.  As the driver stands now, I'd like to keep this function out since
it really wasn't being used before.  Shannon has posted patches that
will hopefully make it into 2.6.24 that allow multiple clients to
request DMA channels, so things like iSCSI, ndb, UDP, and NFS can ask
for channels without depending on TCP.  Once we get to that point, if we
see the need for the function, we can add it back.
	We have tried getting iSCSI accelerated through NET_DMA already,
and didn't see the performance boost that made it worthwhile to do.
Once the multiple client channel patches are in, we can revisit that,
since it can be done "right."

Cheers,
-PJ Waskiewicz
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html