lkml.org 
[lkml]   [2005]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [patch 07/15] Basic x86_64 support
On Thu, Aug 04, 2005 at 02:39:00PM +0200, Andi Kleen wrote:
> > > That doesn't make much sense here. tasklet will only run when interrupts
> > > are enabled, and that is much later. You could move it to there.
> >
> > Where? Keep in mind it's really only x86_64 that isn't able to break
> > sooner.
>
> The local_irq_enable() call in init/main.c:start_kernel()

But as I say, only x86_64 needs this kind of delay.

> If you want to run gdb earlier you need to do it without a tasklet.

We really would like to try again once stacks are setup (IOW, once
if ((&__get_cpu_var(init_tss))[0].ist[0])) is true).

> > > > --- linux-2.6.13-rc3/include/asm-x86_64/hw_irq.h~x86_64-lite 2005-07-29 13:19:10.000000000 -0700
> > > > +++ linux-2.6.13-rc3-trini/include/asm-x86_64/hw_irq.h 2005-07-29 13:19:10.000000000 -0700
> > > > @@ -55,6 +55,7 @@ struct hw_interrupt_type;
> > > > #define TASK_MIGRATION_VECTOR 0xfb
> > > > #define CALL_FUNCTION_VECTOR 0xfa
> > > > #define KDB_VECTOR 0xf9
> > > > +#define KGDB_VECTOR 0xf8
> > >
> > > I already allocated these vectors for something else.
> >
> > Is there another we can use? Just following what looked to be the
> > logical order.
>
> How about you use KDB_VECTOR and rename it to DEBUG_VECTOR
> and then just check if kgdb is currently active?

I can do the kgdb and generic changes at least.

[snip]
> > > > cfg = __prepare_ICR(shortcut, vector, dest);
> > > > + if (vector == KGDB_VECTOR) {
> > > > + /*
> > > > + * KGDB IPI is to be delivered as a NMI
> > > > + */
> > > > + cfg = (cfg&~APIC_VECTOR_MASK)|APIC_DM_NMI;
> > > > + }
> > >
> > > No way adding another ugly special case like this. I wanted
> > > to rip out the KDB version for a long time.
> >
> > I'd be happy to rework it in a cleaner manner, just point me at an
> > example please.
>
> The code is equivalent to passing shortcut|APIC_DM_NMI and vector == 0

OK. I'll see if I can change things around then, thanks.

--
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-08-04 16:10    [W:0.113 / U:0.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site