Re: [dpdk-dev] [PATCH v10] devtools: alert on new instances of rte_panic and rte_exit

2018-07-15 Thread Thomas Monjalon
27/05/2018 22:34, Arnon Warshavsky: > This is consuming stdin. > > I guess the checkpatch.pl will have nothing to check in the next step. > > We should merge Neil's patch first, because he is adding a tmpfile > > to solve the issue of stdin read only once. > > > > Yup. Missed the fact stdin is con

Re: [dpdk-dev] [PATCH v10] devtools: alert on new instances of rte_panic and rte_exit

2018-05-27 Thread Arnon Warshavsky
This is consuming stdin. > I guess the checkpatch.pl will have nothing to check in the next step. > We should merge Neil's patch first, because he is adding a tmpfile > to solve the issue of stdin read only once. > Yup. Missed the fact stdin is consumed. temp file is a good idea > > > +

Re: [dpdk-dev] [PATCH v10] devtools: alert on new instances of rte_panic and rte_exit

2018-05-27 Thread Thomas Monjalon
Hi, 30/04/2018 08:45, Arnon Warshavsky: > --- a/devtools/checkpatches.sh > +++ b/devtools/checkpatches.sh > + check_forbidden_additions - This is consuming stdin. I guess the checkpatch.pl will have nothing to check in the next step. We should merge Neil's patch first, because he is a

Re: [dpdk-dev] [PATCH v10] devtools: alert on new instances of rte_panic and rte_exit

2018-05-04 Thread Kevin Traynor
On 04/30/2018 07:45 AM, Arnon Warshavsky wrote: > This patch adds a new function that is called > per every checked patch, > and alerts for new instances of rte_panic/rte_exit. > The check excludes comments, and alerts in the case > of a positive balance between additions and removals. > > Signed-

[dpdk-dev] [PATCH v10] devtools: alert on new instances of rte_panic and rte_exit

2018-04-29 Thread Arnon Warshavsky
This patch adds a new function that is called per every checked patch, and alerts for new instances of rte_panic/rte_exit. The check excludes comments, and alerts in the case of a positive balance between additions and removals. Signed-off-by: Arnon Warshavsky Reviewed-by: Stephen Hemminger ---