Hello

On Thu, May 02, 2002 at 03:51:14PM -0500, Burton M. Strauss III wrote:
> It's not the bug you think it is!
Hmm but honestly I thing I must object :)

> First off, remember that the purge doesn't free everything, just up to
"the purge" is scanTimedoutTCPSessions()? Anyway that's the function I
focus on. I desperately need it beeing called at least every 1-2 minutes 
as I have asyncronous routing an _only_ incomplete tcp sessions that else
would never get exported via netflow.

The fact I was complaining about was that the variable SESSION_SCAN_DELAY is 
never used in multithreading mode despite the comment above suggerates that 
it is the right one. In stead the PURGE_HOSTS_DELAY is at least one of
the lower borders for the time period the scanTimedoutTCPSessions() is
called. Maybe the comments should be adjusted.
Except for the which-variable-does-what confusion the code itself works
fine!

> The code for the multithreaded version is -abstracted- this:
> 
> scanIdleLoop() {
>    sleep(60);
>    purgeIdleHosts() {
>        updateDeviceThpt(actDevice);
>        if(startTime < (lastPurgeTime[actDevice]+PURGE_HOSTS_DELAY)) return;
> (else) ...do the purge...
>        scanTimedoutTCPSessions();
>    }
> }


> Is anyone actually running single-threaded, who can test the attached patch?  
/me not, sorry.

> -----Burton
thanks,

-christian-


-- 
Christian Hammers    WESTEND GmbH - Aachen und Dueren     Tel 0241/701333-0
[EMAIL PROTECTED]     Internet & Security for Professionals    Fax 0241/911879
          WESTEND ist CISCO Systems Partner - Authorized Reseller

_______________________________________________
Ntop-dev mailing list
[EMAIL PROTECTED]
http://listgateway.unipi.it/mailman/listinfo/ntop-dev

Reply via email to