All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 0/3] Migration-safe ACPI table sizing algorithm
Date: Thu, 20 Nov 2014 13:52:47 +0200	[thread overview]
Message-ID: <20141120115247.GA4973@redhat.com> (raw)
In-Reply-To: <546DBC9D.40407@redhat.com>

On Thu, Nov 20, 2014 at 11:04:13AM +0100, Paolo Bonzini wrote:
> 
> 
> On 20/11/2014 08:55, Michael S. Tsirkin wrote:
> > On Thu, Nov 20, 2014 at 08:11:05AM +0100, Paolo Bonzini wrote:
> >>
> >>
> >> On 20/11/2014 07:55, Michael S. Tsirkin wrote:
> >>> I thought we agreed we'll consider alternate approaches after 2.2?
> >>> I would prefer not to have yet another mode to support
> >>> if we can help it.
> >>
> >> I agree, but:
> >>
> >> 1) looks like there is stronger opposition to your patch than I thought,
> >> so a 2.2 solution as in this patch becomes more palatable
> > 
> > Why the urgency? It's not fixing any regressions, is it?
> > I would rather not add yet another mode for 2.2,
> > we'll likely have a new mode in 2.3 but I'd like that to
> > be the last one.
> 
> I don't think there's a need to add both patches.  If mine goes in, and
> it can go in 2.2 since it is "just another mode",

It's a mode we don't need - adding it does not fix any bugs.

> there is no need for
> resizable MemoryRegions.
> 
> Paolo

There will be need - otherwise each change will keep adding modes.

> >> 2) reviewing patches is always nice, and helps evaluating the advantages
> >> of either approach
> >>
> >> Paolo
> > 
> > I'll do my best, sorry about the delay - I'm trying to prioritize
> > 2.2 work at the moment.
> > 
-- 
MST

      reply	other threads:[~2014-11-20 11:53 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-06 14:56 [Qemu-devel] [PATCH v2 0/3] Migration-safe ACPI table sizing algorithm Paolo Bonzini
2014-10-06 14:56 ` [Qemu-devel] [PATCH v2 1/3] pc: introduce new " Paolo Bonzini
2014-10-06 14:56 ` [Qemu-devel] [PATCH v2 2/3] pc: go back to smaller ACPI tables Paolo Bonzini
2014-10-06 14:56 ` [Qemu-devel] [PATCH v2 3/3] pc: clean up pre-2.1 compatibility code Paolo Bonzini
2014-11-11 17:13 ` [Qemu-devel] [PATCH v2 0/3] Migration-safe ACPI table sizing algorithm Paolo Bonzini
2014-11-20  6:05   ` Paolo Bonzini
2014-11-20  6:55     ` Michael S. Tsirkin
2014-11-20  7:11       ` Paolo Bonzini
2014-11-20  7:55         ` Michael S. Tsirkin
2014-11-20 10:04           ` Paolo Bonzini
2014-11-20 11:52             ` 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=20141120115247.GA4973@redhat.com \
    --to=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --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.