netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ben Greear <greearb@candelatech.com>
To: Robert Olsson <Robert.Olsson@data.slu.se>
Cc: NetDev <netdev@vger.kernel.org>
Subject: Re: pktgen patch available for perusal.
Date: Fri, 03 Nov 2006 08:34:37 -0800	[thread overview]
Message-ID: <454B6F9D.8030300@candelatech.com> (raw)
In-Reply-To: <17739.9326.620495.394705@robur.slu.se>

Robert Olsson wrote:
> Ben Greear writes:
>
>  > I'd be thrilled to have the receive logic go into pktgen, even if it was #if 0 with a comment
>  > showing how to patch dev.c to get it working.  It would make my out-of-tree patch smaller
>  > and should help others who are doing research and driver development...
>
>
>  Just an idea... RX part is pretty separate from TX. How about if we keep the 
>  this code in a small seprate optional module? It can developed into some general 
>  RX probe.
>   
It requires a hook in dev.c, or at least that is the only way I can 
think to implement it.
I don't think that hook is going to be allowed into the kernel, so the 
best I was hoping for
was to have the code in pktgen with #if 0 and let users patch their 
kernel to add the
hook...

Ben

>  Cheers.
> 						--ro
> -
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>   


-- 
Ben Greear <greearb@candelatech.com> 
Candela Technologies Inc  http://www.candelatech.com



  reply	other threads:[~2006-11-03 16:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-30 23:47 pktgen patch available for perusal Ben Greear
2006-11-01 17:14 ` Robert Olsson
2006-11-01 19:11   ` Ben Greear
2006-11-03 11:13     ` Robert Olsson
2006-11-03 16:34       ` Ben Greear [this message]
2006-11-03 20:24         ` Robert Olsson
2006-11-03 20:41           ` Ben Greear
2006-11-04 12:32     ` jamal
2006-11-04 17:29       ` Ben Greear
2006-11-04 18:10         ` jamal
2006-11-04 19:12           ` Ben Greear
2006-11-06 14:06           ` Robert Olsson
2006-11-06 14:14     ` Robert Olsson

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=454B6F9D.8030300@candelatech.com \
    --to=greearb@candelatech.com \
    --cc=Robert.Olsson@data.slu.se \
    --cc=netdev@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).