All of lore.kernel.org
 help / color / mirror / Atom feed
From: jamal <hadi@cyberus.ca>
To: Robert Olsson <Robert.Olsson@data.slu.se>
Cc: Mandeep Singh Baines <mandeep.baines@gmail.com>,
	davem@davemloft.net, rick.jones2@hp.com, msb@google.com,
	netdev@vger.kernel.org, grundler@google.com,
	robert.olsson@its.uu.se, venza@brownhat.org
Subject: Re: pktgen terminating condition
Date: Thu, 30 Aug 2007 08:08:40 -0400	[thread overview]
Message-ID: <1188475721.22423.35.camel@localhost> (raw)
In-Reply-To: <18133.40847.166882.838333@robur.slu.se>

On Wed, 2007-29-08 at 18:32 +0200, Robert Olsson wrote:

>  Yes it's synchronization issue... the test is over and we have sent 
>  all pkts to the device but pktgen cannot free the skb for it still 
>  has refcounts. 

Ok, right.

I was confusing it with another issue where pktgen could send a lot of
packets without waiting for them to be freed; there are some drivers
(10G) which may hold onto 8K skbs. A gazillion ooms start spewing ;-> My
thinking in resolving that was to do something like waht sockets do and
charge pktgen so it doesnt have too many outstanding packets in flight.

>  IMO must drivers have provisions to handle situation like. 

Mandeep was saying he found less than a handful that didnt conform.

> I'll 
>  guess we can discuss last-resort timer if it should be us, ms 
>  or possibly seconds but shouldn't need ping to make this happen. 
>  If so we probably have a ICMP packet sitting there waiting instead.  

I think as long as it doesnt affect throughput calculation (it just adds
to idle time) its fine.

cheers,
jamal


  reply	other threads:[~2007-08-30 12:08 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1188347327.4305.27.camel@localhost>
     [not found] ` <20070829044352.GA4133@ludhiana>
2007-08-29 13:46   ` pktgen terminating condition jamal
2007-08-29 16:32     ` Robert Olsson
2007-08-30 12:08       ` jamal [this message]
2007-08-31  5:19         ` David Miller
2007-08-31 13:46           ` jamal
2007-08-29 16:59     ` Mandeep Baines
2007-08-30 11:33       ` jamal
2007-08-31 12:17       ` Daniele Venzano
2007-08-31 13:50         ` jamal
     [not found]   ` <20070828.220402.15268351.davem@davemloft.net>
     [not found]     ` <535ddc6b0708290914k21d0b71fs21ea3b96c6fd7f4d@mail.gmail.com>
2007-08-29 16:26       ` Grant Grundler
2007-09-01 10:47 Daniele Venzano

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=1188475721.22423.35.camel@localhost \
    --to=hadi@cyberus.ca \
    --cc=Robert.Olsson@data.slu.se \
    --cc=davem@davemloft.net \
    --cc=grundler@google.com \
    --cc=mandeep.baines@gmail.com \
    --cc=msb@google.com \
    --cc=netdev@vger.kernel.org \
    --cc=rick.jones2@hp.com \
    --cc=robert.olsson@its.uu.se \
    --cc=venza@brownhat.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.