All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aurelien Jarno <aurelien@aurel32.net>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] hw/vexpress: set default block type to SD
Date: Fri, 15 Mar 2013 16:42:52 +0100	[thread overview]
Message-ID: <20130315154252.GC27432@hall.aurel32.net> (raw)
In-Reply-To: <CAFEAcA-25sTd00D_NPDH=c7bBqvQe74Nw0QMuUpryyj7C3fgVg@mail.gmail.com>

On Fri, Mar 15, 2013 at 03:35:48PM +0000, Peter Maydell wrote:
> On 5 March 2013 00:44, Aurelien Jarno <aurelien@aurel32.net> wrote:
> > On Tue, Mar 05, 2013 at 08:22:57AM +0800, Peter Maydell wrote:
> >> What effect does this actually have on the user experience?
> >
> > The effect is that the user don't has to specify the interface type.
> > Basically:
> >
> >   -drive file=/path/to/file,if=sd
> > can be replaced by
> >   -drive file=/path/to/file
> >
> > It means the user doesn't have to know the details of the machine to
> > know how to attach a disk. Note that the user here can also be a script,
> > which then becomes a bit simpler.
> 
> I'm not convinced this is a good thing -- I think you should have
> to know that you're attaching an SD card and not a hard disk,
> because the performance is much worse. In particular if you
> don't specify 'cache=writeback' your performance will be
> dreadful, so you need to do something different from hard
> disks anyhow.
> 

Being a good thing or not, the current code is wrong: the default
interface type is set to SCSI (I guess it has been copied from
versatilepb.c), while the vexpress machine has no SCSI 
interface.

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

  reply	other threads:[~2013-03-15 15:43 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-05  0:19 [Qemu-devel] [PATCH] hw/vexpress: set default block type to SD Aurelien Jarno
2013-03-05  0:22 ` Peter Maydell
2013-03-05  0:44   ` Aurelien Jarno
2013-03-06  9:25     ` Markus Armbruster
2013-03-15 15:35     ` Peter Maydell
2013-03-15 15:42       ` Aurelien Jarno [this message]
2013-03-15 15:45         ` Peter Maydell
2013-03-17  0:04           ` Aurelien Jarno
2013-03-17  0:23             ` Peter Maydell
2013-03-17 10:17               ` Aurelien Jarno
2013-03-17 10:56                 ` Peter Maydell
2013-03-17 18:51             ` Paolo Bonzini
2013-03-17 18:54               ` Peter Maydell
2013-03-17 19:22                 ` Aurelien Jarno
2013-03-18  8:31                 ` Paolo Bonzini
2013-03-17 19:20               ` Aurelien Jarno
2013-03-17 18:28       ` Paolo Bonzini

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=20130315154252.GC27432@hall.aurel32.net \
    --to=aurelien@aurel32.net \
    --cc=peter.maydell@linaro.org \
    --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.