Re: [vpp-dev] vpp v17.07 ping assert
Neale,
Got it.
You're all set.
Thanks,
Steve
On 08/08/2017 02:22 PM, Neale Ranns (nranns) wrote:
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
*Date: *Tuesday, 8 August 2017 at 19:16
*To: *"Neale Ranns (nranns)" ,
"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 Hawarimailto:mhaw...@cisco.com>>
07/18/2017 09:25:01 AM +0200
Committed by:John Lomailto:l...@cisco.com>>
07/27/2017 10:10:26 PM +
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: <mailto:vpp-dev-boun...@lists.fd.io> on behalf of
"Heckman, Steve" <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>
<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
Re: [vpp-dev] vpp v17.07 ping assert
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/
you’ll see a button for cherry pick, select it and enter the branch
stable/1707. That’s it ☺ When the CI jobs finish I can merge.
Thanks,
neale
From: Steve Heckman
Date: Tuesday, 8 August 2017 at 19:16
To: "Neale Ranns (nranns)" , "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 mailto:mhaw...@cisco.com>>
07/18/2017 09:25:01 AM +0200
Committed by: John Lo mailto:l...@cisco.com>>
07/27/2017 10:10:26 PM +
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: <mailto:vpp-dev-boun...@lists.fd.io> on
behalf of "Heckman, Steve"
<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>
<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
___
vpp-dev mailing list
vpp-dev@lists.fd.io
https://lists.fd.io/mailman/listinfo/vpp-dev
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:
Mohammed Hawarimailto:mhaw...@cisco.com>>
07/18/2017 09:25:01 AM +0200
Committed by:John Lomailto:l...@cisco.com>>
07/27/2017 10:10:26 PM +
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: on behalf of "Heckman, Steve"
Date: Tuesday, 8 August 2017 at 17:01
To: "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] On
Behalf Of Steve Heckman
Sent: Tuesday, August 8, 2017 10:56 AM
To: 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
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
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
___
vpp-dev mailing list
vpp-dev@lists.fd.io
https://lists.fd.io/mailman/listinfo/vpp-dev
Re: [vpp-dev] vpp v17.07 ping assert
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: on behalf of "Heckman, Steve"
Date: Tuesday, 8 August 2017 at 17:01
To: "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] On
Behalf Of Steve Heckman
Sent: Tuesday, August 8, 2017 10:56 AM
To: 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
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
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
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] On
Behalf Of Steve Heckman
Sent: Tuesday, August 8, 2017 10:56 AM
To: 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
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
https://lists.fd.io/mailman/listinfo/vpp-dev
[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
https://lists.fd.io/mailman/listinfo/vpp-dev
