[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86emul: special case far branch validation outside of long mode



>>> On 21.04.16 at 10:36, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 11/03/16 16:01, Jan Beulich wrote:
>> In that case (with the new value being held in, or now in one case cast
>> to, a 32-bit variable) there's no need to go through the long mode part
>> of the checks.
>>
>> Primarily this was meant to hopefully address Coverity ID 1355278, but
>> since the change produces smaller code as well I think we should use it
>> even if it doesn't help that (benign) warning.
>>
>> Also it's more in line with jmp_rel() for commit_far_branch() to do the
>> _regs.eip update, so adjust that at once.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Wei,

considering that this at least attempts to address a Coverity issue,
I'd like to ask for this to be considered to be allowed in despite
being past code freeze.

Thanks, Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.