[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 14/15] vhost-user-test: use tmpfs by default
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 14/15] vhost-user-test: use tmpfs by default |
Date: |
Tue, 6 Oct 2015 09:41:36 +0100 |
On 6 October 2015 at 00:17, Michael S. Tsirkin <address@hidden> wrote:
> On Mon, Oct 05, 2015 at 11:39:39PM +0100, Peter Maydell wrote:
>> On 2 October 2015 at 14:45, Michael S. Tsirkin <address@hidden> wrote:
>> > Most people don't run make check by default, so they skip vhost-user
>> > unit tests. Solve this by using tmpfs instead, unless hugetlbfs is
>> > specified (using an environment variable).
>> >
>> > Signed-off-by: Michael S. Tsirkin <address@hidden>
>> > Reviewed-by: Marc-André Lureau <address@hidden>
>>
>> Unfortunately I didn't notice before applying the pull, but this
>> is breaking 'make check' on AArch64 host for me:
>>
>> TEST: tests/vhost-user-test... (pid=20205)
>> Warning: path not on HugeTLBFS: /tmp/vhost-test-gRpbwl
>> qemu-system-i386: -netdev vhost-user,id=net0,chardev=chr0,vhostforce:
>> vhost-net support is not compiled in
>> qemu-system-i386: -netdev vhost-user,id=net0,chardev=chr0,vhostforce:
>> failed to init vhost_net for queue 0
>>
>> Broken pipe
>> FAIL: tests/vhost-user-test
>>
>> Probably reproducible on x86 if you configure with --disable-vhost-net,
>> though I haven't tried that.
>>
>> Perhaps tests/vhost-user-test should be set up
>> in tests/Makefile using
>> check-qtest-i386-$(CONFIG_VHOST_USER) rather
>> than CONFIG_LINUX ?
>>
>> I'd appreciate a quick fix, because this machine is in my set
>> of systems I test all pullreqs on now...
>>
>> thanks
>> -- PMM
>
> I think you are right, but just to be on the safe side, let's test
> both for now.
>
> If this helps you, pls feel free to apply.
>
> I will look at cleaning this up later.
Yes, this fixes the test run on aarch64; I'll apply it to
master.
-- PMM
- [Qemu-devel] [PULL 02/15] virtio: introduce virtqueue_discard(), (continued)
- [Qemu-devel] [PULL 02/15] virtio: introduce virtqueue_discard(), Michael S. Tsirkin, 2015/10/08
- [Qemu-devel] [PULL 09/15] virtio-9p: migrate virtio subsections, Michael S. Tsirkin, 2015/10/08
- [Qemu-devel] [PULL 04/15] oslib: rework anonimous RAM allocation, Michael S. Tsirkin, 2015/10/08
- [Qemu-devel] [PULL 15/15] vhost-user-test: fix predictable filename on tmpfs, Michael S. Tsirkin, 2015/10/08
- [Qemu-devel] [PULL 14/15] vhost-user-test: use tmpfs by default, Michael S. Tsirkin, 2015/10/08
Re: [Qemu-devel] [PULL 00/15] virtio,pc features, fixes, Michael S. Tsirkin, 2015/10/08