lkml.org 
[lkml]   [2019]   [Feb]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] sched/x86: Save [ER]FLAGS on context switch
From
Date


> On Feb 15, 2019, at 9:40 AM, Peter Zijlstra <peterz@infradead.org> wrote:
>
>> On Fri, Feb 15, 2019 at 09:18:00AM -0800, Linus Torvalds wrote:
>>> On Thu, Feb 14, 2019 at 11:34 AM Peter Zijlstra <peterz@infradead.org> wrote:
>>>
>>> Something like the below, right?
>>>
>>> + frame->flags = 0;
>>> + frame->flags = 0;
>>
>> Those are not valid flag values.
>>
>> Can you popf them? Yes.
>>
>> Do they make sense? No.
>>
>> It has the IF flag clear, for example. Is that intentional? If it is,
>
> Uhmm. yeah, that's bonkers. We should have interrupts disabled here.
> I'll go read up on the eflags and figure out what they _should_ be right
> about there.

And probably add a comment near the POPF explaining that it will keep IRQs off :)
\
 
 \ /
  Last update: 2019-02-15 19:29    [W:0.213 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site