From: "Michael S. Tsirkin" <mst@redhat.com>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Richard Henderson <rth@twiddle.net>, John Snow <jsnow@redhat.com>,
qemu-devel@nongnu.org, Hannes Reinecke <hare@suse.de>
Subject: Re: [Qemu-devel] [PATCH 0/2] Don't require hw_version to be set on every machine
Date: Mon, 23 Nov 2015 14:54:51 +0200 [thread overview]
Message-ID: <20151123145414-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <20151123125339.GL20436@thinpad.lan.raisama.net>
On Mon, Nov 23, 2015 at 10:53:39AM -0200, Eduardo Habkost wrote:
>
> Ping? Any thoughts on this approach?
>
> -rc2 is too late for this kind of change, maybe?
>
I queued this up for 2.5, will merge later this week
unless anyone objects.
> On Thu, Nov 12, 2015 at 03:29:53PM -0200, Eduardo Habkost wrote:
> > This changes the qemu_hw_version() default to "2.5+" so we don't
> > make every machine class broken by default unless they set
> > hw_version.
> >
> > For reference, these are the current users of qemu_hw_version():
> >
> > hw/arm/nseries.c: pstrcat((void *) w, 12, qemu_hw_version()); /* char version[12] */
> > hw/ide/core.c: pstrcpy(s->version, sizeof(s->version), qemu_hw_version());
> > hw/scsi/megasas.c: snprintf(info.package_version, 0x60, "%s-QEMU", qemu_hw_version());
> > hw/scsi/scsi-bus.c: pstrcpy((char *) &r->buf[32], 4, qemu_hw_version());
> > hw/scsi/scsi-disk.c: s->version = g_strdup(qemu_hw_version());
> > target-i386/cpu.c: qemu_hw_version());
> >
> > Cc: Andrzej Zaborowski <balrogg@gmail.com>
> > Cc: John Snow <jsnow@redhat.com>
> > Cc: Paolo Bonzini <pbonzini@redhat.com>
> > Cc: Hannes Reinecke <hare@suse.de>
> > Cc: Richard Henderson <rth@twiddle.net>
> >
> > Eduardo Habkost (2):
> > osdep: Change default value of qemu_hw_version() to "2.5+"
> > pc: Don't set hw_version on pc-*-2.5
> >
> > hw/i386/pc_piix.c | 1 -
> > hw/i386/pc_q35.c | 1 -
> > include/hw/boards.h | 5 +++++
> > include/qemu/osdep.h | 4 ++++
> > util/osdep.c | 9 ++++++++-
> > 5 files changed, 17 insertions(+), 3 deletions(-)
> >
> > --
> > 2.1.0
> >
> >
>
> --
> Eduardo
prev parent reply other threads:[~2015-11-23 12:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-12 17:29 [Qemu-devel] [PATCH 0/2] Don't require hw_version to be set on every machine Eduardo Habkost
2015-11-12 17:29 ` [Qemu-devel] [PATCH 1/2] osdep: Change default value of qemu_hw_version() to "2.5+" Eduardo Habkost
2015-11-12 17:29 ` [Qemu-devel] [PATCH 2/2] pc: Don't set hw_version on pc-*-2.5 Eduardo Habkost
2015-11-23 12:53 ` [Qemu-devel] [PATCH 0/2] Don't require hw_version to be set on every machine Eduardo Habkost
2015-11-23 12:54 ` Michael S. Tsirkin [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=20151123145414-mutt-send-email-mst@redhat.com \
--to=mst@redhat.com \
--cc=ehabkost@redhat.com \
--cc=hare@suse.de \
--cc=jsnow@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).