Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-23 Thread Michael S. Tsirkin
On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: > This is another attempt to remove old q35 machine code. Now I am > also removing unused compat code to demonstrate the benefit of > throwing away the old code that nobody uses. > > Eduardo Habkost (5): > q35: Remove old machine

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-19 Thread Markus Armbruster
Eduardo Habkost writes: > On Thu, Feb 18, 2016 at 01:39:39PM +0100, Markus Armbruster wrote: >> Eduardo Habkost writes: >> >> > This is another attempt to remove old q35 machine code. Now I am >> > also removing unused compat code to demonstrate the

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-18 Thread Eduardo Habkost
On Thu, Feb 18, 2016 at 01:39:39PM +0100, Markus Armbruster wrote: > Eduardo Habkost writes: > > > This is another attempt to remove old q35 machine code. Now I am > > also removing unused compat code to demonstrate the benefit of > > throwing away the old code that nobody

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-18 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Thu, Feb 18, 2016 at 12:18:04PM +0100, Markus Armbruster wrote: >> No reply from you since Monday. > > Yes but I'm flooded with other stuff to review. I'll get back to it, > 3 days isn't such a long time. > >> You can >> >> (a) Apply Eduardo's

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-18 Thread Markus Armbruster
Eduardo Habkost writes: > This is another attempt to remove old q35 machine code. Now I am > also removing unused compat code to demonstrate the benefit of > throwing away the old code that nobody uses. > > Eduardo Habkost (5): > q35: Remove old machine versions >

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-18 Thread Michael S. Tsirkin
On Thu, Feb 18, 2016 at 12:18:04PM +0100, Markus Armbruster wrote: > No reply from you since Monday. Yes but I'm flooded with other stuff to review. I'll get back to it, 3 days isn't such a long time. > You can > > (a) Apply Eduardo's series. > > (b) Debate its merits. > > (c) Declare the

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-18 Thread Markus Armbruster
Markus Armbruster writes: > "Michael S. Tsirkin" writes: > >> On Thu, Feb 11, 2016 at 01:51:30PM -0200, Eduardo Habkost wrote: >>> On Sat, Feb 06, 2016 at 08:34:07PM +0200, Michael S. Tsirkin wrote: >>> > On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-12 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Thu, Feb 11, 2016 at 01:51:30PM -0200, Eduardo Habkost wrote: >> On Sat, Feb 06, 2016 at 08:34:07PM +0200, Michael S. Tsirkin wrote: >> > On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo Habkost wrote: >> > > On Fri, Feb 05, 2016 at 12:14:16AM

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-11 Thread Eduardo Habkost
On Sat, Feb 06, 2016 at 08:34:07PM +0200, Michael S. Tsirkin wrote: > On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo Habkost wrote: > > On Fri, Feb 05, 2016 at 12:14:16AM +0200, Michael S. Tsirkin wrote: > > > On Thu, Feb 04, 2016 at 05:09:44PM -0200, Eduardo Habkost wrote: > > > > On Thu, Feb

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-11 Thread Paolo Bonzini
On 11/02/2016 17:33, Michael S. Tsirkin wrote: >> They won't start unless the QEMU command-line is changed, because >> they are using a feature QEMU won't support anymore. Why is that >> a problem? > > We don't support installing one machine type, then switching. Uhm, we definitely support

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-11 Thread Paolo Bonzini
On 11/02/2016 17:33, Michael S. Tsirkin wrote: >> They won't start unless the QEMU command-line is changed, because >> they are using a feature QEMU won't support anymore. Why is that >> a problem? > > We don't support installing one machine type, then switching. Uhm, we definitely support

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-11 Thread Michael S. Tsirkin
On Thu, Feb 11, 2016 at 01:51:30PM -0200, Eduardo Habkost wrote: > On Sat, Feb 06, 2016 at 08:34:07PM +0200, Michael S. Tsirkin wrote: > > On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo Habkost wrote: > > > On Fri, Feb 05, 2016 at 12:14:16AM +0200, Michael S. Tsirkin wrote: > > > > On Thu, Feb

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-11 Thread Eduardo Habkost
On Thu, Feb 11, 2016 at 06:33:14PM +0200, Michael S. Tsirkin wrote: > On Thu, Feb 11, 2016 at 01:51:30PM -0200, Eduardo Habkost wrote: > > On Sat, Feb 06, 2016 at 08:34:07PM +0200, Michael S. Tsirkin wrote: > > > On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo Habkost wrote: > > > > On Fri, Feb

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-08 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Fri, Feb 05, 2016 at 09:55:08AM +0100, Markus Armbruster wrote: >> "Michael S. Tsirkin" writes: >> >> > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: >> >> On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-07 Thread Michael S. Tsirkin
On Fri, Feb 05, 2016 at 09:55:08AM +0100, Markus Armbruster wrote: > "Michael S. Tsirkin" writes: > > > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > >> On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > >> > On Sat, Jan 23, 2016 at

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-06 Thread Michael S. Tsirkin
On Fri, Feb 05, 2016 at 12:46:11PM -0200, Eduardo Habkost wrote: > On Fri, Feb 05, 2016 at 12:14:16AM +0200, Michael S. Tsirkin wrote: > > On Thu, Feb 04, 2016 at 05:09:44PM -0200, Eduardo Habkost wrote: > > > On Thu, Feb 04, 2016 at 08:02:30PM +0200, Michael S. Tsirkin wrote: > > > > On Thu, Feb

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-05 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: >> On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: >> > On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: >> > > This is another attempt to remove

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-05 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Thu, Feb 04, 2016 at 05:09:44PM -0200, Eduardo Habkost wrote: >> On Thu, Feb 04, 2016 at 08:02:30PM +0200, Michael S. Tsirkin wrote: >> > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: >> > > On Thu, Feb 04, 2016 at 06:01:50PM

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-05 Thread Eduardo Habkost
On Fri, Feb 05, 2016 at 12:14:16AM +0200, Michael S. Tsirkin wrote: > On Thu, Feb 04, 2016 at 05:09:44PM -0200, Eduardo Habkost wrote: > > On Thu, Feb 04, 2016 at 08:02:30PM +0200, Michael S. Tsirkin wrote: > > > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > > > > On Thu, Feb

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-04 Thread Michael S. Tsirkin
On Thu, Feb 04, 2016 at 05:09:44PM -0200, Eduardo Habkost wrote: > On Thu, Feb 04, 2016 at 08:02:30PM +0200, Michael S. Tsirkin wrote: > > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > > > On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > > > > On Sat, Jan

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-04 Thread Michael S. Tsirkin
On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: > This is another attempt to remove old q35 machine code. Now I am > also removing unused compat code to demonstrate the benefit of > throwing away the old code that nobody uses. The same thing I said applies - we don't know that

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-04 Thread Eduardo Habkost
On Thu, Feb 04, 2016 at 08:02:30PM +0200, Michael S. Tsirkin wrote: > On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > > On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > > > On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: > > > > This is

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-04 Thread Michael S. Tsirkin
On Thu, Feb 04, 2016 at 03:16:17PM -0200, Eduardo Habkost wrote: > On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > > On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: > > > This is another attempt to remove old q35 machine code. Now I am > > > also removing

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-02-04 Thread Eduardo Habkost
On Thu, Feb 04, 2016 at 06:01:50PM +0200, Michael S. Tsirkin wrote: > On Sat, Jan 23, 2016 at 02:02:08PM -0200, Eduardo Habkost wrote: > > This is another attempt to remove old q35 machine code. Now I am > > also removing unused compat code to demonstrate the benefit of > > throwing away the old

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-01-26 Thread Igor Mammedov
On Mon, 25 Jan 2016 12:27:47 +0100 Laszlo Ersek wrote: > On 01/23/16 17:02, Eduardo Habkost wrote: > > This is another attempt to remove old q35 machine code. Now I am > > also removing unused compat code to demonstrate the benefit of > > throwing away the old code that nobody

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-01-26 Thread Igor Mammedov
On Sat, 23 Jan 2016 14:02:08 -0200 Eduardo Habkost wrote: > This is another attempt to remove old q35 machine code. Now I am > also removing unused compat code to demonstrate the benefit of > throwing away the old code that nobody uses. Reviewed-by: Igor Mammedov

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-01-26 Thread Laszlo Ersek
On 01/26/16 11:15, Igor Mammedov wrote: > On Mon, 25 Jan 2016 12:27:47 +0100 > Laszlo Ersek wrote: > >> On 01/23/16 17:02, Eduardo Habkost wrote: >>> This is another attempt to remove old q35 machine code. Now I am >>> also removing unused compat code to demonstrate the

Re: [Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-01-25 Thread Laszlo Ersek
On 01/23/16 17:02, Eduardo Habkost wrote: > This is another attempt to remove old q35 machine code. Now I am > also removing unused compat code to demonstrate the benefit of > throwing away the old code that nobody uses. > > Eduardo Habkost (5): > q35: Remove old machine versions > machine:

[Qemu-devel] [PATCH v2 0/5] q35: Remove old machines and unused compat code

2016-01-23 Thread Eduardo Habkost
This is another attempt to remove old q35 machine code. Now I am also removing unused compat code to demonstrate the benefit of throwing away the old code that nobody uses. Eduardo Habkost (5): q35: Remove old machine versions machine: Remove no_tco field ich9: Remove enable_tco arguments