Xen devel - May 2008 - [PATCH] qemu: Send unit_attention on cd-rom not_ready to ready transition

Patch sends a UNIT_ATTENTION(6), MEDIUM_MAY_HAVE_CHANGED(0x28) sense
when cdrom transitions from not ready to ready. 

ATA Packet interface for CD-ROMS, SFF8020i.pdf. See state diagram Figure
12, page 82 and Table 44 -recommended Sense Key, ASC

With patch in place HVM win2008 server guest sees the CD/DVD contents
have changed when the media is switched.

Please apply to xen-unstable tip.

Signed-off-by: Pat Campbell <plc@novell.com>




_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Pat Campbell writes ("[Xen-devel][PATCH] qemu: Send unit_attention on
cd-rom not_ready to ready transition"):
> Patch sends a UNIT_ATTENTION(6), MEDIUM_MAY_HAVE_CHANGED(0x28) sense
> when cdrom transitions from not ready to ready. 
It seems to me that this patch should go upstream to qemu.
Is there some reason why not ?

If it should, please prepare a version of it against qemu CVS head and
submit it to the qemu list (qemu-devel@nongnu.org).  I''ll keep an eye
on it there too, as the qemu guys are quite busy and have a tendency
to overlook things so a reminder can be necessary.

in general it would be helpful if contributors helped keep to a
minimum the non-Xen-specific changes to ioemu.  For larger patches
which need to vary substantially because of the existing differences
between upstream and ioemu, it''s probably easier for someone who knows
those differences (eg, me) to do this.  But I think that at least for
a small independent change like this it would be good for patch
submitters to talk to upstream as well as us.

I haven''t double-checked the ATA spec but your patch looks reasonable
for xen-unstable, anyway.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Ian Jackson wrote:
> Pat Campbell writes ("[Xen-devel][PATCH] qemu: Send unit_attention on
cd-rom not_ready to ready transition"):
>   
>> Patch sends a UNIT_ATTENTION(6), MEDIUM_MAY_HAVE_CHANGED(0x28) sense
>> when cdrom transitions from not ready to ready. 
>>     
>
> It seems to me that this patch should go upstream to qemu.
> Is there some reason why not ?
>   
Submitted upstream to qemu this morning.
> If it should, please prepare a version of it against qemu CVS head and
> submit it to the qemu list (qemu-devel@nongnu.org).  I''ll keep an
eye
> on it there too, as the qemu guys are quite busy and have a tendency
> to overlook things so a reminder can be necessary.
>
> in general it would be helpful if contributors helped keep to a
> minimum the non-Xen-specific changes to ioemu.  For larger patches
> which need to vary substantially because of the existing differences
> between upstream and ioemu, it''s probably easier for someone who
knows
> those differences (eg, me) to do this.  But I think that at least for
> a small independent change like this it would be good for patch
> submitters to talk to upstream as well as us.
>
> I haven''t double-checked the ATA spec but your patch looks
reasonable
> for xen-unstable, anyway.
>
> Thanks,
> Ian.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>   

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel