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) <[email protected]<mailto:[email protected]>> 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) <[email protected]<mailto:[email protected]>> a écrit : Excellent, thanks Ed! And let’s see … -Maciek On 26 Oct 2016, at 22:40, Edward Warnicke <[email protected]<mailto:[email protected]>> 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) <[email protected]<mailto:[email protected]>> 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) <[email protected]<mailto:[email protected]>> 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) <[email protected]<mailto:[email protected]>> Cc: Andrew Theurer <[email protected]<mailto:[email protected]>>; Douglas Shakshober <[email protected]<mailto:[email protected]>>; Damjan Marion (damarion) <[email protected]<mailto:[email protected]>>; Bill Michalowski <[email protected]<mailto:[email protected]>>; vpp-dev <[email protected]<mailto:[email protected]>>; Rashid Khan <[email protected]<mailto:[email protected]>>; [email protected]<mailto:[email protected]>; [email protected]<mailto:[email protected]>; Edward Warnicke <[email protected]<mailto:[email protected]>>; Thomas F Herbert <[email protected]<mailto:[email protected]>> 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 [email protected]<mailto:[email protected]> https://lists.fd.io/mailman/listinfo/csit-dev _______________________________________________ vpp-dev mailing list [email protected]<mailto:[email protected]> https://lists.fd.io/mailman/listinfo/vpp-dev
_______________________________________________ vpp-dev mailing list [email protected] https://lists.fd.io/mailman/listinfo/vpp-dev
