Messages in this thread | | | From | Andy Lutomirski <> | Date | Mon, 3 Nov 2014 15:02:43 -0800 | Subject | Re: [tip:x86/asm] sched/x86_64: Don't save flags on context switch |
| |
On Mon, Nov 3, 2014 at 2:57 PM, Oleg Nesterov <oleg@redhat.com> wrote: > Argh, sorry for confusion... > > On 11/03, Andy Lutomirski wrote: >> >> On Mon, Nov 3, 2014 at 1:58 PM, Oleg Nesterov <oleg@redhat.com> 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
| |