On 08/10/2007 04:23 AM, Andi Kleen wrote:
>
> This is the patch i came up with in the end. Passes testing.
> I also fixed some more minor things.
Looks good to me. Missed adding KERN_EMERG for the printk()
of the TSS address, though.
>
> Fix double fault handler
>
> From: Chuck Ebbert <[EMAIL
On Thu, Aug 09, 2007 at 07:35:39PM -0400, Chuck Ebbert wrote:
> On 08/09/2007 07:16 PM, Andi Kleen wrote:
> >
> > I tested it. Even on a box without spin lock debugging I get a hard
> > hang after
> >
> > double fault, gdt at c1404000 [255 bytes]
> >
> > even though it should have printed the re
On 08/09/2007 07:16 PM, Andi Kleen wrote:
>
> I tested it. Even on a box without spin lock debugging I get a hard
> hang after
>
> double fault, gdt at c1404000 [255 bytes]
>
> even though it should have printed the registers.
> So it looks like there is more broken in the DF handler than just
>
On Thu, Aug 09, 2007 at 02:40:27PM -0400, Chuck Ebbert wrote:
> On 08/09/2007 01:49 PM, Andi Kleen wrote:
> > Chuck Ebbert <[EMAIL PROTECTED]> writes:
> >> Initializing FS in the doublefault_tss should fix it.
> >>
> >> Signed-off-by: Chuck Ebbert <[EMAIL PROTECTED]>
> >>
> >> ---
> >>
> >> NOTE:
Chuck Ebbert wrote:
> On 08/09/2007 01:49 PM, Andi Kleen wrote:
>
>> Chuck Ebbert <[EMAIL PROTECTED]> writes:
>>
>>> Initializing FS in the doublefault_tss should fix it.
>>>
>>> Signed-off-by: Chuck Ebbert <[EMAIL PROTECTED]>
>>>
>>> ---
>>>
>>> NOTE: not even compile tested.
>>>
>
On 08/09/2007 01:49 PM, Andi Kleen wrote:
> Chuck Ebbert <[EMAIL PROTECTED]> writes:
>> Initializing FS in the doublefault_tss should fix it.
>>
>> Signed-off-by: Chuck Ebbert <[EMAIL PROTECTED]>
>>
>> ---
>>
>> NOTE: not even compile tested.
>
> Can you please test it?
>
It compiles but I can'
Chuck Ebbert <[EMAIL PROTECTED]> writes:
>
> Initializing FS in the doublefault_tss should fix it.
>
> Signed-off-by: Chuck Ebbert <[EMAIL PROTECTED]>
>
> ---
>
> NOTE: not even compile tested.
Can you please test it?
-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-ke
The new percpu code has apparently broken the doublefault handler
when CONFIG_DEBUG_SPINLOCK is set. Doublefault is handled by
a hardware task, making the check
SPIN_BUG_ON(lock->owner == current, lock, "recursion");
fault because it uses the FS register to access the percpu data
for curr
8 matches
Mail list logo