Re: [PATCH] Define EFLAGS_IF

2007-04-06 Thread Andi Kleen
On Thu, Apr 05, 2007 at 06:06:16PM -0700, H. Peter Anvin wrote: Andi Kleen wrote: No processor.h is such a hodgepodge of unrelated stuff that any splitting up is a good thing. Fair enough. However, I'd still like to see the X86_CR* constants moved, too (and constants added for at

Re: [PATCH] Define EFLAGS_IF

2007-04-06 Thread H. Peter Anvin
Andi Kleen wrote: On Thu, Apr 05, 2007 at 06:06:16PM -0700, H. Peter Anvin wrote: Andi Kleen wrote: No processor.h is such a hodgepodge of unrelated stuff that any splitting up is a good thing. Fair enough. However, I'd still like to see the X86_CR* constants moved, too (and constants

Re: [PATCH] Define EFLAGS_IF

2007-04-05 Thread H. Peter Anvin
Rusty Russell wrote: There is now more than one place where we use the fact that bit 9 of eflags is the interrupt-enabled flag, so define EFLAGS_IF. We make it 512 so it can be used in asm, too. How about defining all the other EFLAGS in one place? -hpa

Re: [PATCH] Define EFLAGS_IF

2007-04-05 Thread Jeremy Fitzhardinge
H. Peter Anvin wrote: Rusty Russell wrote: There is now more than one place where we use the fact that bit 9 of eflags is the interrupt-enabled flag, so define EFLAGS_IF. We make it 512 so it can be used in asm, too. How about defining all the other EFLAGS in one place? That

Re: [PATCH] Define EFLAGS_IF

2007-04-05 Thread H. Peter Anvin
Jeremy Fitzhardinge wrote: That patch got dropped, and replaced by one which pulled all the flags definitions out of asm/processor.h Saw that a little too late :) In general, it would be nice if the various CPU constants were all defined in one place, so I'd rather suggest protecting the

Re: [PATCH] Define EFLAGS_IF

2007-04-05 Thread Andi Kleen
On Thu, Apr 05, 2007 at 05:29:52PM -0700, H. Peter Anvin wrote: Jeremy Fitzhardinge wrote: That patch got dropped, and replaced by one which pulled all the flags definitions out of asm/processor.h Saw that a little too late :) In general, it would be nice if the various CPU constants

Re: [PATCH] Define EFLAGS_IF

2007-04-05 Thread H. Peter Anvin
Andi Kleen wrote: No processor.h is such a hodgepodge of unrelated stuff that any splitting up is a good thing. Fair enough. However, I'd still like to see the X86_CR* constants moved, too (and constants added for at least CR0 as well.) -hpa

[PATCH] Define EFLAGS_IF

2007-03-21 Thread Rusty Russell
There is now more than one place where we use the fact that bit 9 of eflags is the interrupt-enabled flag, so define EFLAGS_IF. We make it 512 so it can be used in asm, too. Signed-off-by: Rusty Russell [EMAIL PROTECTED] --- a/arch/i386/lguest/lguest.c +++ b/arch/i386/lguest/lguest.c @@ -107,9