Hi Daniel,

On Tue, Oct 12, 2010 at 03:36:21PM +0200, Daniel Wagner wrote:
> Hi,
> 
> with these patches the number of open files for stats is reduced
> by half. Hopefully with these patches we don't get 'Too many open' 
> anymore.
Thanks, both patches applied.

Unrelated note:
While reviewing this code, I looked at the stats_create() routine. I think it
should return a struct stats_file pointer. That would avoid doing an hash
table lookup from __connman_stats_service_register() right after inserting the
stats_file pointer into it.

Cheers,
Samuel.


> cheers,
> daniel
> 
> v1: - use UINT_MAX to mark invalid home/roaming indexes
> v0: - initial version
> 
> _______________________________________________
> connman mailing list
> [email protected]
> http://lists.connman.net/listinfo/connman

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to