Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Eduardo Habkost
On Tue, May 26, 2015 at 01:54:06PM +0100, Ikey Doherty wrote: > The target-x86_64.conf sysconfig file has been empty and essentially ignored > now for several years. This change removes the unused file to enable moving > towards a stateless configuration. > > Signed-off-by: Ikey Doherty Reviewed

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Eduardo Habkost
On Tue, May 26, 2015 at 06:51:18PM +0200, Paolo Bonzini wrote: > > > On 26/05/2015 18:40, Eduardo Habkost wrote: > > > What is the usecase? Was /etc/qemu/target-*.conf actually meant to be > > > user-customizable when it hosted the CPU models? > > > > I don't know what's the use case, that's why

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Paolo Bonzini
On 26/05/2015 18:40, Eduardo Habkost wrote: > > What is the usecase? Was /etc/qemu/target-*.conf actually meant to be > > user-customizable when it hosted the CPU models? > > I don't know what's the use case, that's why I think we should really > remove it. I only want to warn users before remov

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Eduardo Habkost
On Tue, May 26, 2015 at 06:29:25PM +0200, Paolo Bonzini wrote: > > > On 26/05/2015 18:25, Eduardo Habkost wrote: > > Can you separate this into two patches? First deleting the empty > > target-x86_64.conf file from the tree & Makefile, then another patch > > deleting the > > { CONFIG_QEMU_CONFD

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Ikey Doherty
On 26/05/15 17:25, Eduardo Habkost wrote: On Tue, May 26, 2015 at 04:00:45PM +0200, Paolo Bonzini wrote: CCing maintainer. Paolo On 26/05/2015 14:54, Ikey Doherty wrote: The target-x86_64.conf sysconfig file has been empty and essentially ignored now for several years. This change removes t

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Paolo Bonzini
On 26/05/2015 18:25, Eduardo Habkost wrote: > Can you separate this into two patches? First deleting the empty > target-x86_64.conf file from the tree & Makefile, then another patch > deleting the > { CONFIG_QEMU_CONFDIR "/target-" TARGET_NAME ".conf", true } > line in arch_init.c? > > We can

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Eduardo Habkost
On Tue, May 26, 2015 at 04:00:45PM +0200, Paolo Bonzini wrote: > CCing maintainer. > > Paolo > > On 26/05/2015 14:54, Ikey Doherty wrote: > > The target-x86_64.conf sysconfig file has been empty and essentially ignored > > now for several years. This change removes the unused file to enable movin

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Eric Blake
On 05/26/2015 06:54 AM, Ikey Doherty wrote: [meta-comment] > The target-x86_64.conf sysconfig file has been empty and essentially ignored > now for several years. This change removes the unused file to enable moving > towards a stateless configuration. > > Signed-off-by: Ikey Doherty > --- > M

Re: [Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Paolo Bonzini
CCing maintainer. Paolo On 26/05/2015 14:54, Ikey Doherty wrote: > The target-x86_64.conf sysconfig file has been empty and essentially ignored > now for several years. This change removes the unused file to enable moving > towards a stateless configuration. > > Signed-off-by: Ikey Doherty > --

[Qemu-devel] [PATCH v3 1/2] arch_init: Drop target-x86_64.conf

2015-05-26 Thread Ikey Doherty
The target-x86_64.conf sysconfig file has been empty and essentially ignored now for several years. This change removes the unused file to enable moving towards a stateless configuration. Signed-off-by: Ikey Doherty --- Makefile | 7 +-- arch_init.c