Oliver Neukum <oneu...@suse.de> writes:

> On Thu, 2015-01-15 at 17:52 +0100, Bjørn Mork wrote:
>> Oliver Neukum <oneu...@suse.de> writes:
>> 
>> > Hi,
>> >
>> > I think I found a race between pre_reset() and the bottom half.
>> > In wdm_rxwork() WDM_RESETTING is not checked for. That means that
>> >
>> > wdm_pre_reset() cannot reliable end IO. What do you think?
>> 
>> Yes, it certainly looks that way.  There are too many flags meaning
>> "end IO"... 
>
> So you think I should consolidate the flags rather add a check?

If possible.  At least that was my initial thought after a quick look.
But I don't know how feasible it is.  I can imagine that we might need
to separate the flags we want to clear.  At least the ones that can be
set at the same time, if any.


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

Reply via email to