From: "Liu, Yong" <yong.liu@intel.com>
To: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>, Felix Marti <felix@chelsio.com>,
Nirranjan Kirubaharan <nirranjan@chelsio.com>,
Kumar Sanghvi <kumaras@chelsio.com>
Subject: Re: [dpdk-dev] |ERROR| pw 9018 ethdev: add a missing sanity check for nb_tx_desc during txq setup
Date: Mon, 23 Nov 2015 09:05:59 +0000 [thread overview]
Message-ID: <86228AFD5BCD8E4EBFD2B90117B5E81E10F6CBB9@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <20151123074132.GE2325@yliu-dev.sh.intel.com>
Yuanhan,
First three reports is for patch sets in the same email thread. I think patchwork scanned the mail and strip all of them out.
So the patch names did not change any more. For the failure one, it caused by validation environment. We're trying to find the root cause.
> -----Original Message-----
> From: Yuanhan Liu [mailto:yuanhan.liu@linux.intel.com]
> Sent: Monday, November 23, 2015 3:42 PM
> To: Liu, Yong
> Cc: dev@dpdk.org; Felix Marti; Kumar Sanghvi; Nirranjan Kirubaharan
> Subject: Re: [dpdk-dev] |ERROR| pw 9018 ethdev: add a missing sanity check
> for nb_tx_desc during txq setup
>
> On Mon, Nov 23, 2015 at 07:36:38AM +0000, Liu, Yong wrote:
> > Yuanhan,
> > The validation system is based on dpkdk.org patchwork, all patch sets
> recorded by patchwork should be validated.
> > Can you supply the patch Id or just patch name? Thus, we can check out
> what happened.
>
> Sure, here you go:
>
> 589 T Nov 18 sys_stv@intel.c ( 19) |SUCCESS| pw 8988 vhost: avoid
> buffer overflow in update_secure_len
> 733 r T Nov 17 sys_stv@intel.c ( 19) |SUCCESS| pw 8965 vhost: avoid
> buffer overflow in update_secure_len
> 863 T Nov 17 sys_stv@intel.c ( 19) |SUCCESS| pw 8960 vhost: avoid
> buffer overflow in update_secure_len
> 1230 r T Nov 11 sys_stv@intel.c ( 87) |ERROR| pw 8870 vhost: reset
> device properly
>
> --yliu
>
> >
> > > -----Original Message-----
> > > From: Yuanhan Liu [mailto:yuanhan.liu@linux.intel.com]
> > > Sent: Monday, November 23, 2015 3:27 PM
> > > To: Liu, Yong
> > > Cc: dev@dpdk.org; Felix Marti; Kumar Sanghvi; Nirranjan Kirubaharan
> > > Subject: Re: [dpdk-dev] |ERROR| pw 9018 ethdev: add a missing sanity
> check
> > > for nb_tx_desc during txq setup
> > >
> > > On Mon, Nov 23, 2015 at 07:06:45AM +0000, Liu, Yong wrote:
> > > > Hi Rahul,
> > > > Your patch passed "checksum_offload_with_vlan" after rerun this case.
> > > Please ignore this error report.
> > >
> > > I got a same report few weeks ago. And my patch was very vhost-user
> > > specific, touching nothing else. Even though, I sent a report for
> > > double check, and got no reply.
> > >
> > > I also got several "report" while replying to the same patch (not from
> > > me). I sent another report, yet no reply received.
> > >
> > >
> > > --yliu
> > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Rahul Lakkireddy [mailto:rahul.lakkireddy@chelsio.com]
> > > > > Sent: Friday, November 20, 2015 9:37 PM
> > > > > To: dev@dpdk.org
> > > > > Cc: Liu, Yong; Kumar Sanghvi; Nirranjan Kirubaharan; Felix Marti
> > > > > Subject: Re: |ERROR| pw 9018 ethdev: add a missing sanity check
> for
> > > > > nb_tx_desc during txq setup
> > > > >
> > > > > Hi all,
> > > > >
> > > > > I received this auto email regarding 'Unexpected Packets Drop' and
> > > > > it is referring to my patch. I don't think I've changed anything
> that
> > > > > could cause this. Kindly advise.
> > > > >
> > > > > Thanks,
> > > > > Rahul
> > > > >
> > > > > On Friday, November 11/20/15, 2015 at 03:14:51 -0800,
> > > sys_stv@intel.com
> > > > > wrote:
> > > > > > Test-Label: Intel Niantic on Fedora
> > > > > > Test-Status: ERROR
> > > > > > Patchwork: http://www.dpdk.org/dev/patchwork/patch/9018/
> > > > > >
> > > > > > DPDK git baseline: e64833f2273ac67becbca10d9f2f1598872dc99e
> > > > > > Patchwork ID: 9018
> > > > > > http://www.dpdk.org/dev/patchwork/patch/9018/
> > > > > > Submitter: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> > > > > > Date: Fri, 20 Nov 2015 15:56:37 +0530
> > > > > >
> > > > > > Source Compilation:
> > > > > > OS: fedora
> > > > > > Nic: niantic
> > > > > > i686-native-linuxapp-gcc: compile pass
> > > > > > x86_64-native-linuxapp-gcc: compile pass
> > > > > >
> > > > > > DTS validation:
> > > > > > OS: fedora
> > > > > > Nic: Niantic
> > > > > > TestType: auto
> > > > > > GCC: 4
> > > > > > x86_64-native-linuxapp-gcc: total 79, passed 78, failed 1.
> > > > > > Failed Case List:
> > > > > > Target: x86_64-native-linuxapp-gcc
> > > > > > OS: fedora
> > > > > > Failed DTS case:
> > > > > > checksum_offload_with_vlan:
> > > > >
> > >
> http://dpdk.org/browse/tools/dts/tree/test_plans/checksum_offload_test_pla
> > > > > n.rst
> > > > > >
> > > > > > DTS Validation Error:
> > > > > >
> > > > >
> > >
> ==========================================================================
> > > > > ==============================================
> > > > > >
> > > > >
> > >
> ==========================================================================
> > > > > ==============================================
> > > > > > TEST SUITE : TestChecksumOffload
> > > > > >
> > > > > > ----------------------------------------------------------------
> ----
> > > ----
> > > > > -------
> > > > > > Begin: Test Casetest_checksum_offload_with_vlan
> > > > > > --------------------------------------------------
> > > > > > FAILED 'Unexpected Packets Drop'
> > > > > > --------------------------------------------------
> > > > > > [ SUITE_DUT_CMD] ./x86_64-native-linuxapp-gcc/app/testpmd -c
> 0x6
> > > -n 4
> > > > > -- -i --portmask=0x3 --disable-hw-vlan --enable-rx-cksum --crc-
> strip -
> > > -
> > > > > txqflags=0
> > > > > > [ SUITE_DUT_CMD] set verbose 1
> > > > > > [ SUITE_DUT_CMD] set fwd csum
> > > > > > [ SUITE_DUT_CMD] csum set ip hw 0
> > > > > > [ SUITE_DUT_CMD] csum set udp hw 0
> > > > > > [ SUITE_DUT_CMD] csum set tcp hw 0
> > > > > > [ SUITE_DUT_CMD] csum set sctp hw 0
> > > > > > [ SUITE_DUT_CMD] csum set ip hw 1
> > > > > > [ SUITE_DUT_CMD] csum set udp hw 1
> > > > > > [ SUITE_DUT_CMD] csum set tcp hw 1
> > > > > > [ SUITE_DUT_CMD] csum set sctp hw 1
> > > > > > [ SUITE_DUT_CMD] start
> > > > > > [SUITE_TESTER_CMD] scapy
> > > > > > [SUITE_TESTER_CMD] sys.path.append("./")
> > > > > > [SUITE_TESTER_CMD] from sctp import *
> > > > > > [SUITE_TESTER_CMD] p =
> > > > > Ether(dst="90:e2:ba:4a:54:81")/IPv6(src="::2")/UDP()/("X"*46)
> > > > > > [SUITE_TESTER_CMD] p.show2()
> > > > > > [SUITE_TESTER_CMD] p =
> > > > > Ether(dst="90:e2:ba:4a:54:81")/IP(src="127.0.0.2")/SCTP()/("X"*48)
> > > > > > [SUITE_TESTER_CMD] p.show2()
> > > > > > [SUITE_TESTER_CMD] p =
> > > > > Ether(dst="90:e2:ba:4a:54:81")/IPv6(src="::2")/TCP()/("X"*46)
> > > > > > [SUITE_TESTER_CMD] p.show2()
> > > > > > [SUITE_TESTER_CMD] p =
> > > > > Ether(dst="90:e2:ba:4a:54:81")/IP(src="127.0.0.2")/UDP()/("X"*46)
> > > > > > [SUITE_TESTER_CMD] p.show2()
> > > > > > [SUITE_TESTER_CMD] p =
> > > > > Ether(dst="90:e2:ba:4a:54:81")/IP(src="127.0.0.2")/TCP()/("X"*46)
> > > > > > [SUITE_TESTER_CMD] p.show2()
> > > > > > [SUITE_TESTER_CMD] exit()
> > > > > > [SUITE_TESTER_CMD] echo -n '' > scapyResult.txt
> > > > > > [SUITE_TESTER_CMD] scp sniff.py root@10.239.128.80:~/
> > > > > > [SUITE_TESTER_CMD] SCAPY Receive setup:
> > > > > > [SUITE_TESTER_CMD] scapy
> > > > > > [SUITE_TESTER_CMD] subprocess.call("scapy -c sniff.py &",
> > > shell=True)
> > > > > > [SUITE_TESTER_CMD] sys.path.append("./")
> > > > > > [SUITE_TESTER_CMD] import sctp
> > > > > > [SUITE_TESTER_CMD] from sctp import *
> > > > > > [SUITE_TESTER_CMD]
> > > > >
> > >
> sendp([Ether(dst="90:e2:ba:4a:54:81")/IPv6(src="::1")/UDP(chksum=0xf)/("X"
> > > > > *46)], iface="p785p2")
> > > > > > [SUITE_TESTER_CMD]
> > > > >
> > >
> sendp([Ether(dst="90:e2:ba:4a:54:81")/IP(chksum=0x0)/SCTP(chksum=0xf)/("X"
> > > > > *48)], iface="p785p2")
> > > > > > [SUITE_TESTER_CMD]
> > > > >
> > >
> sendp([Ether(dst="90:e2:ba:4a:54:81")/IPv6(src="::1")/TCP(chksum=0xf)/("X"
> > > > > *46)], iface="p785p2")
> > > > > > [SUITE_TESTER_CMD]
> > > > >
> > >
> sendp([Ether(dst="90:e2:ba:4a:54:81")/IP(chksum=0x0)/UDP(chksum=0xf)/("X"*
> > > > > 46)], iface="p785p2")
> > > > > > [SUITE_TESTER_CMD]
> > > > >
> > >
> sendp([Ether(dst="90:e2:ba:4a:54:81")/IP(chksum=0x0)/TCP(chksum=0xf)/("X"*
> > > > > 46)], iface="p785p2")
> > > > > > [SUITE_TESTER_CMD] exit()
> > > > > > [SUITE_TESTER_CMD] cat scapyResult.txt
> > > > > > [SUITE_TESTER_CMD] SCAPY Result:
> > > > > > End test_checksum_offload_with_vlan
> > > > > > ----------------------------------------------------------------
> ----
> > > ----
> > > > > -------
> > > > > > [SUITE_DUT_CMD] quit
> > > > > >
> > > > > >
> > > > > >
> > > > > > DPDK STV team
prev parent reply other threads:[~2015-11-23 9:06 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <edcf24$hvv9db@FMSMGA003.fm.intel.com>
2015-11-20 13:37 ` Rahul Lakkireddy
2015-11-20 14:10 ` Mcnamara, John
2015-11-23 7:06 ` Liu, Yong
2015-11-23 7:27 ` Yuanhan Liu
2015-11-23 7:36 ` Liu, Yong
2015-11-23 7:41 ` Yuanhan Liu
2015-11-23 9:05 ` Liu, Yong [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86228AFD5BCD8E4EBFD2B90117B5E81E10F6CBB9@SHSMSX103.ccr.corp.intel.com \
--to=yong.liu@intel.com \
--cc=dev@dpdk.org \
--cc=felix@chelsio.com \
--cc=kumaras@chelsio.com \
--cc=nirranjan@chelsio.com \
--cc=yuanhan.liu@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).