[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduc
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition |
Date: |
Wed, 21 Nov 2018 08:57:31 -0500 |
On Wed, Nov 21, 2018 at 02:50:30PM +0100, Samuel Ortiz wrote:
> Hi Michael,
>
> On Wed, Nov 21, 2018 at 07:35:47AM -0500, Michael S. Tsirkin wrote:
> > On Mon, Nov 19, 2018 at 04:31:10PM +0100, Igor Mammedov wrote:
> > > On Fri, 16 Nov 2018 17:37:54 +0100
> > > Paolo Bonzini <address@hidden> wrote:
> > >
> > > > On 16/11/18 17:29, Igor Mammedov wrote:
> > > > > General suggestions for this series:
> > > > > 1. Preferably don't do multiple changes within a patch
> > > > > neither post huge patches (unless it's pure code movement).
> > > > > (it's easy to squash patches later it necessary)
> > > > > 2. Start small, pick a table generalize it and send as
> > > > > one small patchset. Tables are often independent
> > > > > and it's much easier on both author/reviewer to agree upon
> > > > > changes and rewrite it if necessary.
> > > >
> > > > How would that be done? This series is on the bigger side, agreed, but
> > > > most of it is really just code movement. It's a starting point, having
> > > > a generic ACPI library is way beyond what this is trying to do.
> > > I've tried to give suggestions how to restructure series
> > > on per patch basis. In my opinion it quite possible to split
> > > series in several smaller ones and it should really help with
> > > making series cleaner and easier/faster to review/amend/merge
> > > vs what we have in v5.
> > > (it's more frustrating to rework large series vs smaller one)
> > >
> > > If something isn't clear, it's easy to reach out to me here
> > > or directly (email/irc/github) for clarification/feed back.
> >
> > I assume the #1 goal is to add reduced HW support.
> >From our perspective, yes. From the project's point of view, it's about
> making the current ACPI code more generic and not bound to any specific
> machine type.
>
> > So another
> > option to speed up merging is to just go ahead and duplicate a
> > bunch of code e.g. in pc_virt.c acpi/reduced.c or in any other
> > file.
> It's precisely what we wanted to avoid in the very first place and we
> assumed this would be largely frowned upon by the community. It's also a
> burden for everyone to maintain that amount of duplicated code. Also I
> suppose this would also mean we'd have to eventually de-duplicate and
> factorize things in.
For sure, that's the plan.
> Honestly I'd rather not rush things out and work on code sharing first.
> I'll answer Igor's numerous comments today and will start addressing
> some of his concerns right aways as well.
>
> Cheers,
> Samuel.
OK, no problem then - just trying to make sure you aren't blocked.
--
MST
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, (continued)
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Michael S. Tsirkin, 2018/11/21
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Samuel Ortiz, 2018/11/21
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Igor Mammedov, 2018/11/21
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Samuel Ortiz, 2018/11/21
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Igor Mammedov, 2018/11/22
- Re: [Qemu-devel] [PATCH v5 00/24] ACPI reorganization for hardware-reduced API addition, Samuel Ortiz, 2018/11/21