Hi Steve,
No worries.
We use gerrit for our git interactions. In the gerrit link for that patch:
https://gerrit.fd.io/r/#/c/7661/
<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgerrit.fd.io%2Fr%2F%23%2Fc%2F7661%2F&data=01%7C01%7Csteve.heckman%40arris.com%7C4008d53c73d34bd4e8fa08d4de8a7f74%7Cf27929ade5544d55837ac561519c3091%7C1&sdata=cfnWPgC2r9PM9tRFk61CuBHTqYyZS43RWbXsCysjCZY%3D&reserved=0>
you’ll see a button for cherry pick, select it and enter the branch
stable/1707. That’s it JWhen the CI jobs finish I can merge.
Thanks,
neale
*From: *Steve Heckman <steve.heck...@arris.com>
*Date: *Tuesday, 8 August 2017 at 19:16
*To: *"Neale Ranns (nranns)" <nra...@cisco.com>,
"vpp-dev@lists.fd.io" <vpp-dev@lists.fd.io>
*Subject: *Re: [vpp-dev] vpp v17.07 ping assert
Neale,
Sorry, I'm a bit new to this.
I'm working with git cli. What GUI are you referring to? Eclipse/EGit?
This is the change:
mage removed by sender.
Mohammed Hawari<mhaw...@cisco.com <mailto:mhaw...@cisco.com>>
07/18/2017 09:25:01 AM +0200
Committed by:John Lo<l...@cisco.com <mailto:l...@cisco.com>>
07/27/2017 10:10:26 PM +0000
ping: fixing wrong value when there are worker threads - the
echo_reply_node is now notifying the cli process on the main
thread/vlib_main - the timestamp for the icmp reply is now
acquired in the echo_reply_node and not in the cli process to
avoid an off by 10ms error (see 【vpp-dev】delay is error in ping
with multi worker thread) Change-Id:
I21d37002b0376b4f2ccab08d8f04c2f2944b9b39 Signed-off-by: Mohammed
Hawari
I've seen references to gerrit-cherry-pick. Should I use that?
Thanks, -Steve
On 08/08/2017 01:00 PM, Neale Ranns (nranns) wrote:
Hi Steve,
Could you please press the cherry pick button on those review[s] to get
them into stable/1707.
Thanks,
neale
-----Original Message-----
From:<vpp-dev-boun...@lists.fd.io> <mailto:vpp-dev-boun...@lists.fd.io> on behalf of
"Heckman, Steve"<steve.heck...@arris.com> <mailto:steve.heck...@arris.com>
Date: Tuesday, 8 August 2017 at 17:01
To:"vpp-dev@lists.fd.io" <mailto:vpp-dev@lists.fd.io> <vpp-dev@lists.fd.io>
<mailto:vpp-dev@lists.fd.io>
Subject: Re: [vpp-dev] vpp v17.07 ping assert
FWIW...I back-merged the changes to ping.[ch] from master to 17.07
locally and now it works.
-Steve
-----Original Message-----
From:vpp-dev-boun...@lists.fd.io
<mailto:vpp-dev-boun...@lists.fd.io> [mailto:vpp-dev-boun...@lists.fd.io] On
Behalf Of Steve Heckman
Sent: Tuesday, August 8, 2017 10:56 AM
To:vpp-dev@lists.fd.io <mailto:vpp-dev@lists.fd.io>
Subject: [vpp-dev] vpp v17.07 ping assert
I've been running into issues on certain servers where I get an
assert:
Aug 08 09:51:52 p5s3 systemd[1]: vpp.service: Main process exited,
code=killed, status=6/ABRT Aug 08 09:51:52 p5s3 systemd[1]: vpp.service: Unit
entered failed state.
Aug 08 09:51:52 p5s3 systemd[1]: vpp.service: Failed with result
'signal'.
here in ping.c
vlib_process_signal_event (vm, pr->cli_process_id, event_type,
bi0_copy);
which calls vec_elt, where the assert actually occurs.
/** \brief Get vector value at index i checking that i is in
bounds. */
#define vec_elt_at_index(v,i) \
({ \
ASSERT ((i) < vec_len (v)); \
(v) + (i); \
})
/** \brief Get vector value at index i */ #define vec_elt(v,i)
(vec_elt_at_index(v,i))[0]
pr->cli_process_id is set in run_ping_ip46_address() and always
seems to
be 344.
uword curr_proc = vlib_current_process (vm);
:
pr->cli_process_id = curr_proc;
vec_len(v) seems to be 32 (Does that make sense?).
Everything else seems to work OK. I can do "show interface (addr)",
"show ip arp".
Any ideas?
This is stock apt install on Ubuntu 16.04 LTS on HPG9 and SMC
servers with Intel 82599 NICs.
Thanks,
Steve Heckman
Principal Software Engineer
Arris
_______________________________________________
vpp-dev mailing list
vpp-dev@lists.fd.io <mailto:vpp-dev@lists.fd.io>
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.fd.io%2Fmailman%2Flistinfo%2Fvpp-dev&data=01%7C01%7CSteve.Heckman%40arris.com%7C07ca668477d64175ab0308d4de6db8ee%7Cf27929ade5544d55837ac561519c3091%7C1&sdata=vc%2Flg81kKLNwVjxd6n33tzkL5%2B%2BObfEhpnCg7y16zRA%3D&reserved=0
_______________________________________________
vpp-dev mailing list
vpp-dev@lists.fd.io <mailto:vpp-dev@lists.fd.io>
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.fd.io%2Fmailman%2Flistinfo%2Fvpp-dev&data=01%7C01%7CSteve.Heckman%40arris.com%7C003ce87b5bd647d251da08d4de7eecee%7Cf27929ade5544d55837ac561519c3091%7C1&sdata=5W8hRzZRZazYdt1RnrpYkxhQOH7Ho0ABq%2FEQsN%2BqofU%3D&reserved=0
