[PATCH 0/2] f_fs: better handle excess data on read

From: Michal Nazarewicz
Date: Sat May 21 2016 - 14:48:01 EST


Between this set and Du, Changbinâs patch, we now have implementation
for the following three possibilities to choose from:
1. Buffer excess data (this whole patch set).
2. Fail the transfer (Du, Changbinâs patch).
3. Drop excess data (the first patch from this set).

As per earlier comments, I think 3. is the correct, i.e. the second
patch should not be ocmmited because:
* it complicates the code,
* doesnât handle AIO anyway,
* introduces weird behaviours when partial read happens just before
endpoint is disabled (we may end up silently dropping excess data
anyway),
* goes beyond what UDC does and
* breaks one read -> one request model which has been true so far.

Michal Nazarewicz (2):
usb: gadget: f_fs: printk error when excess data is dropped on read
usb: gadget: f_fs: buffer data from âoversizedâ OUT requests

drivers/usb/gadget/function/f_fs.c | 179 ++++++++++++++++++++++++++++++-------
1 file changed, 145 insertions(+), 34 deletions(-)

--
Best regards
ããã âðððð86â ãããããã
ÂIf at first you donât succeed, give up skydivingÂ