linux-kernel - Re: 2.6.18-rc6-mm1: GPF loop on early boot

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20060911075504.GA5328@elte.hu>
Date:	Mon, 11 Sep 2006 09:55:05 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	Jeremy Fitzhardinge <jeremy@...p.org>
Cc:	Andrew Morton <akpm@...l.org>, Andi Kleen <ak@...e.de>,
	Laurent Riffard <laurent.riffard@...e.fr>,
	Arjan van de Ven <arjan@...radead.org>,
	Kernel development list <linux-kernel@...r.kernel.org>,
	Jeremy Fitzhardinge <jeremy@...source.com>
Subject: Re: 2.6.18-rc6-mm1: GPF loop on early boot


* Jeremy Fitzhardinge <jeremy@...p.org> wrote:

> Ingo Molnar wrote:
> >btw., what's the connection of %gs based PDA to Xen and 
> >paravirtualization in general - %esp based current is just as 
> >Xen-friendly, or am i wrong? I guess there must be some connection, 
> >given that you are working on this ;)
> >  
> 
> Yep.  The goal is to put the Xen VCPU structure into the PDA, so that it 
> can be easily accessed.  At present, masking events (ie, cli), is 
> something along the lines of
> 
>    xen_shared_info->vcpu[smp_processor_id()].mask = 1
> 
> which comes out to something like 20 bytes of code, and is probably too 
> awkward to inline.  If the vcpu is in the PDA, it would come out to:
> 
>    movb $1, %gs:xen_vcpu_mask
> 
> which has the added benefit of not needing a register.

take a look at lockdep: amongst other things it adds the TRACE_IRQFLAGS 
/ irqflags.h infrastructure, which could easily be modified to allow the 
easy replacement of cli/sti/pushf/popf. I wrote it with the side-goal of 
paravirtualization. I.e. instead of the DISABLE_INTERRUPT / 
ENABLE_INTERRUPT duplication you do in -mm currently please just enhance 
irqflags.h to cover the needs of paravirtualization too. (most of which 
should be the moving of cli/sti into the assembly callbacks)

into that the PDA would plug in a natural way: current->hardirqs_enabled 
is basically just an alias for 
!xen_shared_info->vcpu[smp_processor_id()].mask.

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