[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive |
Date: |
Mon, 17 May 2010 17:23:15 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.33-2-amd64; KDE/4.4.3; x86_64; ; ) |
> > I don't see a difference between the results. Apparently the barrier
> > option doesn't change a thing.
>
> Ok. I don't like it, but I can see how it's compelling. I'd like to
> see the documentation improved though. I also think a warning printed
> on stdio about the safety of the option would be appropriate.
I disagree with this last bit.
Errors should be issued if the user did something wrong.
Warnings should be issued if qemu did (or will soon do) something other than
what the user requested, or otherwise made questionable decisions on the
user's behalf.
In this case we're doing exactly what the user requested. The only plausible
failure case is where a user is blindly trying options that they clearly don't
understand or read the documentation for. I have zero sympathy for complaints
like "Someone on the Internet told me to use --breakme, and broke thinks".
Paul
- [Qemu-devel] [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Kevin Wolf, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive,
Paul Brook <=
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Jamie Lokier, 2010/05/17
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Kevin Wolf, 2010/05/18
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/25
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/25
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Alexander Graf, 2010/05/25
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Avi Kivity, 2010/05/26
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Aurelien Jarno, 2010/05/25
- Re: [Qemu-devel] Re: [PATCH] Add cache=volatile parameter to -drive, Anthony Liguori, 2010/05/25