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