Il 31/05/2012 14:33, Paolo Bonzini ha scritto:
>>>> >> >          case 0x02: /* write cache enable */
>>>> >> > +            bdrv_set_enable_write_cache(s->bs, true);
>>>> >> > +            identify_data = (uint16_t *)s->identify_data;
>>>> >> > +            put_le16(identify_data + 85, (1 << 14) | (1 << 5) | 1);
>> > How about just s->identify_set = 0; instead, so that all IDENTIFY logic
>> > stays together in one place?
>> > 
> Good idea.

Hmm, this is a rat's nest because of the other patching we do for "set
transfer mode".  I'd prefer to refactor it later.

Paolo

Reply via email to