lkml.org 
[lkml]   [2014]   [Jul]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] i2c: stub: Add support for SMBus block commands
Forgive me for jumping in.   I'm a noob at I2C.   But I have built a 
couple substantial error injection frameworks over the years, one for a
mainframe DBMS and one for Java's checked exceptions. So while I have
nothing useful to say about how to inject exceptions here, I have
thought a lot about use cases.

Failing all the time is the necessary first step. It allows for testing
error paths without manually inserting a failure and recompiling, and
makes it possible to build unit tests.

Failing randomly (or pseudo-randomly) is important for testing the
overall recovery mechanism, particularly where you have an inherently
unreliable subsystem like networks or I2C. By changing the failure
rate you can explore, for example, at what point the failure rate of the
lower level system becomes so great that it makes the upper level system
unreliable.

The one use case I would add, and it may be outside the scope here, is
data errors. I've been using the DDC protocol over I2C to communicate
with monitors. The DDC Get Capabilities request entails multiple
write/read exchanges, with responses of up to 37 bytes each. Most of
the time this works ok, but I have one monitor that produces a high
volume of data errors (double bytes or missing bytes). This is only
detected by examining the data itself (fixed fields and checksum).

Sanford



On 07/13/2014 08:46 AM, Guenter Roeck wrote:
> On 07/13/2014 08:13 AM, Jean Delvare wrote:
>> On Sun, 13 Jul 2014 08:04:54 -0700, Guenter Roeck wrote:
>>> On 07/13/2014 12:21 AM, Jean Delvare wrote:
>>>> Hi Guenter,
>>>>
>>>> On Sat, 12 Jul 2014 08:05:49 -0700, Guenter Roeck wrote:
>>>>> Any idea how we could inject errors ? Error path testing would be
>>>>> quite useful.
>>>>
>>>> Good idea. This should probably be done with a sysfs attribute so that
>>>> it can be turned on and off as desired. Off by default, of course.
>>>> Some
>>>> other subsystems already support error injection, you could check how
>>>> they are doing it, do that we do not diverge needlessly.
>>>>
>>>> Do you think there is any value in failing with different error codes,
>>>> or just -EIO is enough?
>>>
>>> How about writing the error code to return into the attribute ?
>>> Write anything negative, and it is returned as error. Write 0,
>>> and the driver works as normal.
>>
>> This is smart, I like it :)
>>
>>>> Do you think it should fail all the time when error injection is
>>>> enabled, or is there a value in having only a certain % of commands
>>>> fail?
>>>
>>> For my purposes I would want it to fail reliably. We could add some
>>> fanciness,
>>> though: Provide a second attribute which specifies how many
>>> operations should
>>> pass before the first failure.
>>
>> Let's start simple and just implement what you need.
>>
>
> I would actually benefit from both. The ability to return an error
> unconditionally
> lets me test the first error path. The ability to return an error
> starting with the
> n-th transfer lets me test the n-th error path.
>
> Guenter
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>



\
 
 \ /
  Last update: 2014-07-13 21:01    [W:0.055 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site