Nailed it. The bug was only happening in setups with single working thread, which was not the case in my test setup. Test succeeded. The patch is ready to be merged.
I am going to create a new vpp-sandbox directory called vhost-test. This sandbox effort intends to provide reference scripts for testing VPP vhost user, in the most user-friendly fashion (hopefully). Goal being to give pointers to people willing to test vhost user, but also to give reference parameters and configuration for people willing to use it. Thanks everyone, - Pierre Le 28 oct. 2016 à 10:55, Pierre Pfister (ppfister) <ppfis...@cisco.com<mailto:ppfis...@cisco.com>> a écrit : So I tested in my setup with qemu from git with tags: v2.5.0-rc0 v2.5.0-rc1 v2.5.0 v2.5.1 They all work. Could you please have a look at the patch and give me some more logs ? VPP with VHOST_USER_DEBUG_SOCKET set to 1 usually helps a lot, as it logs all vhost messages received by VPP. Thanks, - Pierre Le 26 oct. 2016 à 23:45, Maciek Konstantynowicz (mkonstan) <mkons...@cisco.com<mailto:mkons...@cisco.com>> a écrit : Excellent, thanks Ed! And let’s see … -Maciek On 26 Oct 2016, at 22:40, Edward Warnicke <hagb...@gmail.com<mailto:hagb...@gmail.com>> wrote: Maciek, Looks like the patch was based at a point in the past that was pointing to the old CSIT branch. I rebased the patch, so hopefully it will now pick up the correct CSIT branch. Ed On Wed, Oct 26, 2016 at 2:15 PM, Maciek Konstantynowicz (mkonstan) <mkons...@cisco.com<mailto:mkons...@cisco.com>> wrote: //reduced list, updated subject Peter is right: https://gerrit.fd.io/r/#/c/2922/16 looking at the last failed job - https://jenkins.fd.io/job/vpp-csit-verify-virl-master/1940/consoleFull - it's using old csit branch - "CSIT_BRANCH=oper-161017", without ubuntu1604, so pointless. you should rebase to pick up this commit https://gerrit.fd.io/r/#/c/3553/ that's updating to latest csit branch oper-161024. -Maciek On 26 Oct 2016, at 10:06, Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) <pmi...@cisco.com<mailto:pmi...@cisco.com>> wrote: Looks like it is still testing with oper-161017 not oper-161024. + CSIT_BRANCH=oper-161017 + git clone https://gerrit.fd.io/r/csit --branch oper-161017 Can you please use the latest one (code rebase)? Thanks. Peter Mikus Engineer – Software Cisco Systems Limited Planned absence: 28.10., 1.11., 17.11., 9.12., 19.-31.12. From: Pierre Pfister (ppfister) Sent: Wednesday, October 26, 2016 10:52 AM To: Peter Mikus -X (pmikus - PANTHEON TECHNOLOGIES at Cisco) <pmi...@cisco.com<mailto:pmi...@cisco.com>> Cc: Andrew Theurer <atheu...@redhat.com<mailto:atheu...@redhat.com>>; Douglas Shakshober <dsh...@redhat.com<mailto:dsh...@redhat.com>>; Damjan Marion (damarion) <damar...@cisco.com<mailto:damar...@cisco.com>>; Bill Michalowski <bmich...@redhat.com<mailto:bmich...@redhat.com>>; vpp-dev <vpp-dev@lists.fd.io<mailto:vpp-dev@lists.fd.io>>; Rashid Khan <rk...@redhat.com<mailto:rk...@redhat.com>>; kris...@redhat.com<mailto:kris...@redhat.com>; csit-...@lists.fd.io<mailto:csit-...@lists.fd.io>; Edward Warnicke <hagb...@gmail.com<mailto:hagb...@gmail.com>>; Thomas F Herbert <therb...@redhat.com<mailto:therb...@redhat.com>> Subject: Re: [vpp-dev] updated ovs vs. vpp results for 0.002% and 0% loss The recheck failed. https://jenkins.fd.io/job/vpp-csit-verify-virl-master/1937/console Are you sure you updated qemu ? Thanks, - Pierre _______________________________________________ csit-dev mailing list csit-...@lists.fd.io<mailto:csit-...@lists.fd.io> https://lists.fd.io/mailman/listinfo/csit-dev _______________________________________________ vpp-dev mailing list vpp-dev@lists.fd.io<mailto:vpp-dev@lists.fd.io> https://lists.fd.io/mailman/listinfo/vpp-dev
_______________________________________________ vpp-dev mailing list vpp-dev@lists.fd.io https://lists.fd.io/mailman/listinfo/vpp-dev
