Re: [PATCH] Increase usbfs bulk buffer size

From: Greg KH
Date: Thu Aug 18 2011 - 14:40:41 EST


On Wed, Aug 17, 2011 at 07:07:01PM +0200, Markus Rechberger wrote:
> Hi,
>
> this patch increases the maximum buffersize for bulk transfers, our
> devices support at least up to 46k bytes for bulk transfers.
> This patch allows us to lower the iterations between kernel and
> userspace and lower the system pressure.

But as Oliver pointed out, this increases the memory pressure. I'm
curious why you say there is "system pressure" with the current buffer
size. What is the problems here, you are just passing the same logic
issues down to the kernel, that you should be handling in userspace if
you want to use usbfs.

> Signed-off-by: Markus Rechberger <mrechberger@xxxxxxxxx>

Also note that this is a user/kernel API that you are changing, are you
_sure_ you didn't just break some user code with this change that was
assuming that the buffer size was 16Kb, as it's been that way for the
past 10 years?

thanks,

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