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
| ||
|
Date: Thu, 13 Nov 2014 17:28:22 +0100 From: Krzysztof Opasiak <k.opasiak@...sung.com> To: balbi@...com Cc: 'Robert Baldyga' <r.baldyga@...sung.com>, 'David Cohen' <david.a.cohen@...ux.intel.com>, gregkh@...uxfoundation.org, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org, mina86@...a86.com, andrzej.p@...sung.com, m.szyprowski@...sung.com Subject: RE: [PATCH v4] usb: gadget: f_fs: add "no_disconnect" mode Hello, > -----Original Message----- > From: Felipe Balbi [mailto:balbi@...com] > Sent: Thursday, November 13, 2014 3:44 AM > To: Felipe Balbi > Cc: Robert Baldyga; David Cohen; gregkh@...uxfoundation.org; linux- > usb@...r.kernel.org; linux-kernel@...r.kernel.org; > mina86@...a86.com; andrzej.p@...sung.com; m.szyprowski@...sung.com; > k.opasiak@...sung.com > Subject: Re: [PATCH v4] usb: gadget: f_fs: add "no_disconnect" mode (...) > > > David, you have been messing with ffs lately, can I get a > Tested-by > > > from you on this patch ? > > > > ping. David ? Any chance you can test this one out ? > > a reminder on this request, it's getting close to tree lockdown > (probably another 2 to 3 weeks). I have tested this patch today but I'm afraid that I can't give tested-by tag: When I kill the daemon and then reconnect my device I get on my host: [776092.877175] usb 1-1.1: can't set config #1, error -32 While on device there is only: [ 2147.035340] configfs-gadget gadget: high-speed config #1: The only one Moreover, I'm not very enthusiastic about the convention in which we allow to reconnect the device without any changes. In my opinion when daemon has been killed we should allow user to finish communication with other functions and then simply unbind our gadget. This should happen not only on reconnection but also when host is trying to enable configuration which contains this function "zombie". This function is not fully operational so it is very good idea to allow user to finish his job with other functions but when he reconnects the device we should unbind gadget to notify userspace that something is wrong. The same when we switch configuration. What do you think about this Michal and Felipe? -- Krzysiek -- 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/
Powered by blists - more mailing lists