[s]sd-config-list version 2 and retry count tuning [PSARC/2007/505 FastTrack timeout 09/11/2007]

[Bill Sommerfeld]

On Wed, 2007-09-05 at 19:18 +0800, yu larry liu wrote:
> We didn't change the syntax for property encoding from version 1.  I 
> agree with you it is error prone. But it is simple and efficient. And it 
> leaves door open to future version development. If splitting the 
> properties, it will be complex for the combination of VID, PID and 
> single property.  Any suggestion?

I'm not 100% familiar with the constraints on entries in driver.conf
files but an attribute=value encoding like:

t4-data="nonvolatile-cache=1,timeout-retry-count=42,min-throttle=2,max-throttle=17";

would be way better than

t4-data=2,0x303,2,17,0,0,0,0,0,0,0,42,1

(I *think* I got that right).

(Maybe the answer here is to provide tools or scripts to construct and
decode these entries, which wouldn't be this project but a worthy
followon).  

> In version 2, we plan to only support those properties defined in 
> sd_tunables table which is commonly used.

The word "support" is ambiguous and should generally be avoided in
specifications.  I think you mean "allow".  

BTW, it looks like the spec is missing a reference to 
2007/053 (for the "non-volatile cache" bit) and perhaps a few others.

					- Bill