Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Igor Mammedov
On Wed, 20 Mar 2019 15:24:42 + Daniel P. Berrangé wrote: > On Wed, Mar 20, 2019 at 04:20:19PM +0100, Igor Mammedov wrote: > > > This could be solved if QEMU has some machine type based property > > > that indicates whether "memdev" is required for a given machine, > > > but crucially *does no

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Igor Mammedov
On Wed, 20 Mar 2019 13:46:59 + Daniel P. Berrangé wrote: > On Wed, Mar 20, 2019 at 10:32:53AM -0300, Eduardo Habkost wrote: > > On Wed, Mar 20, 2019 at 11:51:51AM +, Daniel P. Berrangé wrote: > > > On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: > > [...] [...] > > If

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Daniel P . Berrangé
On Wed, Mar 20, 2019 at 04:20:19PM +0100, Igor Mammedov wrote: > > This could be solved if QEMU has some machine type based property > > that indicates whether "memdev" is required for a given machine, > > but crucially *does not* actually activate that property until > > several releases later. >

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Daniel P . Berrangé
On Wed, Mar 20, 2019 at 11:46:20AM -0300, Eduardo Habkost wrote: > On Wed, Mar 20, 2019 at 01:46:59PM +, Daniel P. Berrangé wrote: > > On Wed, Mar 20, 2019 at 10:32:53AM -0300, Eduardo Habkost wrote: > > > On Wed, Mar 20, 2019 at 11:51:51AM +, Daniel P. Berrangé wrote: > > > > On Wed, Mar 2

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Igor Mammedov
On Wed, 20 Mar 2019 11:51:51 + Daniel P. Berrangé wrote: > On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: > > On Tue, 19 Mar 2019 14:51:07 + > > Daniel P. Berrangé wrote: > > > > > On Tue, Mar 19, 2019 at 02:08:01PM +0100, Igor Mammedov wrote: > > > > On Thu, 7 Mar 2

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Eduardo Habkost
On Wed, Mar 20, 2019 at 01:46:59PM +, Daniel P. Berrangé wrote: > On Wed, Mar 20, 2019 at 10:32:53AM -0300, Eduardo Habkost wrote: > > On Wed, Mar 20, 2019 at 11:51:51AM +, Daniel P. Berrangé wrote: > > > On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: > > [...] > > > > S

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Daniel P . Berrangé
On Wed, Mar 20, 2019 at 10:32:53AM -0300, Eduardo Habkost wrote: > On Wed, Mar 20, 2019 at 11:51:51AM +, Daniel P. Berrangé wrote: > > On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: > [...] > > > So it's rather questionable if we should care about arbitrarily old > > > l

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Eduardo Habkost
On Wed, Mar 20, 2019 at 11:51:51AM +, Daniel P. Berrangé wrote: > On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: [...] > > So it's rather questionable if we should care about arbitrarily old > > libvirt with new QEMU in case of new machines (especially upstream). > > As

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Daniel P . Berrangé
On Wed, Mar 20, 2019 at 11:26:34AM +0100, Igor Mammedov wrote: > On Tue, 19 Mar 2019 14:51:07 + > Daniel P. Berrangé wrote: > > > On Tue, Mar 19, 2019 at 02:08:01PM +0100, Igor Mammedov wrote: > > > On Thu, 7 Mar 2019 10:07:05 + > > > Daniel P. Berrangé wrote: > > > > > > > On Wed, Ma

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-20 Thread Igor Mammedov
On Tue, 19 Mar 2019 14:51:07 + Daniel P. Berrangé wrote: > On Tue, Mar 19, 2019 at 02:08:01PM +0100, Igor Mammedov wrote: > > On Thu, 7 Mar 2019 10:07:05 + > > Daniel P. Berrangé wrote: > > > > > On Wed, Mar 06, 2019 at 07:54:17PM +0100, Igor Mammedov wrote: > > > > On Wed, 6 Mar 20

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-19 Thread Daniel P . Berrangé
On Tue, Mar 19, 2019 at 02:08:01PM +0100, Igor Mammedov wrote: > On Thu, 7 Mar 2019 10:07:05 + > Daniel P. Berrangé wrote: > > > On Wed, Mar 06, 2019 at 07:54:17PM +0100, Igor Mammedov wrote: > > > On Wed, 6 Mar 2019 18:16:08 + > > > Daniel P. Berrangé wrote: > > > > > > > On Wed, Mar

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-19 Thread Igor Mammedov
On Thu, 7 Mar 2019 10:07:05 + Daniel P. Berrangé wrote: > On Wed, Mar 06, 2019 at 07:54:17PM +0100, Igor Mammedov wrote: > > On Wed, 6 Mar 2019 18:16:08 + > > Daniel P. Berrangé wrote: > > > > > On Wed, Mar 06, 2019 at 06:33:25PM +0100, Igor Mammedov wrote: > > > > Amend -numa optio

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-07 Thread Daniel P . Berrangé
On Wed, Mar 06, 2019 at 07:54:17PM +0100, Igor Mammedov wrote: > On Wed, 6 Mar 2019 18:16:08 + > Daniel P. Berrangé wrote: > > > On Wed, Mar 06, 2019 at 06:33:25PM +0100, Igor Mammedov wrote: > > > Amend -numa option docs and print warnings if 'mem' option or default RAM > > > splitting betwe

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-06 Thread Igor Mammedov
On Wed, 6 Mar 2019 18:16:08 + Daniel P. Berrangé wrote: > On Wed, Mar 06, 2019 at 06:33:25PM +0100, Igor Mammedov wrote: > > Amend -numa option docs and print warnings if 'mem' option or default RAM > > splitting between nodes is used. It's intended to discourage users from > > using > > con

Re: [Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-06 Thread Daniel P . Berrangé
On Wed, Mar 06, 2019 at 06:33:25PM +0100, Igor Mammedov wrote: > Amend -numa option docs and print warnings if 'mem' option or default RAM > splitting between nodes is used. It's intended to discourage users from using > configuration that allows only to fake NUMA on guest side while leading > to r

[Qemu-devel] [PATCH v2] numa: warn if numa 'mem' option or default RAM splitting between nodes is used.

2019-03-06 Thread Igor Mammedov
Amend -numa option docs and print warnings if 'mem' option or default RAM splitting between nodes is used. It's intended to discourage users from using configuration that allows only to fake NUMA on guest side while leading to reduced performance of the guest due to inability to properly configure