Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 20 Dec 2013 11:26:59 -0800
From:      Neel Natu <neelnatu@gmail.com>
To:        Benjamin Kaduk <bjkfbsd@gmail.com>
Cc:        "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>
Subject:   Re: svn commit: r259641 - in head/sys/amd64: amd64 include vmm/intel
Message-ID:  <CAFgRE9HxkjHuP21hEwyL09iNz_w3QdmOVnQbHoXmMXvYeZtwTQ@mail.gmail.com>
In-Reply-To: <CAJ5_RoA4G_Vx%2BXJM8kSTeSn=D%2BEFQMY7YWwVYLYO=u6gFq07Ow@mail.gmail.com>
References:  <201312200550.rBK5oNmw098487@svn.freebsd.org> <CAJ5_RoA4G_Vx%2BXJM8kSTeSn=D%2BEFQMY7YWwVYLYO=u6gFq07Ow@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi Ben,

On Fri, Dec 20, 2013 at 10:27 AM, Benjamin Kaduk <bjkfbsd@gmail.com> wrote:
> On Thu, Dec 19, 2013 at 9:50 PM, Neel Natu <neel@freebsd.org> wrote:
>>
>> Author: neel
>> Date: Fri Dec 20 05:50:22 2013
>> New Revision: 259641
>> URL: http://svnweb.freebsd.org/changeset/base/259641
>>
>> Log:
>>   Re-arrange bits in the amd64/pmap 'pm_flags' field.
>>
>>   The least significant 8 bits of 'pm_flags' are now used for the IPI
>> vector
>>   to use for nested page table TLB shootdown.
>>
>>   Previously we used IPI_AST to interrupt the host cpu which is
>> functionally
>>   correct but could lead to misleading interrupt counts for AST handler.
>> The
>>   AST handler was also doing a lot more than what is required for the
>> nested
>>   page table TLB shootdown (EOI and IRET).
>
>
> This represents a KBI change, does it not?  Should __FreeBSD_version be
> bumped?
>

I was under the impression that a bump would be required only if this
change is MFCed to a stable branch. I could be wrong about this and if
so would be happy to rectify it.

best
Neel

> -Ben



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFgRE9HxkjHuP21hEwyL09iNz_w3QdmOVnQbHoXmMXvYeZtwTQ>