[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event |
Date: |
Fri, 17 Feb 2012 09:56:07 -0200 |
On Fri, 17 Feb 2012 12:44:28 +0100
Markus Armbruster <address@hidden> wrote:
> >> I figure Luiz's patch works. But maybe it could be simplified some by
> >> replacing bdrv_dev_change_media_cb() by a "open/close tray" callback
> >> that returns whether it moved. bdrv_dev_change_media_cb() would then
> >> simply open the tray, emit event if it moved, close the tray, emit event
> >> if it moved.
> >
> > Yes, I had the same impression after reading the first few paragraphs of
> > your analysis (which looks right to me).
>
> If I understand Luiz correctly, he'd like to do that in a follow-up
> patch.
Yes, but as I said I'm not completely sure what's the best thing to do here,
and this may take some time, but I'll do it.
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, (continued)
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Markus Armbruster, 2012/02/16
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Luiz Capitulino, 2012/02/16
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Kevin Wolf, 2012/02/17
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Paolo Bonzini, 2012/02/17
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Markus Armbruster, 2012/02/17
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Markus Armbruster, 2012/02/17
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event,
Luiz Capitulino <=
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Paolo Bonzini, 2012/02/17
- Re: [Qemu-devel] [PATCH 4/4] qmp: add BLOCK_MEDIUM_EJECT event, Markus Armbruster, 2012/02/17
Re: [Qemu-devel] [PATCH v2 0/4]: QMP: add BLOCK_MEDIUM_EJECT event, Paolo Bonzini, 2012/02/17