On Wed, Oct 21, 2015 at 03:06:57PM +0100, Peter Maydell wrote: > On 21 October 2015 at 11:26, Michael S. Tsirkin <m...@redhat.com> wrote: > > The following changes since commit 6d57410a79d51d92673c54f26624b44f27fa6214: > > > > Merge remote-tracking branch > > 'remotes/pmaydell/tags/pull-target-arm-20151016' into staging (2015-10-17 > > 12:31:33 +0100) > > > > are available in the git repository at: > > > > git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream > > > > for you to fetch changes up to 4f53cd7788185f437a4275df6456eb60b90d4837: > > > > hw/isa/lpc_ich9: inject the SMI on the VCPU that is writing to APM_CNT > > (2015-10-21 13:24:49 +0300) > > > > ---------------------------------------------------------------- > > vhost, pc, virtio features, fixes, cleanups > > > > New features: > > VT-d support for devices behind a bridge > > vhost-user migration support > > > > Signed-off-by: Michael S. Tsirkin <m...@redhat.com> > > > > ---------------------------------------------------------------- > > Test failure on 64-bit ARM and 32-bit ARM: > > TEST: tests/vhost-user-test... (pid=28701) > Warning: path not on HugeTLBFS: /tmp/vhost-test-pLQQzA > qemu-system-i386: -netdev > vhost-user,id=net0,chardev=chr-test,vhostforce: vhost-net support is > not compiled in > qemu-system-i386: -netdev > vhost-user,id=net0,chardev=chr-test,vhostforce: failed to init > vhost_net for queue 0 > > Broken pipe > FAIL: tests/vhost-user-test
Sorry about that. I rebased and dropped the offending patch. tag moved to 3c23402d4032f69af44a87fdb8019ad3229a4f31 Not resending full pull request since that is the only change. > My clang build gives this: > > Warning: path not on HugeTLBFS: /tmp/vhost-test-EFF6ul > /home/petmay01/linaro/qemu-for-merges/target-i386/translate.c:2421:26: > runtime error: left shift of negative value -8 > Warning: path not on HugeTLBFS: /tmp/vhost-test-EFF6ul > /home/petmay01/linaro/qemu-for-merges/target-i386/translate.c:2421:26: > runtime error: left shift of negative value -8 > Warning: path not on HugeTLBFS: /tmp/vhost-test-EFF6ul > /home/petmay01/linaro/qemu-for-merges/target-i386/translate.c:2421:26: > runtime error: left shift of negative value -8 > > The tests pass, but I'd prefer it if our tests > didn't print out random warnings in the non-verbose > case if possible; it makes it hard to sift out > "this is a problem" from "this test has always > been grumpy". (Alternatively, if "path not on HugeTLBFS" > is a problem then the test should fail.) > > The negative-shift stuff is clang's undefined > sanitizer; the code in question has been doing a > "-8 << s->dflag" since at least 2013, so this is just > your test code exercising something it hadn't before. > Sanitizer issues aren't currently something I'm > requiring pull requests to deal with, so we can fix > this one later. > > thanks > -- PMM