Re: CVS commit: src/sys/arch/x86/x86

["Jared D. McNeill" <jmcneill%invisible.ca@localhost>]

No wonder, in the thread if uvmexp.free < uvmexp.freemin it skips the 
"mod_autotime == 0" check.

On 2011-10-18, at 8:06 AM, Jukka Ruohonen wrote:

> On Tue, Oct 18, 2011 at 07:59:35AM -0400, Jared D. McNeill wrote:
>> Well I loaded vmt manually and it was auto unloaded on me a few minutes
>> later, that's why I made this change in the first place.
>> 
>> On 2011-10-18, at 7:57 AM, Paul Goyette wrote:
>> 
>>> On Tue, 18 Oct 2011, Jukka Ruohonen wrote:
>>> 
>>>> On Tue, Oct 18, 2011 at 06:39:49AM -0400, Jared McNeill wrote:
>>>>> I would argue that any manually loaded module shouldn't be autounloaded.
>>>>> What do you think about flagging modules as autoloaded and only
>>>>> autounloading the autoloaded ones?
>>>> 
>>>> That sounds right to me.
>>> 
>>> I was under the impression that we would already only auto-unload
>>> modules that were auto-loaded.  The comment in module_thread() says
>>> 
>>> *   Automatically unload modules.  We try once to unload autoloaded
>>> *   modules after module_autotime seconds.  ...
> 
> True. But like Jared, I've also seen magic unloading of some of my driver
> modules. I guess there is a bug somewhere.
> 
> - Jukka.