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-src
mailing list