linux-kernel - Re: [PATCH] Increase usbfs bulk buffer size

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <201108190823.06638.oliver@neukum.org>
Date:	Fri, 19 Aug 2011 08:23:06 +0200
From:	Oliver Neukum <oliver@...kum.org>
To:	Markus Rechberger <mrechberger@...il.com>
Cc:	"Greg Kroah-Hartman" <gregkh@...e.de>, linux-usb@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Increase usbfs bulk buffer size

Am Mittwoch, 17. August 2011, 19:44:16 schrieb Markus Rechberger:
> I understand that USBFS is not 100% optimized, although we are already
> using 3-4 times that much buffer with isochronous and it was working
> fine for years so far as long as the system doesn't run out of memory
> .. but even kerneldrivers would have issues with it in such a
> situation.

The problem is not what happens on your test system dedicated to this use.
There it'll work. But you allowed everybody else to make the kernel request
the second largest chunk of memory there is. On a otherwise busy system
with fragmented memory this is not good.

As I said, you want scatter/gather if you need this.
There still would be the problem with the API, but if this is really a problem,
you could define a new ioctl()

	Regards
		Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/