This is the mail archive of the ecos-discuss@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: packet dumping


"Trenton D. Adams" wrote:
> 
> Not hard at all! :)
> 
> extern int net_debug;
> 
> net_debug = 1;
> 
> With that as 1, diag_dump_buf() is called on each packet.
> 
> Is that a good way of doing it?

You're right!
 
> Maybe we should add some #ifdefs along with a CDL MACRO instead of using
> "if (net_debug) {do net debug code}"?  After all, the if (net_debug) is
> always going to be compiled in right now, isn't it?

Yes.

> If you want, I'll make the changes and submit a patch.  I think the only
> modification required are in around 2 places in eth_drv.c!?
> 
> If you want me to, which package CDL should have the option?

io/eth. I just wrote a bunch of stuff describing what to do, but I may as
well just go ahead and do it, so I will.

Jifl
-- 
Red Hat, Rustat House, Clifton Road, Cambridge, UK. Tel: +44 (1223) 271062
Maybe this world is another planet's Hell -Aldous Huxley || Opinions==mine


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]