without
-traditional.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
---
arch/x86/kernel/Makefile_64 |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/arch/x86/kernel/Makefile_64 b/arch/x86/kernel/Makefile_64
index 82da906..03a01f6 100644
--- a/arch/x86/kernel
in the world... just that token pasting in
asm files will diverge from the way in which we (-)traditionally do
;-) )
So, does any one have a word on that?
--
Glauber de Oliveira Costa.
"Free as in Freedom"
http://glommer.net
"The less confident you are, the more serious
() and pmd_populate_kernel() are usually macros that expand
to simpler variants that does not use the first parameter at all.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
---
mm/sparse-vmemmap.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/mm/sparse-vmemmap.
in the world... just that token pasting in
asm files will diverge from the way in which we (-)traditionally do
;-) )
So, does any one have a word on that?
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you are, the more serious you have to act.
-
To unsubscribe
() and pmd_populate_kernel() are usually macros that expand
to simpler variants that does not use the first parameter at all.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
---
mm/sparse-vmemmap.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/mm/sparse-vmemmap.c b/mm/sparse
much simpler.
>
> Signed-off-by: Jeremy Fitzhardinge <[EMAIL PROTECTED]>
> Cc: Andi Kleen <[EMAIL PROTECTED]>
> Cc: Zach Amsden <[EMAIL PROTECTED]>
> Cc: Rusty Russell <[EMAIL PROTECTED]>
> Cc: Avi Kivity <[EMAIL PROTECTED]>
> Cc: Anthony Liguory <[
PROTECTED]
Cc: Rusty Russell [EMAIL PROTECTED]
Cc: Avi Kivity [EMAIL PROTECTED]
Cc: Anthony Liguory [EMAIL PROTECTED]
Cc: Glauber de Oliveira Costa [EMAIL PROTECTED]
Cc: Nakajima, Jun [EMAIL PROTECTED]
---
arch/i386/kernel/paravirt.c | 58
+++
arch/i386
tools would be using the output of uname -m, so a
possible way to fix this would be fixing the data passed to userspace
from uname. However, that might be the case that it creates a new set
of problems too, with tools relying on the output of uname -m to
determine wheter the machine is 32 or 64 bit
a
possible way to fix this would be fixing the data passed to userspace
from uname. However, that might be the case that it creates a new set
of problems too, with tools relying on the output of uname -m to
determine wheter the machine is 32 or 64 bit, and so on.
--
Glauber de Oliveira Costa.
Free
e sort
of acessor is preferred anyway.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
---
drivers/lguest/io.c | 10 +-
drivers/lguest/lg.h |3 +++
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers/lguest/io.c b/drivers/lguest/io.c
index ea68
of acessor is preferred anyway.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
---
drivers/lguest/io.c | 10 +-
drivers/lguest/lg.h |3 +++
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers/lguest/io.c b/drivers/lguest/io.c
index ea68613..70bab38 100644
he puppies ate a bit too
> > much. No more lean and mean puppies, now we got big fat lazy ones.
> > Running on the hardware that's too lazy to do full virutalization. Yes,
> > lguest now runs on x86_64!
>
> Totally phat!
Whatever this means!
--
Glauber de Oliveira Cost
puppies, now we got big fat lazy ones.
Running on the hardware that's too lazy to do full virutalization. Yes,
lguest now runs on x86_64!
Totally phat!
Whatever this means!
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you are, the more serious you
can also try to start following LKML's flow. Maybe you won't
understand much in the beginning, but your comprehension on the
discussions will improve in the future. (Maybe reading a subsystem
mailing list - less traffic - is a good idea, if you have some
specific interests)
--
Glauber de Ol
oes. anyway, by this phrase, I only meant: "Dropping support
for 3.4 series won't help in this case"
--
Glauber de Oliveira Costa.
"Free as in Freedom"
http://glommer.net
"The less confident you are, the more serious you have to act."
-
To unsubscribe from this list: s
nt newer kernels.
Last time I tried a mips build, it would fail the compile unless I was
using _exactly_ 3.4.4 (I didn't tried older versions, but did try
3.4.6, for ex.). So I also think the 3.4 series will still have to be
around for a while.
--
Glauber de Oliveira Costa.
"Free as in Fre
These two callsites should really be errx instead of err, since there is
no errno associated with them in the moment they are issued.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
---
Documentation/lguest/lguest.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
On 8/21/07, Christian Borntraeger <[EMAIL PROTECTED]> wrote:
> Am Montag, 20. August 2007 schrieb Glauber de Oliveira Costa:
> > Although I don't know KVM to a that deep level, I think it should be
> > possible to keep the virtual cpus in different process (or threads),
> &
On 8/21/07, Laurent Vivier <[EMAIL PROTECTED]> wrote:
> Glauber de Oliveira Costa wrote:
> >> by doing this at kernel level, we can:
> >> - measure exactly the guest time,
> >> - move this part of system time to user time (as you think it should be
> >>
On 8/21/07, Andi Kleen <[EMAIL PROTECTED]> wrote:
> On Tue, Aug 21, 2007 at 04:30:10AM -0300, Glauber de Oliveira Costa wrote:
> > On 8/20/07, Jeremy Fitzhardinge <[EMAIL PROTECTED]> wrote:
> > > Chris Wright wrote:
> > > > That did get backed out (at l
but my night have to get
some value, after all! ;-)
So I'm with Jeremy. We don't lose too much by putting a memcpy there,
this code is not exactly critical. It also seems cleaner, and less
error prone. I have a patch ready here, but I think by this time, you
guys have too ;-)
--
Glauber de Olivei
you won't
understand much in the beginning, but your comprehension on the
discussions will improve in the future. (Maybe reading a subsystem
mailing list - less traffic - is a good idea, if you have some
specific interests)
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
too much by putting a memcpy there,
this code is not exactly critical. It also seems cleaner, and less
error prone. I have a patch ready here, but I think by this time, you
guys have too ;-)
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you are, the more
On 8/21/07, Andi Kleen [EMAIL PROTECTED] wrote:
On Tue, Aug 21, 2007 at 04:30:10AM -0300, Glauber de Oliveira Costa wrote:
On 8/20/07, Jeremy Fitzhardinge [EMAIL PROTECTED] wrote:
Chris Wright wrote:
That did get backed out (at least the part that broke paravirt patching
On 8/21/07, Laurent Vivier [EMAIL PROTECTED] wrote:
Glauber de Oliveira Costa wrote:
by doing this at kernel level, we can:
- measure exactly the guest time,
- move this part of system time to user time (as you think it should be
user time),
- have consistency between system, user
On 8/21/07, Christian Borntraeger [EMAIL PROTECTED] wrote:
Am Montag, 20. August 2007 schrieb Glauber de Oliveira Costa:
Although I don't know KVM to a that deep level, I think it should be
possible to keep the virtual cpus in different process (or threads),
and take the accounting time
These two callsites should really be errx instead of err, since there is
no errno associated with them in the moment they are issued.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
---
Documentation/lguest/lguest.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff
a mips build, it would fail the compile unless I was
using _exactly_ 3.4.4 (I didn't tried older versions, but did try
3.4.6, for ex.). So I also think the 3.4 series will still have to be
around for a while.
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you
in this case
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you are, the more serious you have to act.
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http
KVM to a that deep level, I think it should be
possible to keep the virtual cpus in different process (or threads),
and take the accounting time from there. Perfectly possible to know
the time we spent running (user time), and the time the hypervisor
spent doing things on our behalf (system time).
$0.0
be
possible to keep the virtual cpus in different process (or threads),
and take the accounting time from there. Perfectly possible to know
the time we spent running (user time), and the time the hypervisor
spent doing things on our behalf (system time).
$0.02.
--
Glauber de Oliveira Costa.
Free
. In this file, usage was properly
converted to __stringify()
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
---
arch/x86_64/ia32/ia32_binfmt.c |5 +
include/asm-x86_64/hw_irq.h|3 ---
include/asm-x86_64/system.h|3 ---
3 files changed, 1 insertions(+), 10 del
. In this file, usage was properly
converted to __stringify()
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
---
arch/x86_64/ia32/ia32_binfmt.c |5 +
include/asm-x86_64/hw_irq.h|3 ---
include/asm-x86_64/system.h|3 ---
3 files changed, 1 insertions(+), 10 deletions
Jeremy Fitzhardinge escreveu:
Glauber de Oliveira Costa wrote:
Thanks for the explanation, Andi. I understand it much better now, and
agree with you.
As alternatives what we have now, we can either keep the paravirt_ops
as it is now for the native case, just hooking the vsmp functions in
place
On 8/15/07, Chris Wright <[EMAIL PROTECTED]> wrote:
> * Glauber de Oliveira Costa ([EMAIL PROTECTED]) wrote:
> > Only caveat, is that it has to be done before smp gets in the game, and
> > with interrupts disabled. (which makes the function in vsmp.c not eligible).
>
Chris Wright escreveu:
* Glauber de Oliveira Costa ([EMAIL PROTECTED]) wrote:
As alternatives what we have now, we can either keep the paravirt_ops as
it is now for the native case, just hooking the vsmp functions in place
of the normal one, (there are just three ops anyway), refill
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 12:09:42PM -0300, Glauber de Oliveira Costa wrote:
Again, this is the code of such function:
static inline int raw_irqs_disabled_flags(unsigned long flags)
{
return !(flags & X86_EFLAGS_IF);
}
so all it is doing is getting a parameter (f
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 11:25:43AM -0300, Glauber de Oliveira Costa wrote:
On 8/15/07, Andi Kleen <[EMAIL PROTECTED]> wrote:
+#ifdef CONFIG_PARAVIRT
+extern unsigned long *cpu_gdt_descr;
No externs in .c files
Normally they should be where the variable is defined
a
Avi Kivity escreveu:
Glauber de Oliveira Costa wrote:
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 11:18:25AM -0300, Glauber de Oliveira Costa
wrote:
Didn't we agree this should be a pvops client?
-Andi
No. I exposed my reasoning, asked you back, but got no answer.
I'll do it again
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 11:18:25AM -0300, Glauber de Oliveira Costa wrote:
Didn't we agree this should be a pvops client?
-Andi
No. I exposed my reasoning, asked you back, but got no answer.
I'll do it again:
This operations are just manipulating bits, and are doing
ad.S, what do you propose?
AFAICT, this is the very duty of this file (x8664_syms.c), and the
extern is also used in other symbols like the memcpy family
--
Glauber de Oliveira Costa.
"Free as in Freedom"
http://glommer.net
"The less confident you are, the more serious you have to act.
>
> Is the .globl really required here? I would think that the reference in
> the fixup section would be resolved with in the compiling of this object.
>
For the fixup yes, you are right. But I'm using this symbol in
paravirt.c now, to denote the native operation.
--
Glauber de Ol
proper sense. Altough we do can introduce such operations for
clarity of code, I personally believe it is not the way to go.
What I did, then, was move this outside the PARAVIRT ifdef, which
lead to a much cleaner code.
--
Glauber de Oliveira Costa.
"Free as in Freedom"
http://glommer.net
&
)
* modifications needed to accomodate other parts of the
patch that changed, such as getting rid of ebda_info
* put the integers at struct paravirt_ops at the end
(Jeremy)
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL
)
* Make paravirt_setup_arch() void (Andi)
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/setup.c | 32 +++-
include/asm-x86_64/e820.h |6 ++
include/asm-x
This patch turns the irq_flags and halt routines into the
native versions.
[ updates from v1
Move raw_irqs_disabled_flags outside of the PARAVIRT ifdef to
avoid increasing the mess, suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by:
This patch introduces apply_paravirt(), a function that shall
be called by i386/alternative.c to apply replacements to
paravirt_functions. It is defined to an do-nothing function
if paravirt is not enabled.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven R
This is hopefully the last iteration of the pvops64 patch.
>From the last version, we have only one change, which is
>include/asm-x86_64/processor.h: There were still one survivor in raw asm.
Also, git screwed me up for some reason, and the 25th patch was missing the new
files, paravirt.{c,h}.
.
The timer interrupt setup also have to be replaced.
The job is done by time_init_hook().
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/time.c | 37 +
includ
under paravirt, read cr2 cannot be issued directly anymore.
So wrap it in a macro, defined to the operation itself in case
paravirt is off, but to something else if we have paravirt
in the game
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt &
-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/x8664_ksyms.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/arch/x86_64/kernel/x8664_ksyms.c b/arch/x86_64/kernel/x8664_ksyms.c
index 77c2
This patch replaces syscall_init by x86_64_syscall_init.
The former will be later replaced by a paravirt replacement
in case paravirt is on
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/setu
This patch introduces a new macro/function that informs a paravirt
guest when its page table is not more in use, and can be released.
In case we're not paravirt, just do nothing.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL
to SWAPGS_UNSAFE_STACK
* change the unexpressive 1b label to do_iret
All suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/entry.S | 130 +
This function/macro will allow a paravirt guest to be notified we changed
the current task cr3, and act upon it. It's up to them
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/asm-x86_64/mmu_cont
This patch turns the page operations (set and make a page table)
into native_ versions. The operations itself will be later
overriden by paravirt.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/asm-x86_64/p
to be patched. We
also patch call sites that now needs a typecast.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/head64.c |2 +-
arch/x86_64/kernel/ldt.c |6 +-
arch/x86_64/ker
Besides not elegant, it is now even forbidden, since it can
break paravirtualized guests. load_cr3 should call write_cr3()
instead.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/asm-x86_64/mmu_context.h
are moved to below the #ifdef CONFIG_PARAVIRT
site, as they are users of the newly defined set_* macros.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/asm-x86_64/pgtable.h | 152
duplication of code in get_cycles_sync
* Get rid of rdtsc(), since it is used nowhere else
All three suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/ia32/syscall32.c |2 +-
arch/x
Export math_state_restore symbol, so it can be used for hypervisors.
They are commonly loaded as modules.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/traps.c |1 +
1 files changed, 1 inse
When paravirtualization is disabled, the kernel is always
running at ring 0. So report it in the appropriate macro
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/asm-x86_64/segment.h |4
1 file
-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/kernel/i8259.c |5 -
include/asm-x86_64/irq.h |2 ++
2 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/arch/x86_64/kernel/i8259.c b/arch/x
This patch switches the cli and sti instructions into macros.
In this header, they're just defined to the instructions they
refer to. Later on, when paravirt is defined, they will be
defined to something with paravirt abilities.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Time for the apic handling functions to get their native counterparts.
Also, put the native hook for the boot clocks functions in the apic.h header
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/k
This patch turns the flush_tlb routines into native versions.
In case paravirt is not defined, the natives are defined into
the actually used ones. flush_tlb_others() goes in smp.c, unless
smp is not in the game
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by:
This patch adds native hooks for debugreg handling functions,
and for the native load_rsp0 function. The later also have its
call sites patched.
[ updates from v2
* there were still a raw reference to cr4 missing
]
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Sign
This patch adds the native hook for the functions in system.h
They are the read/write_crX, clts and wbinvd. The later, also
gets its call sites patched.
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
arch/x86_64/
Later on, the paravirt_ops patch will deference the vm_area_struct
in asm/pgtable.h. It means this define must be after the struct
definition
Signed-off-by: Glauber de Oliveira Costa <[EMAIL PROTECTED]>
Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]>
---
include/linux
Later on, the paravirt_ops patch will deference the vm_area_struct
in asm/pgtable.h. It means this define must be after the struct
definition
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/linux/mm.h | 14 +-
1
This patch turns the flush_tlb routines into native versions.
In case paravirt is not defined, the natives are defined into
the actually used ones. flush_tlb_others() goes in smp.c, unless
smp is not in the game
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven
This patch adds native hooks for debugreg handling functions,
and for the native load_rsp0 function. The later also have its
call sites patched.
[ updates from v2
* there were still a raw reference to cr4 missing
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off
This patch adds the native hook for the functions in system.h
They are the read/write_crX, clts and wbinvd. The later, also
gets its call sites patched.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/tce.c
Time for the apic handling functions to get their native counterparts.
Also, put the native hook for the boot clocks functions in the apic.h header
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/apic.c|2
-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/i8259.c |5 -
include/asm-x86_64/irq.h |2 ++
2 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/arch/x86_64/kernel/i8259.c b/arch/x86_64/kernel/i8259
This patch switches the cli and sti instructions into macros.
In this header, they're just defined to the instructions they
refer to. Later on, when paravirt is defined, they will be
defined to something with paravirt abilities.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed
When paravirtualization is disabled, the kernel is always
running at ring 0. So report it in the appropriate macro
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/asm-x86_64/segment.h |4
1 files changed, 4
Export math_state_restore symbol, so it can be used for hypervisors.
They are commonly loaded as modules.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/traps.c |1 +
1 files changed, 1 insertions(+), 0
are moved to below the #ifdef CONFIG_PARAVIRT
site, as they are users of the newly defined set_* macros.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/asm-x86_64/pgtable.h | 152 -
1
duplication of code in get_cycles_sync
* Get rid of rdtsc(), since it is used nowhere else
All three suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/ia32/syscall32.c |2 +-
arch/x86_64/kernel
to be patched. We
also patch call sites that now needs a typecast.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/head64.c |2 +-
arch/x86_64/kernel/ldt.c |6 +-
arch/x86_64/kernel/reboot.c
This function/macro will allow a paravirt guest to be notified we changed
the current task cr3, and act upon it. It's up to them
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/asm-x86_64/mmu_context.h | 17
This patch turns the page operations (set and make a page table)
into native_ versions. The operations itself will be later
overriden by paravirt.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/asm-x86_64/page.h | 36
This patch introduces a new macro/function that informs a paravirt
guest when its page table is not more in use, and can be released.
In case we're not paravirt, just do nothing.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED
to SWAPGS_UNSAFE_STACK
* change the unexpressive 1b label to do_iret
All suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/entry.S | 130 +---
1 files
-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/x8664_ksyms.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/arch/x86_64/kernel/x8664_ksyms.c b/arch/x86_64/kernel/x8664_ksyms.c
index 77c25b3..2d3932d
This patch replaces syscall_init by x86_64_syscall_init.
The former will be later replaced by a paravirt replacement
in case paravirt is on
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/setup64.c |8
Besides not elegant, it is now even forbidden, since it can
break paravirtualized guests. load_cr3 should call write_cr3()
instead.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
include/asm-x86_64/mmu_context.h |2 +-
1 files
.
The timer interrupt setup also have to be replaced.
The job is done by time_init_hook().
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/time.c | 37 +
include/asm-x86_64/time.h
under paravirt, read cr2 cannot be issued directly anymore.
So wrap it in a macro, defined to the operation itself in case
paravirt is off, but to something else if we have paravirt
in the game
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL
This patch introduces apply_paravirt(), a function that shall
be called by i386/alternative.c to apply replacements to
paravirt_functions. It is defined to an do-nothing function
if paravirt is not enabled.
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt
This is hopefully the last iteration of the pvops64 patch.
From the last version, we have only one change, which is
include/asm-x86_64/processor.h: There were still one survivor in raw asm.
Also, git screwed me up for some reason, and the 25th patch was missing the new
files, paravirt.{c,h}.
)
* Make paravirt_setup_arch() void (Andi)
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED]
---
arch/x86_64/kernel/setup.c | 32 +++-
include/asm-x86_64/e820.h |6 ++
include/asm-x86_64/page.h |1
This patch turns the irq_flags and halt routines into the
native versions.
[ updates from v1
Move raw_irqs_disabled_flags outside of the PARAVIRT ifdef to
avoid increasing the mess, suggested by Andi Kleen
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt
)
* modifications needed to accomodate other parts of the
patch that changed, such as getting rid of ebda_info
* put the integers at struct paravirt_ops at the end
(Jeremy)
]
Signed-off-by: Glauber de Oliveira Costa [EMAIL PROTECTED]
Signed-off-by: Steven Rostedt [EMAIL PROTECTED
do can introduce such operations for
clarity of code, I personally believe it is not the way to go.
What I did, then, was move this outside the PARAVIRT ifdef, which
lead to a much cleaner code.
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you
Is the .globl really required here? I would think that the reference in
the fixup section would be resolved with in the compiling of this object.
For the fixup yes, you are right. But I'm using this symbol in
paravirt.c now, to denote the native operation.
--
Glauber de Oliveira Costa.
Free
duty of this file (x8664_syms.c), and the
extern is also used in other symbols like the memcpy family
--
Glauber de Oliveira Costa.
Free as in Freedom
http://glommer.net
The less confident you are, the more serious you have to act.
-
To unsubscribe from this list: send the line unsubscribe linux
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 11:18:25AM -0300, Glauber de Oliveira Costa wrote:
Didn't we agree this should be a pvops client?
-Andi
No. I exposed my reasoning, asked you back, but got no answer.
I'll do it again:
This operations are just manipulating bits, and are doing
Avi Kivity escreveu:
Glauber de Oliveira Costa wrote:
Andi Kleen escreveu:
On Wed, Aug 15, 2007 at 11:18:25AM -0300, Glauber de Oliveira Costa
wrote:
Didn't we agree this should be a pvops client?
-Andi
No. I exposed my reasoning, asked you back, but got no answer.
I'll do it again
701 - 800 of 1034 matches
Mail list logo
