cvs commit: src/sys/amd64/amd64 pmap.c src/sys/i386/i386 pmap.c

Stephan Uphoff ups at freebsd.org
Wed Apr 25 19:07:39 UTC 2007


Yar Tikhiy wrote:
> On Sat, Apr 21, 2007 at 09:54:12AM -0600, Coleman Kane wrote:
>   
>> On Sat, 2007-04-21 at 17:03 +0200, Andre Oppermann wrote:
>>     
>>> Stephan Uphoff wrote:
>>>       
>>>> ups         2007-04-21 14:17:30 UTC
>>>>
>>>>   FreeBSD src repository
>>>>
>>>>   Modified files:
>>>>     sys/amd64/amd64      pmap.c 
>>>>     sys/i386/i386        pmap.c 
>>>>   Log:
>>>>   Modify TLB invalidation handling.
>>>>   
>>>>   Reviewed by:    alc@, peter@
>>>>   MFC after:      1 week
>>>>         
>>> Could you be a bit more verbose what changed here and why it
>>> was done?
>>>
>>>       
>> I agree. I would really like to know what the modification accomplishes.
>>     
>
> Alas, we don't live in an ideal world.  If we did, our commit
> messages would always follow the well-known guideline:
>
> 0. Tell the essence of the change.
> 1. Give the reason for the change.
> 2. Explain the change unless it's trivial.
>
>   
In the ideal world there are no NDAs :-)
As planned I forced a commit with a better comment once I was able to.
Thanks for your patients,

Stephan


More information about the cvs-all mailing list