Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Michael S. Tsirkin
On Mon, Nov 23, 2015 at 06:10:08PM +, Daniel P. Berrange wrote: > On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: > > On Mon, Nov 23, 2015 at 6:40 PM, Paolo Bonzini wrote: > > > Before: object-initial, chardev, qtest, object-late (not in the patch) > >

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Daniel P. Berrange
On Tue, Nov 24, 2015 at 04:10:54PM +0200, Michael S. Tsirkin wrote: > On Mon, Nov 23, 2015 at 06:10:08PM +, Daniel P. Berrange wrote: > > On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: > > > On Mon, Nov 23, 2015 at 6:40 PM, Paolo Bonzini > > > wrote:

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Paolo Bonzini
On 24/11/2015 15:25, Michael S. Tsirkin wrote: > I'm worried that things go slow and people don't have a way to find out > why, and documentation isn't the first place people look for when facing > a performance issue. > > At the moment we call MADV_HUGEPAGE unconditionally and unfortunately >

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Michael S. Tsirkin
On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: > On Mon, Nov 23, 2015 at 6:40 PM, Paolo Bonzini wrote: > > Before: object-initial, chardev, qtest, object-late (not in the patch) > > > > After: chardev, qtest, object-initial, object-late (not in the patch)

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 02:19:36PM +, Daniel P. Berrange wrote: > On Tue, Nov 24, 2015 at 04:10:54PM +0200, Michael S. Tsirkin wrote: > > On Mon, Nov 23, 2015 at 06:10:08PM +, Daniel P. Berrange wrote: > > > On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: > > > > On Mon,

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Daniel P. Berrange
On Tue, Nov 24, 2015 at 04:25:40PM +0200, Michael S. Tsirkin wrote: > On Tue, Nov 24, 2015 at 02:19:36PM +, Daniel P. Berrange wrote: > > On Tue, Nov 24, 2015 at 04:10:54PM +0200, Michael S. Tsirkin wrote: > > > On Mon, Nov 23, 2015 at 06:10:08PM +, Daniel P. Berrange wrote: > > > > On

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Markus Armbruster
"Daniel P. Berrange" writes: > On Tue, Nov 24, 2015 at 04:10:54PM +0200, Michael S. Tsirkin wrote: >> On Mon, Nov 23, 2015 at 06:10:08PM +, Daniel P. Berrange wrote: >> > On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: >> > > On Mon, Nov 23, 2015 at

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-24 Thread Michael S. Tsirkin
On Tue, Nov 24, 2015 at 03:30:23PM +0100, Paolo Bonzini wrote: > > > On 24/11/2015 15:25, Michael S. Tsirkin wrote: > > I'm worried that things go slow and people don't have a way to find out > > why, and documentation isn't the first place people look for when facing > > a performance issue. >

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-23 Thread Daniel P. Berrange
On Mon, Nov 23, 2015 at 07:01:33PM +0100, Marc-André Lureau wrote: > On Mon, Nov 23, 2015 at 6:40 PM, Paolo Bonzini wrote: > > Before: object-initial, chardev, qtest, object-late (not in the patch) > > > > After: chardev, qtest, object-initial, object-late (not in the patch)

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-23 Thread Paolo Bonzini
On 18/11/2015 21:49, Michael S. Tsirkin wrote: > On Wed, Nov 18, 2015 at 10:02:58AM +0100, marcandre.lur...@redhat.com wrote: >> From: Marc-André Lureau >> >> vhost-user-test prints a warning. A test should not need to run on >> hugetlbfs, let's silence the warning

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-23 Thread Daniel P. Berrange
On Mon, Nov 23, 2015 at 06:40:58PM +0100, Paolo Bonzini wrote: > > > On 18/11/2015 21:49, Michael S. Tsirkin wrote: > > On Wed, Nov 18, 2015 at 10:02:58AM +0100, marcandre.lur...@redhat.com wrote: > >> From: Marc-André Lureau > >> > >> vhost-user-test prints a

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-23 Thread Daniel P. Berrange
On Mon, Nov 23, 2015 at 05:46:06PM +, Daniel P. Berrange wrote: > On Mon, Nov 23, 2015 at 06:40:58PM +0100, Paolo Bonzini wrote: > > > > > > On 18/11/2015 21:49, Michael S. Tsirkin wrote: > > > On Wed, Nov 18, 2015 at 10:02:58AM +0100, marcandre.lur...@redhat.com > > > wrote: > > >> From:

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-23 Thread Marc-André Lureau
On Mon, Nov 23, 2015 at 6:40 PM, Paolo Bonzini wrote: > Before: object-initial, chardev, qtest, object-late (not in the patch) > > After: chardev, qtest, object-initial, object-late (not in the patch) > > Objects must be initialized before chardev (except rng-egd) since in

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-18 Thread Michael S. Tsirkin
On Wed, Nov 18, 2015 at 10:02:58AM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > vhost-user-test prints a warning. A test should not need to run on > hugetlbfs, let's silence the warning under qtest. The > condition can't check on

[Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-18 Thread marcandre . lureau
From: Marc-André Lureau vhost-user-test prints a warning. A test should not need to run on hugetlbfs, let's silence the warning under qtest. The condition can't check on qtest_enabled() since vhost-user-test actually doesn't use qtest accel. However, qtest_driver()

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-12 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 05:29:43PM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > vhost-user-test prints a warning. A test should not need to run on > hugetlbfs, let's silence the warning under qtest. Unfortunately, the > condition can't

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-08 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 05:29:43PM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > vhost-user-test prints a warning. A test should not need to run on > hugetlbfs, let's silence the warning under qtest. Unfortunately, the > condition can't

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-11-08 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > On Tue, Oct 27, 2015 at 05:29:43PM +0100, marcandre.lur...@redhat.com wrote: >> From: Marc-André Lureau >> >> vhost-user-test prints a warning. A test should not need to run on >> hugetlbfs, let's silence the warning

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-10-29 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 05:29:43PM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > vhost-user-test prints a warning. A test should not need to run on > hugetlbfs, let's silence the warning under qtest. Unfortunately, the > condition can't

[Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-10-27 Thread marcandre . lureau
From: Marc-André Lureau vhost-user-test prints a warning. A test should not need to run on hugetlbfs, let's silence the warning under qtest. Unfortunately, the condition can't check on qtest_enabled() or qtest_driver() since they are initialized later.

Re: [Qemu-devel] [PATCH] exec: silence hugetlbfs warning under qtest

2015-10-27 Thread Michael S. Tsirkin
On Tue, Oct 27, 2015 at 05:29:43PM +0100, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > vhost-user-test prints a warning. A test should not need to run on > hugetlbfs, let's silence the warning under qtest. Unfortunately, the > condition can't