Re: assorted changes for squid 2 HEAD cvs

On mån, 2007-10-08 at 13:00 +0400, lav@netis.ru wrote:
> Hi!
>
> Here are three changes for squid2 HEAD cvs.
>
> 1. initialize mem_obj (uninitialized can cause seg. fault).

Thanks!

Fixed this slightly differently however as entry might be NULL in some
corner cases..

> 2. use THREAD_FACTOR instead of hard coded constant 16.

Applied to both aiops.c and aiops_win32.c

> 3. call squidaio_sync in aufs dir maintenance to limit queue growth
> (e.g. when repository size or high/low are decreased).

Not sure about this one.. this will make Squid block under load and
additionally screws up fs load metrics. Better to move unlinking to a
separate thread I think, similar to unlinkd, or to make store
maintenance back off when the load is too high..

Can you please test if the attached patch solves the problem for you? It
makes the maintenance back off if load gets too high.

Regards
Henrik