Re: [PATCH] JMicron JM20337 USB-SATA data corruption bugfix - device 152d:2338

[Robert Hancock <hancockr@shaw.ca>]

Tomas Styblo wrote:
> * Robert Hancock <hancockr@shaw.ca> [Thu, 24 Jul 2008]:
>>>> Anyway, I'll send the patch to this person so he can try it. I guess it 
>>>> won't fix his problem. This patch is much simpler and doesn't need any 
>>>> delays - I really think this is a different situation.
>>> It isn't.  And your patch is an ad-hoc correction that doesn't address
>>> the true underlying reasons for the errors.
>> Tomas, you should try the patch listed in that bug report (well, except 
>> maybe the part that's actually adding the JMicron unusual devices entry) 
>> and see if that fixes the problem.
> 
> I'll try that patch next week - but I have no confidence in the
> reliability of fixing the problem by inserting random delays. All
> I have wanted to achieve is to detect the errors and thus prevent
> the silent data corruption. 
> 
> I don't really mind those sporadic forced resets since they cause
> no data corruption, at least in my setup. My simple patch
> accomplishes the goal of preventing data corruption more reliably,
> I think.

I agree, that's why I mentioned to leave out the part that adds the 
unusual devices entry which actually enables the random delays. There's 
another part of the patch which fixes underflow detection, which might 
have the same effect as your patch.