netdev - Re: [net-next PATCH 09/10] pktgen: add sample script pktgen_sample03_burst_single

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 21 May 2015 11:47:35 +0200
From:	Jesper Dangaard Brouer <brouer@...hat.com>
To:	Cong Wang <cwang@...pensource.com>
Cc:	netdev <netdev@...r.kernel.org>,
	Daniel Borkmann <borkmann@...earbox.net>,
	Alexei Starovoitov <ast@...mgrid.com>,
	Robert Olsson <robert@...julf.se>,
	Ben Greear <greearb@...delatech.com>, brouer@...hat.com
Subject: Re: [net-next PATCH 09/10] pktgen: add sample script
 pktgen_sample03_burst_single_flow.sh

On Wed, 20 May 2015 14:33:42 -0700
Cong Wang <cwang@...pensource.com> wrote:

> On Tue, May 19, 2015 at 2:36 PM, Jesper Dangaard Brouer
> <brouer@...hat.com> wrote:
> > +# Threads are specified with parameter -t value in $THREADS
> > +for ((thread = 0; thread < $THREADS; thread++)); do
> > +    dev=${DEV}@...hread}
> > +
> > +    # Add remove all other devices and add_device $dev to thread
> > +    pg_thread $thread "rem_device_all"
> > +    pg_thread $thread "add_device" $dev
> > +
> > +    # Base config
> > +    pg_set $dev "flag QUEUE_MAP_CPU"
> > +    pg_set $dev "count $COUNT"
> > +    pg_set $dev "clone_skb $CLONE_SKB"
> > +    pg_set $dev "pkt_size $PKT_SIZE"
> > +    pg_set $dev "delay $DELAY"
> > +    pg_set $dev "flag NO_TIMESTAMP"
> > +
> > +    # Destination
> > +    pg_set $dev "dst_mac $DST_MAC"
> > +    pg_set $dev "dst $DEST_IP"
> > +
> > +    # Setup burst, for easy testing -b 0 disable bursting
> > +    # (internally in pktgen default and minimum burst=1)
> > +    if [[ ${BURST} != 0 ]]; then
> 
> 
> Nit: probably you mean: [[ ${BURST} -ne 0 ]]

With bash this still works (comparing it as a string), but I guess it
is more correct as you propose to compare it as a number.

You do realize that this cause $BURST to be evaluated as an arithmetic
expression, thus input like "10-10" will also disable bursting.

In bash it's also possible to write as: (( $BURST != 0 )) to be more
explicit that this is an arithmetic expression, and then we don't use
the -eq, -ne etc.

I'm fine with taking your proposal, as it also works with single "["
(although it does not fall for the arithmetic expansion).


> > +       pg_set $dev "burst $BURST"
> > +    else
> > +       info "$dev: Not using burst"
> > +    fi
> > +done



-- 
Best regards,
  Jesper Dangaard Brouer
  MSc.CS, Sr. Network Kernel Developer at Red Hat
  Author of http://www.iptv-analyzer.org
  LinkedIn: http://www.linkedin.com/in/brouer
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html