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: <52643326-3a43-43b3-9be1-f13a255a7fc8@rowland.harvard.edu>
Date:   Wed, 13 Sep 2023 17:11:22 -0400
From:   Alan Stern <stern@...land.harvard.edu>
To:     Greg KH <gregkh@...uxfoundation.org>
Cc:     Randy Li <ayaka@...lik.info>, linux-usb@...r.kernel.org,
        Linux Media Mailing List <linux-media@...r.kernel.org>,
        linux-kernel@...r.kernel.org, corbet@....net,
        linux-doc@...r.kernel.org
Subject: Re: USB: DMA: mapping existing buffer is not supported?

On Wed, Sep 13, 2023 at 10:32:12PM +0200, Greg KH wrote:
> On Thu, Sep 14, 2023 at 04:09:06AM +0800, Randy Li wrote:
> > 
> > On 2023/9/14 03:19, Greg KH wrote:
> > > On Thu, Sep 14, 2023 at 03:06:58AM +0800, Randy Li wrote:
> > > > Hello
> > > > 
> > > > I was trying to understand why USB webcams (UVC) have to copy video data
> > > > through the CPU (uvc_video_complete() schedules uvc_video_copy_data_work()
> > > > for this purpose). During my investigation, I noticed that functions
> > > > like|usb_sg_*() and |usb_buffer_*() are not available since kernel version
> > > > 2.6.12.
> > > What do you mean by "not available"?  I see them in the tree today, why
> > > do you think they are not present?
> > > 
> > usb_buffer_dmasync_sg(), usb_buffer_map(), usb_buffer_dmasync() and usb_buffer_unmap() are all disabled
> > by #if 0 in include/usb/usb.h
> > 
> > usb_buffer_map_sg() and usb_buffer_unmap_sg() are just declaration without definition.
> 
> Sorry, I was looking at the usb_sg_* calls, those are there if you want
> to use them.
> 
> But again, why not just use the normal sg field in the urb itself for
> the scatter-gather pointer?  Will that not work for you?
> 
> > > > If the USB subsystem can no longer work with existing buffers, I propose
> > > > that we consider removing the remaining documentation in the "Working with
> > > > existing buffers" section of Documentation/driver-api/usb/dma.rst.
> > > I don't understand, what is wrong with the information there exactly?
> > > Have you tried following the suggestions there?
> > Besides my answer to first question, I found no code use them today.

That's exactly what it says at the end of the file:

----------------------------------------------------------------------------
Note that several of those interfaces are currently commented out, since
they don't have current users.  See the source code.  Other than the dmasync
calls (where the underlying DMA primitives have changed), most of them can
easily be commented back in if you want to use them.
----------------------------------------------------------------------------

So Randy, the answer to your question is that those routines are not 
available any more because nothing needs to use them.  If you would like 
to submit a patch removing the "Working with existing buffers" section 
from the documentation file, feel free to do so.

Alan Stern

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ