On 2011-02-09 04:00, Huang Ying wrote:
In Linux kernel HWPoison processing implementation, the virtual
address in processes mapping the error physical memory page is marked
as HWPoison. So that, the further accessing to the virtual
address will kill corresponding processes with SIGBUS.
If
Anthony Liguori anth...@codemonkey.ws writes:
On 02/08/2011 11:13 AM, Markus Armbruster wrote:
Chris Wrightchr...@redhat.com writes:
[...]
- qdev/vmstate both examples of partially completed work that need more
attention
As far as qdev's concerned, I can see two kinds of
On 2011-02-08 19:59, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:15PM +0100, Jan Kiszka wrote:
index d6556c9..3397566 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -2194,14 +2194,14 @@ static void gdb_vm_state_change(void *opaque, int
running, int reason)
const char *type;
On 2011-02-08 19:50, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:13PM +0100, Jan Kiszka wrote:
Avoid duplicate use of the function name cpu_has_work, it's confusing.
Refactor cpu_has_work to cpu_is_idle and do the same with
any_cpu_has_work.
Signed-off-by: Jan Kiszka
Peter Maydell peter.mayd...@linaro.org writes:
On 8 February 2011 17:13, Markus Armbruster arm...@redhat.com wrote:
As far as qdev's concerned, I can see two kinds of to-dos:
* Further develop qdev so that more of the machine init code can becomes
qdev declarations. Specific ideas welcome.
On 9 February 2011 08:11, Markus Armbruster arm...@redhat.com wrote:
Peter Maydell peter.mayd...@linaro.org writes:
Markus Armbruster arm...@redhat.com wrote:
I've said this before: at some point in time (sooner rather than
later, if you ask me), we need to shoot the stragglers.
...and my
Aurelien Jarno aurel...@aurel32.net writes:
On Tue, Feb 08, 2011 at 06:13:53PM +0100, Markus Armbruster wrote:
Chris Wright chr...@redhat.com writes:
[...]
- qdev/vmstate both examples of partially completed work that need more
attention
As far as qdev's concerned, I can see two
Peter Maydell peter.mayd...@linaro.org writes:
On 9 February 2011 08:11, Markus Armbruster arm...@redhat.com wrote:
Peter Maydell peter.mayd...@linaro.org writes:
Markus Armbruster arm...@redhat.com wrote:
I've said this before: at some point in time (sooner rather than
later, if you ask
On 02/08/2011 01:55 PM, Jan Kiszka wrote:
Code under this lock requires non-preemptibility. Ensure this also over
-rt by converting it to raw spinlock.
Please add a code comment documenting what data under this lock requires
it to be a raw spinlock.
--
error compiling committee.c: too many
On 02/09/2011 11:24 AM, Avi Kivity wrote:
On 02/08/2011 01:55 PM, Jan Kiszka wrote:
Code under this lock requires non-preemptibility. Ensure this also over
-rt by converting it to raw spinlock.
Please add a code comment documenting what data under this lock
requires it to be a raw spinlock.
On 02/09/2011 03:50 AM, Michael Goldish wrote:
This is a reimplementation of the dict generator. It is much faster than the
current implementation and uses a very small amount of memory. Running time
and memory usage scale polynomially with the number of defined variants,
compared to
On Tue, Feb 08, 2011 at 07:22:29PM -0500, Andre Przywara wrote:
Somehow the code line advancing the RIP and checking for exceptions
got dropped between the post on the ML and the commit.
Add it again to let guests boot on upcoming AMD CPUs again.
Reported-by: Joerg Roedel
On 02/09/2011 11:28 AM, Avi Kivity wrote:
On 02/09/2011 03:50 AM, Michael Goldish wrote:
This is a reimplementation of the dict generator. It is much faster
than the
current implementation and uses a very small amount of memory.
Running time
and memory usage scale polynomially with the
isr_ack logic was added by e48258009d to avoid unnecessary IPIs. Back
then it made sense, but now the code checks that vcpu is ready to accept
interrupt before sending IPI, so this logic is no longer needed. The
patch removes it.
Signed-off-by: Gleb Natapov g...@redhat.com
diff --git
On 02/09/2011 12:07 PM, Michael Goldish wrote:
On 02/09/2011 11:28 AM, Avi Kivity wrote:
On 02/09/2011 03:50 AM, Michael Goldish wrote:
This is a reimplementation of the dict generator. It is much faster
than the
current implementation and uses a very small amount of memory.
Running
On 02/09/2011 02:01 AM, Markus Armbruster wrote:
Anthony Liguorianth...@codemonkey.ws writes:
On 02/08/2011 11:13 AM, Markus Armbruster wrote:
Chris Wrightchr...@redhat.com writes:
[...]
- qdev/vmstate both examples of partially completed work that need more
This is a cleanup patch for virtio_console:
1) Use the safer is None instead of == None or similar
comparisons
2) Remove some unused imports
3) Remove some unneded parenthesis
4) Correct typos
Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com
---
On 02/08/2011 01:30 PM, Aurelien Jarno wrote:
On Tue, Feb 08, 2011 at 06:13:53PM +0100, Markus Armbruster wrote:
Chris Wrightchr...@redhat.com writes:
[...]
- qdev/vmstate both examples of partially completed work that need more
attention
As far as qdev's concerned, I
Roedel, Joerg wrote:
On Tue, Feb 08, 2011 at 07:22:29PM -0500, Andre Przywara wrote:
Somehow the code line advancing the RIP and checking for exceptions
got dropped between the post on the ML and the commit.
Add it again to let guests boot on upcoming AMD CPUs again.
Reported-by: Joerg Roedel
On 02/09/2011 05:27 AM, Andrew Morton wrote:
On Tue, 01 Feb 2011 22:25:45 +0530
Balbir Singh bal...@linux.vnet.ibm.com wrote:
Changelog v4
1. Add max_unmapped_ratio and use that as the upper limit
to check when to shrink the unmapped page cache (Christoph
Lameter)
Changelog v2
1. Use a
On Wed, 09 Feb 2011 14:46:32 +0800
Lai Jiangshan la...@cn.fujitsu.com wrote:
On 02/01/2011 09:29 PM, Luiz Capitulino wrote:
On Thu, 27 Jan 2011 16:20:27 +0800
Lai Jiangshan la...@cn.fujitsu.com wrote:
cpu-index which uses hyphen is better name.
Signed-off-by: Lai Jiangshan
- Original Message -
This is a cleanup patch for virtio_console:
1) Use the safer is None instead of == None or similar
comparisons
2) Remove some unused imports
3) Remove some unneded parenthesis
4) Correct typos
Thank you for your corrections.
Signed-off-by: Lucas Meneghel
Anthony Liguori anth...@codemonkey.ws writes:
On 02/09/2011 02:01 AM, Markus Armbruster wrote:
Anthony Liguorianth...@codemonkey.ws writes:
[...]
We need to unify the property model. We have QemuOpts, qdev
properties, and QObject which basically reinvents variant typing three
different
On 2011-02-09 10:25, Avi Kivity wrote:
On 02/09/2011 11:24 AM, Avi Kivity wrote:
On 02/08/2011 01:55 PM, Jan Kiszka wrote:
Code under this lock requires non-preemptibility. Ensure this also over
-rt by converting it to raw spinlock.
Please add a code comment documenting what data under this
On Wed, Feb 09, 2011 at 05:43:28AM -0500, Andre Przywara wrote:
Roedel, Joerg wrote:
On Tue, Feb 08, 2011 at 07:22:29PM -0500, Andre Przywara wrote:
Somehow the code line advancing the RIP and checking for exceptions
got dropped between the post on the ML and the commit.
Add it again to
On 02/09/2011 02:31 PM, Jan Kiszka wrote:
On 2011-02-09 10:25, Avi Kivity wrote:
On 02/09/2011 11:24 AM, Avi Kivity wrote:
On 02/08/2011 01:55 PM, Jan Kiszka wrote:
Code under this lock requires non-preemptibility. Ensure this also over
-rt by converting it to raw spinlock.
Please
On 02/08/2011 01:55 PM, Jan Kiszka wrote:
Code under this lock requires non-preemptibility. Ensure this also over
-rt by converting it to raw spinlock.
Applied, thanks.
--
error compiling committee.c: too many arguments to function
--
To unsubscribe from this list: send the line unsubscribe
On 02/09/2011 02:56 PM, Roedel, Joerg wrote:
On Wed, Feb 09, 2011 at 05:43:28AM -0500, Andre Przywara wrote:
Roedel, Joerg wrote:
On Tue, Feb 08, 2011 at 07:22:29PM -0500, Andre Przywara wrote:
Somehow the code line advancing the RIP and checking for exceptions
got dropped between
The goal of this document shall be
- overview of all locks used in KVM core
- provide details on the scope of each lock
- explain the lock type, specifically of a raw spin locks
- provide a lock ordering guide
Start with one dependency chain and two locks.
Signed-off-by: Jan Kiszka
On 2011-02-09 15:11, Jan Kiszka wrote:
The goal of this document shall be
- overview of all locks used in KVM core
- provide details on the scope of each lock
- explain the lock type, specifically of a raw spin locks
- provide a lock ordering guide
Start with one dependency chain and two
On Wed, Feb 09, 2011 at 08:59:23AM +0100, Jan Kiszka wrote:
On 2011-02-09 00:39, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:18PM +0100, Jan Kiszka wrote:
Mixing up TCG bits with KVM already led to problems around eflags
emulation on x86. Moreover, quite some code that TCG requires
On Wed, Feb 09, 2011 at 09:07:01AM +0100, Jan Kiszka wrote:
On 2011-02-08 19:59, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:15PM +0100, Jan Kiszka wrote:
index d6556c9..3397566 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -2194,14 +2194,14 @@ static void gdb_vm_state_change(void
On Wed, Feb 09, 2011 at 09:07:50AM +0100, Jan Kiszka wrote:
Do you really find it easier to read evaluations grouped with || ? I
don't.
I do, specifically as the old version was even more confusing in that
important detail return 0 vs. return 1. But even the new benefits
from the grouping
On 02/09/2011 06:28 AM, Markus Armbruster wrote:
Except that construction of a device requires initialization from an
array of variants (which is then type checked). The way we store the
variants is lossy because we convert back and forth to a string.
Yes, there's overlap, but no, a qdev
On 2011-02-09 15:17, Marcelo Tosatti wrote:
On Wed, Feb 09, 2011 at 09:07:01AM +0100, Jan Kiszka wrote:
On 2011-02-08 19:59, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:15PM +0100, Jan Kiszka wrote:
index d6556c9..3397566 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -2194,14 +2194,14
On 2011-02-09 15:44, Marcelo Tosatti wrote:
On Wed, Feb 09, 2011 at 08:59:23AM +0100, Jan Kiszka wrote:
On 2011-02-09 00:39, Marcelo Tosatti wrote:
On Mon, Feb 07, 2011 at 12:19:18PM +0100, Jan Kiszka wrote:
Mixing up TCG bits with KVM already led to problems around eflags
emulation on x86.
Avoid duplicate use of the function name cpu_has_work, it's confusing,
also their scope. Refactor cpu_has_work to cpu_thread_is_idle and do the
same with any_cpu_has_work.
Signed-off-by: Jan Kiszka jan.kis...@siemens.com
---
Changes in v2:
- renaming: cpu_thread_is_idle and all_cpu_threads_idle
Define and use dedicated constants for vm_stop reasons, they actually
have nothing to do with the EXCP_* defines used so far. At this chance,
specify more detailed reasons so that VM state change handlers can
evaluate them.
Signed-off-by: Jan Kiszka jan.kis...@siemens.com
---
Changes in v2:
-
qemu-kvm-0.14.0-rc1 is now available. This release is based on the
upstream qemu 0.14.0-rc1, plus kvm-specific enhancements. Please see the
original qemu 0.14.0-rc1 release announcement for details.
This release can be used with the kvm kernel modules provided by your
distribution kernel, or
On Wed, Feb 09, 2011 at 10:06:03AM -0600, Ryan Harper wrote:
Instead of regular expressions in the filters, the following syntax is used:
, means OR
.. means AND
. means IMMEDIATELY-FOLLOWED-BY
Is there any reason we can't use | for or, and for AND? I know this
is just nit
Prasad Joshi wrote:
I was previously seeing a problem with the ROM BIOS, but I could solve
it by passing a correct BIOS file in function assigned_initfn. For now
I am using hardcoded file name in the code.
Did you recheck video bios inside guest? (for example in sysfs)
If this is bootstrap
On 07.02.2011, at 20:56, Yoder Stuart-B08248 wrote:
-Original Message-
From: Wood Scott-B07421
Sent: Monday, February 07, 2011 12:52 PM
To: Alexander Graf
Cc: Yoder Stuart-B08248; Wood Scott-B07421; kvm-...@vger.kernel.org;
kvm@vger.kernel.org; qemu-de...@nongnu.org
Subject:
On 07.02.2011, at 21:15, Scott Wood wrote:
On Mon, 7 Feb 2011 16:43:02 +0100
Alexander Graf ag...@suse.de wrote:
On 04.02.2011, at 23:33, Scott Wood wrote:
On Thu, 3 Feb 2011 10:19:06 +0100
Alexander Graf ag...@suse.de wrote:
Makes sense. So we basically need an ioctl that tells KVM
This patch implements two new vm-ioctls to get and set the
virtual_tsc_khz if the machine supports tsc-scaling. Setting
the tsc-frequency is only possible before userspace creates
any vcpu.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/kvm_host.h |1 +
Hi Avi, Marcelo,
here is the patch-set to implement the TSC-scaling feature of upcoming
AMD CPUs. When this feature is supported the CPU provides a new MSR
which holds a multiplier for the hardware TSC which is applied on the
value rdtsc[p] and reads of MSR 0x10. This feature can be used to
In the dr_intercept function a new cpu-feature called
decode-assists is implemented and used when available. This
code-path does not advance the guest-rip causing the guest
to dead-loop over mov-dr instructions. This is fixed by this
patch.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
With TSC scaling in SVM the tsc-offset needs to be
calculated differently. This patch propagates this
calculation into the architecture specific modules so that
this complexity can be handled there.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/kvm_host.h |1 +
This patch changes the kvm_guest_time_update function to use
TSC frequency the guest actually has for updating its clock.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/kvm_host.h |2 ++
arch/x86/kvm/svm.c |8
arch/x86/kvm/vmx.c
This patch implements the propagation of the VM
virtual_tsc_khz into each vcpu data-structure to enable the
tsc-scaling feature.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/kvm/svm.c | 32
1 files changed, 32 insertions(+), 0 deletions(-)
This patch enhances the kvm_amd module with functions to
support the TSC_RATE_MSR which can be used to set a given
tsc frequency for the guest vcpu.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/msr-index.h |1 +
arch/x86/kvm/svm.c | 37
On Wed, Feb 9, 2011 at 12:43 PM, Anthony Liguori anth...@codemonkey.ws wrote:
On 02/08/2011 01:30 PM, Aurelien Jarno wrote:
On Tue, Feb 08, 2011 at 06:13:53PM +0100, Markus Armbruster wrote:
Chris Wrightchr...@redhat.com writes:
[...]
- qdev/vmstate both examples of partially completed
On Wed, Feb 9, 2011 at 4:44 PM, Anthony Liguori anth...@codemonkey.ws wrote:
On 02/09/2011 06:28 AM, Markus Armbruster wrote:
Except that construction of a device requires initialization from an
array of variants (which is then type checked). The way we store the
variants is lossy because we
On 02/09/2011 06:48 PM, Blue Swirl wrote:
We can just do:
ISASerialState dev;
isa_serial_init(dev, 0, 0x274, 0x07, NULL, NULL);
Do you mean that there should be a generic way of doing that, like
sysbus_create_varargs() for qdev, or just add inline functions which
hide qdev property
On 02/09/2011 06:48 PM, Blue Swirl wrote:
ISASerialState dev;
isa_serial_init(dev, 0, 0x274, 0x07, NULL, NULL);
Do you mean that there should be a generic way of doing that, like
sysbus_create_varargs() for qdev, or just add inline functions which
hide qdev property setup?
I still think
On Wed, Feb 9, 2011 at 9:59 PM, Anthony Liguori anth...@codemonkey.ws wrote:
On 02/09/2011 06:48 PM, Blue Swirl wrote:
ISASerialState dev;
isa_serial_init(dev, 0, 0x274, 0x07, NULL, NULL);
Do you mean that there should be a generic way of doing that, like
sysbus_create_varargs() for qdev,
On Wed, Feb 9, 2011 at 4:33 PM, Konstantin Khlebnikov
khlebni...@openvz.org wrote:
Prasad Joshi wrote:
I was previously seeing a problem with the ROM BIOS, but I could solve
it by passing a correct BIOS file in function assigned_initfn. For now
I am using hardcoded file name in the code.
Avi Kivity avi at redhat.com writes:
Check reverse dns resolution and firewall rules. Don't seem to be a kvm
problem.
Thanks for the hint, it really was a typo in the zone config. Wouldn't have
thought to look there.
Volker
--
To unsubscribe from this list: send the line unsubscribe
On Wed, 9 Feb 2011 18:21:40 +0100
Alexander Graf ag...@suse.de wrote:
On 07.02.2011, at 21:15, Scott Wood wrote:
That's pretty much what the proposed API does -- except it uses a void
pointer instead of uint64_t *.
Oh? Did I miss something there? The proposal looked as if it only
* Eduardo Habkost ehabk...@redhat.com [2011-02-09 10:22]:
On Wed, Feb 09, 2011 at 10:06:03AM -0600, Ryan Harper wrote:
Instead of regular expressions in the filters, the following syntax is
used:
, means OR
.. means AND
. means IMMEDIATELY-FOLLOWED-BY
Is there any
On Thu, 3 Feb 2011 10:19:06 +0100
Alexander Graf ag...@suse.de wrote:
Yeah, that one's tricky. Usually the way the memory resolver in qemu works is
as follows:
* kvm goes to qemu
* qemu fetches all mmu and register data from kvm
* qemu runs its mmu resolution function as if the target
On Wed, 2011-02-09 at 16:00 +0800, Jan Kiszka wrote:
On 2011-02-09 04:00, Huang Ying wrote:
In Linux kernel HWPoison processing implementation, the virtual
address in processes mapping the error physical memory page is marked
as HWPoison. So that, the further accessing to the virtual
On Wed, Feb 09, 2011 at 11:28:56AM +0200, Avi Kivity wrote:
On 02/09/2011 03:50 AM, Michael Goldish wrote:
This is a reimplementation of the dict generator. It is much faster than the
current implementation and uses a very small amount of memory. Running time
and memory usage scale
Sorry for late response.
On Fri, Jan 28, 2011 at 8:18 PM, Balbir Singh bal...@linux.vnet.ibm.com wrote:
* MinChan Kim minchan@gmail.com [2011-01-28 16:24:19]:
But the assumption for LRU order to change happens only if the page
cannot be successfully freed, which means it is in some
I don't know why the part of message is deleted only when I send you.
Maybe it's gmail bug.
I hope mail sending is successful in this turn. :)
On Thu, Feb 10, 2011 at 2:33 PM, Minchan Kim minchan@gmail.com wrote:
Sorry for late response.
On Fri, Jan 28, 2011 at 8:18 PM, Balbir Singh
Prasad Joshi wrote:
On Wed, Feb 9, 2011 at 4:33 PM, Konstantin Khlebnikov
khlebni...@openvz.org mailto:khlebni...@openvz.org wrote:
Prasad Joshi wrote:
I was previously seeing a problem with the ROM BIOS, but I could
solve
it by passing a correct BIOS file in
On 02/09/2011 09:15 PM, Blue Swirl wrote:
On Wed, Feb 9, 2011 at 9:59 PM, Anthony Liguorianth...@codemonkey.ws wrote:
On 02/09/2011 06:48 PM, Blue Swirl wrote:
ISASerialState dev;
isa_serial_init(dev, 0, 0x274, 0x07, NULL, NULL);
Do you mean that there should be a
66 matches
Mail list logo