Re: [tip:x86/asm] sched/x86_64: Don't save flags on context switch

[Andy Lutomirski]

On Mon, Nov 3, 2014 at 2:57 PM, Oleg Nesterov <oleg@xxxxxxxxxx> wrote:
> Argh, sorry for confusion...
>
> On 11/03, Andy Lutomirski wrote:
>>
>> On Mon, Nov 3, 2014 at 1:58 PM, Oleg Nesterov <oleg@xxxxxxxxxx> wrote:
>> > On 11/03, Oleg Nesterov wrote:
>> >>
>> > Btw, why retint_kernel() checks "interrupts on" ? It seems to me that
>> > that "interrupts off" is not possible, no? And this will be more clear
>> > when we remove the "exit_intr" label.
>>
>> We might get there from #MC or from any of a number of synchronous
>> errors (#GP from xyz_safe, #PF from some atomic uaccess thing or a
>> vmap fault, etc), and all of those have interrupts off.
>
> Yes, yes, exactly.
>
> I actually tried to say that irqs should be always disabled (afaics!).
> IOW "interrupts on" should not be possible, not "interrupts off".

But this is checking whether interrupts were on in the frame we're
returning to, not whether they're on right now, right?

--Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/