Home » Mailing list archives » linux-kernel » 2008 » January » 18

Re: [PATCH 0/10] Tree fixes for PARAVIRT

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Zachary Amsden <zach@...>
To: Ingo Molnar <mingo@...>
Cc: Glauber de Oliveira Costa <gcosta@...>, <linux-kernel@...>, <akpm@...>, <glommer@...>, <tglx@...>, <ehabkost@...>, <jeremy@...>, <avi@...>, <anthony@...>, <virtualization@...>, <rusty@...>, <ak@...>, <chrisw@...>, <rostedt@...>, <hpa@...>, <roland@...>, <mtosatti@...>
Subject: Re: [PATCH 0/10] Tree fixes for PARAVIRT
Date: Friday, January 18, 2008 - 5:54 pm

On Fri, 2008-01-18 at 22:37 +0100, Ingo Molnar wrote:

Why are we rushing so much to do 64-bit paravirt that we are breaking
working configurations?  If the developement is going to be this
chaotic, it should be done and tested out of tree until it can
stabilize.

I do not like having to continuously retest and review the x86 branch
because the paravirt-ops are constantly in flux and the 32-bit code
keeps breaking.

We won't be doing 64-bit paravirt-ops for exactly this reason - is there
a serious justification from the performance angle on modern 64-bit
hardware?  If not, why justify the complexity and hackery to Linux?

Zach

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/10] Tree fixes for PARAVIRT, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH] fill in missing pv_mmu_ops entries for PAGETABLE_LEV..., Marcelo Tosatti, (Sat Jan 19, 2:19 pm)
Re: [PATCH] fill in missing pv_mmu_ops entries for PAGETABLE..., Ingo Molnar, (Tue Jan 22, 8:30 am)
Re: [PATCH] fill in missing pv_mmu_ops entries for PAGETABLE..., Glauber de Oliveira Costa..., (Mon Jan 28, 6:33 pm)
Re: [PATCH] fill in missing pv_mmu_ops entries for PAGETABLE..., Eduardo Pereira Habkost, (Mon Jan 21, 4:44 pm)
Re: [PATCH] fill in missing pv_mmu_ops entries for PAGETABLE..., Jeremy Fitzhardinge, (Mon Jan 21, 5:19 pm)
Re: [PATCH] fill in missing pv_mmu_ops entries for PAGETABLE..., Jeremy Fitzhardinge, (Sun Jan 20, 1:05 am)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Ingo Molnar, (Fri Jan 18, 4:32 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Ingo Molnar, (Fri Jan 18, 5:37 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Zachary Amsden, (Fri Jan 18, 5:54 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Jeremy Fitzhardinge, (Fri Jan 18, 6:31 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Ingo Molnar, (Fri Jan 18, 6:02 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Glauber de Oliveira Costa..., (Fri Jan 18, 9:24 pm)
Re: [PATCH 0/10] Tree fixes for PARAVIRT, Ingo Molnar, (Tue Jan 22, 8:20 am)
[PATCH 1/10] add missing parameter for lookup_address, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
Re: [PATCH 1/10] add missing parameter for lookup_address, Chris Wright, (Fri Jan 18, 4:26 pm)
Re: [PATCH 1/10] add missing parameter for lookup_address, Andi Kleen, (Fri Jan 18, 9:16 pm)
[PATCH 2/10] add stringify header, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 3/10] provide a native_init_IRQ function to x86_64, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 4/10] put generic mm_hooks include into PARAVIRT, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 5/10] puts read and write cr8 into pv_cpu_ops, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 6/10] provide read and write cr8 paravirt hooks, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 7/10] fill pv_cpu_ops structure with cr8 fields, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 8/10] add asm_offset PARAVIRT constants, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
[PATCH 9/10] provide __parainstructions section, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
Re: [PATCH 9/10] provide __parainstructions section, Sam Ravnborg, (Fri Jan 18, 4:41 pm)
Re: [PATCH 9/10] provide __parainstructions section, Jeremy Fitzhardinge, (Fri Jan 18, 6:47 pm)
[PATCH 10/10] change function orders in paravirt.h, Glauber de Oliveira Costa..., (Fri Jan 18, 1:20 pm)
Re: [PATCH 10/10] change function orders in paravirt.h, Jeremy Fitzhardinge, (Fri Jan 18, 4:24 pm)