Re: gadgetfs broken since 7f7f25e8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 02.03.2015 um 14:02 schrieb Alexander Holler:
Am 02.03.2015 um 12:39 schrieb Alexander Holler:
Am 02.03.2015 um 11:20 schrieb Al Viro:
On Mon, Mar 02, 2015 at 10:13:27AM +0100, Richard Weinberger wrote:
On Mon, Mar 2, 2015 at 9:28 AM, Alexander Holler
<holler@xxxxxxxxxxxxx> wrote:
Hello.

Commit 7f7f25e82d54870df24d415a7007fbd327da027b (introduced with
3.16) broke
dynamic changing of file_operations->[read|write].

At least gadgetfs is a victim.

Just for your amusement and as an example:

This bug lead to me to examine and search bugs in the userland piece I've tried to use and ended up in around

===
aholler@laptopahbt ~/Source/USBProxy.git/src $ PAGER= git diff 7d2506648e3404bf7070bae6ab8da4a702ed093c --stat doc/gadgetfs_kernel_above_3.15.patch | 50 +++++++++++++++++++++++++++++++++++++++++++++++
 src/Plugins/Hosts/GadgetFS_helpers.c     |   4 ++--
 src/Plugins/Hosts/HostProxy_GadgetFS.cpp |  12 ++++++++++++
 src/debian/header-check.c                |   1 -
 src/lib/CMakeLists.txt                   |   2 --
 src/lib/ConfigParser.cpp                 |   9 +++------
 src/lib/ConfigParser.h                   |   2 +-
 src/lib/FDInfo.c                         |   2 +-
src/lib/HaltSignal.c | 54 ---------------------------------------------------
 src/lib/HaltSignal.h                     |  19 ------------------
 src/lib/Injector.cpp                     |  23 +++++-----------------
 src/lib/Injector.h                       |  11 +++++++----
src/lib/Manager.cpp | 122 +++++++++++++++++++++++++++++++++++++++++++-----------------------------------------------------------------------
 src/lib/Manager.h                        |  15 +++++++-------
src/lib/PluginManager.cpp | 47 +++++++++++++++++++++++++++++++++-----------
 src/lib/Proxy.h                          |  12 ++++++++++++
src/lib/RelayReader.cpp | 39 ++++++++++++-------------------------
 src/lib/RelayReader.h                    |   9 ++++++---
src/lib/RelayWriter.cpp | 69 ++++++++++++++++------------------------------------------------
 src/lib/RelayWriter.h                    |   8 +++++---
 src/tools/usb-mitm.cpp                   |   2 --
 21 files changed, 223 insertions(+), 289 deletions(-)
 ===

without counting at least a dozen patches I did on that userland piece before those which are counted in the above stat. All in order to find the bug.

So, you can see, I've already spend some hours before I've dived into the kernel to search for the bug. Of course, the problem in the kernel is innocent for all the problems I've found in userland which lead me to the assumption that the -EINVAL returned from a read() after a poll() is because of some problem in userspace (like memory or stack corruption).

Just in case someone thinks I'm lazy because I don't want to rewrite gadgetfs and deal with kernel maintainers.

Regards,

Alexander Holler
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux