From: "Andreas Färber" <afaerber@suse.de>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
Markus Armbruster <armbru@redhat.com>,
Alexander Graf <agraf@suse.de>
Subject: Re: [Qemu-devel] [PATCH 1/2] qom-test: Run for all available machines
Date: Thu, 23 Jan 2014 16:45:41 +0100 [thread overview]
Message-ID: <52E13925.5000506@suse.de> (raw)
In-Reply-To: <CAFEAcA-q6zQZerT5Z_c=5Wjd9hafCad3p=ybtz_=r4zADO9otA@mail.gmail.com>
Am 23.01.2014 16:40, schrieb Peter Maydell:
> On 23 January 2014 14:41, Andreas Färber <afaerber@suse.de> wrote:
>> As indicated on IRC, I want to get those fixed before transforming one
>> notation to another. Especially this patch is throwing them all in one
>> bucket dropping any annotation hinting at why and how it may be fixed.
>
> Would you accept a version where every item on the black list
> had a comment describing why it was there and what the required
> fix is? I think this patch is a vast improvement on the current
> whitelist setup and we shouldn't block it waiting for every
> obscure target to get fixed.
Like I said, I have already sent a patch for PReP, which Alex has
queued. The intent is to rebase this series on it, obsoleting part of
the blacklist.
I have also now sent out a patch covering Macs and e500. So the answer
is, I will accept a patch that has a reasonably small blacklist. The
time you're arguing about this on IRC and here you could've spent
actually helping investigate and fixing this... Even if I queued it and
sent a pull, it would take days or weeks to go in, so I really don't see
why I should rush this series into my tree.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
next prev parent reply other threads:[~2014-01-23 15:45 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-10 13:31 [Qemu-devel] [PATCH 0/2] qom-test: Improve coverage armbru
2014-01-10 13:31 ` [Qemu-devel] [PATCH 1/2] qom-test: Run for all available machines armbru
2014-01-10 13:42 ` Markus Armbruster
2014-01-10 13:51 ` Alexander Graf
2014-01-10 14:17 ` Andreas Färber
2014-01-23 14:41 ` Andreas Färber
2014-01-23 15:40 ` Peter Maydell
2014-01-23 15:45 ` Andreas Färber [this message]
2014-01-23 16:00 ` Peter Maydell
2014-01-23 17:14 ` Andreas Färber
2014-01-23 14:54 ` Markus Armbruster
2014-01-31 12:33 ` Andreas Färber
2014-01-31 13:42 ` Markus Armbruster
2014-02-03 12:40 ` Markus Armbruster
2014-02-05 15:47 ` Andreas Färber
2014-02-05 17:51 ` Stefano Stabellini
2014-01-10 13:31 ` [Qemu-devel] [PATCH 2/2] qom-test: Test shutdown in addition to startup armbru
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=52E13925.5000506@suse.de \
--to=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=armbru@redhat.com \
--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.