Thread (7 messages) flat view 7 messages, 4 authors, 2021-01-06

Re: [PATCH] USB: usblp: fix DMA to stack

From: Michael Sweet <hidden>
Date: 2021-01-04 15:41:07
Also in: linux-usb

Johan/Greg,

Since CUPS uses libusb to communicate with printers these days (well, for over a decade now) so that printing and scanning can coexist, the usblp driver really doesn't get any usage anymore.

On Jan 4, 2021, at 10:11 AM, Johan Hovold [off-list ref] wrote:

On Mon, Jan 04, 2021 at 03:59:02PM +0100, Greg Kroah-Hartman wrote:
quoted
On Mon, Jan 04, 2021 at 03:53:02PM +0100, Johan Hovold wrote:
quoted
Stack-allocated buffers cannot be used for DMA (on all architectures).

Replace the HP-channel macro with a helper function that allocates a
dedicated transfer buffer so that it can continue to be used with
arguments from the stack.
quoted
Wow, no one uses this driver anymore it seems, this should have
triggered a runtime warning on newer kernels :(
This helper is only used by the IOCNR_HP_SET_CHANNEL ioctl so perhaps
it's just that one which isn't used much.

Johan
________________________
Michael Sweet


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help