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]
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