Re: [Qemu-devel] [PATCH v3] ui/cocoa.m: Machine menu patch for Mac OS X

[Kevin Wolf <kwolf@redhat.com>]

Am 14.02.2015 um 03:28 hat Peter Maydell geschrieben:
> On 14 February 2015 at 01:43, Programmingkid <programmingkidx@gmail.com> wrote:
> > Added features:
> > Menu items to switch floppy and CD image files.
> > Menu items to eject floppy and CD image files.
> > Menu item to use /dev/cdrom.
> > Verifies with the user before quitting QEMU by displaying a dialog box.
> >
> > Signed-off-by: John Arbuckle <programmingkidx@gmail.com>
> 
> Stefan, Kevin -- could you review the bits of this patch
> which determine whether the machine has a floppy/cdrom
> drive and if so let the user insert/inject it, please?
> (that's the emulatorHasDevice and ejectFloppy/changeFloppy
> functions, mostly). I don't know the block layer APIs so
> I can't really say if this patch is doing it in the best/
> non-deprecated/etc way or not...

Well, it's trying to detect the floppy/cdrom device by comparing string
with default IDs that can be overridden by the user, so no, that's
probably far from the best way to do it. The code also doesn't consider
that you could have more than one floppy or cdrom drive.

The correct way is probably to just display any removable block device,
and ideally also to implement some notifiers to deal with hotplug.

Kevin