From: Laurent Vivier <Laurent.Vivier@bull.net>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qemu vl.c hw/omap.c
Date: Tue, 04 Dec 2007 10:28:49 +0100 [thread overview]
Message-ID: <1196760529.4095.10.camel@frecb07144> (raw)
In-Reply-To: <200712040120.26286.paul@codesourcery.com>
[-- Attachment #1: Type: text/plain, Size: 1708 bytes --]
Le mardi 04 décembre 2007 à 01:20 +0000, Paul Brook a écrit :
> On Tuesday 04 December 2007, andrzej zaborowski wrote:
> > On 04/12/2007, Paul Brook <paul@codesourcery.com> wrote:
> > > > Log message:
> > > > Always create an SD bdrv, so that PXA and OMAP boards can boot
> > > > with no card inserted again. Eventually SD, CDROM and floppy should
> > > > all be registered conditionally depending on machine.
> > >
> > > This seems the wrong way to solve this problem. The SD emulation should
> > > be able to cope with no device being present.
> >
> > The bdrv's represent the concept of "drive" rather than "media", and
> > also in the case the bdrv must be present whenever there is a SD slot,
> > not an SD card (so that monitor command change and eject can be used).
> > I agree that not all machines have an SD slot, as not all machines
> > have a cd-rom drive or floppy drive.
>
> Right, but I think you're going in the wrong direction. Most scsi interfaces
> do support hotplug of devices. For USB mass storage the whole controller is
> hotplug. Pre-allocating all possible devices simply doesn't scale.
Hi Paul,
I think Andrzez is doing it correctly.
He only adds an entry in drives_table[] without creating any bdrv
(because he doesn't provide file).
Then in omap.c, he creates the device, not the media. And as Palm
Tunsgten is the only machine calling omap310_mpu_init(), the device is
only created for it. And then he will be able to introduce card (file)
in the SD reader.
Laurent
--
------------- Laurent.Vivier@bull.net --------------
"Any sufficiently advanced technology is
indistinguishable from magic." - Arthur C. Clarke
[-- Attachment #2: Ceci est une partie de message numériquement signée --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
prev parent reply other threads:[~2007-12-04 9:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-04 0:10 [Qemu-devel] qemu vl.c hw/omap.c Andrzej Zaborowski
2007-12-04 1:02 ` Paul Brook
2007-12-04 1:11 ` andrzej zaborowski
2007-12-04 1:20 ` Paul Brook
2007-12-04 1:28 ` andrzej zaborowski
2007-12-04 9:28 ` Laurent Vivier [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1196760529.4095.10.camel@frecb07144 \
--to=laurent.vivier@bull.net \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.