On Thu, 22 Dec 2011, Asias He wrote:
It's quite annoying to print in the guest side. And we can reflect the
guest status in 'lkvm list' now. So I'd like to remove the guest side
notification.
Sure. It's not going to work either once we start supporting X. I'm pretty
sure we'd need to get the g
On 12/21/2011 05:14 PM, Wanlong Gao wrote:
> On 12/21/2011 04:07 PM, Asias He wrote:
>
>> ./lkvm {pause,resume} do not give any feedback to user who uses these
>> commands in the console where the command run.
>>
>> This patch makes the command output in the command console instead of
>> guest con
On 12/22/2011 04:30 AM, Pekka Enberg wrote:
> On Wed, 21 Dec 2011, Asias He wrote:
>> This is useful when client submiting multiple IPC cmd in one socket
>> connection.
>>
>> Signed-off-by: Asias He
>
> This patch doesn't apply on top of master after Lai's patches. Can you
> please resend?
Sure.
No need to send IPC msg using bare write() any more.
Signed-off-by: Asias He
---
tools/kvm/builtin-list.c | 20
1 files changed, 4 insertions(+), 16 deletions(-)
diff --git a/tools/kvm/builtin-list.c b/tools/kvm/builtin-list.c
index e74b636..eb0ac6c 100644
--- a/tools/kvm
Remove this annoying message if user resumes or pauses a guest whcih
is already resumed or paused.
Warning: (builtin-run.c) handle_pause:520: failed condition: 1
Signed-off-by: Asias He
---
tools/kvm/builtin-run.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/tools
lkvm {pause,resume} do not give any feedback to user who uses these
commands in the console where the command run.
This patch makes the command output in the command console instead of
guest console.
Signed-off-by: Asias He
---
tools/kvm/builtin-pause.c | 10 +-
tools/kvm/builtin-res
'./lkvm list' does not give the correct vm status if the vm is pasued
currently.
This patch fixes this by using KVM_IPC_VMSTATE IPC cmd to query the vm
status.
Suppose the guest is paused by ./lkvm pause:
Before:
asias@hj$ ./lkvm list
PID NAME STATE
--
This can be used to get vm status information:
vm is running or pasued.
Signed-off-by: Asias He
---
tools/kvm/builtin-run.c | 20 +---
tools/kvm/include/kvm/kvm-ipc.h |1 +
tools/kvm/include/kvm/kvm.h |5 +
tools/kvm/powerpc/i
This is useful when client submiting multiple IPC cmd in one socket
connection.
Signed-off-by: Asias He
---
tools/kvm/kvm-ipc.c | 15 ---
1 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/tools/kvm/kvm-ipc.c b/tools/kvm/kvm-ipc.c
index 2036b91..dbe2363 100644
--- a/to
Hi Vivek,
On Wed, Dec 21, 2011 at 02:11:17PM -0500, Vivek Goyal wrote:
> On Wed, Dec 21, 2011 at 10:00:48AM +0900, Minchan Kim wrote:
> > This patch is follow-up of Christohp Hellwig's work
> > [RFC: ->make_request support for virtio-blk].
> > http://thread.gmane.org/gmane.linux.kernel/1199763
> >
Remove useless arguments kvm_dev and name.
Signed-off-by: Wanlong Gao
---
tools/kvm/include/kvm/kvm.h |2 +-
tools/kvm/kvm.c |2 +-
tools/kvm/powerpc/kvm.c |2 +-
tools/kvm/x86/kvm.c |2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/too
On Tue, Dec 20, 2011 at 04:23:28PM +0100, Alexander Graf wrote:
> > - vcpu = kzalloc(sizeof(struct kvm_vcpu), GFP_KERNEL);
> > + vcpu = kmem_cache_zalloc(kvm_vcpu_cache, GFP_KERNEL);
>
> Paul, is there any rationale on why not to use the kmem cache? Are
> we bound by real mode magic again?
N
This provides the base group management with conduits to the
IOMMU driver and VFIO bus drivers.
Signed-off-by: Alex Williamson
---
drivers/vfio/vfio_main.c| 1201 +++
drivers/vfio/vfio_private.h | 36 +
2 files changed, 1237 insertions(+), 0 deletio
Enable the base code.
Signed-off-by: Alex Williamson
---
MAINTAINERS |8
drivers/Kconfig |2 ++
drivers/Makefile |1 +
drivers/vfio/Kconfig |8
drivers/vfio/Makefile |3 +++
5 files changed, 22 insertions(+), 0 deletions(-)
create mod
Backing for operations on the IOMMU object, including DMA
mapping and unmapping.
Signed-off-by: Alex Williamson
---
drivers/vfio/vfio_iommu.c | 593 +
1 files changed, 593 insertions(+), 0 deletions(-)
create mode 100644 drivers/vfio/vfio_iommu.c
d
This defines both the user and bus driver APIs.
Signed-off-by: Alex Williamson
---
Documentation/ioctl/ioctl-number.txt |1
include/linux/vfio.h | 353 ++
2 files changed, 354 insertions(+), 0 deletions(-)
create mode 100644 include/linux/v
Including rationale for design, example usage and API description.
Signed-off-by: Alex Williamson
---
Documentation/vfio.txt | 352
1 files changed, 352 insertions(+), 0 deletions(-)
create mode 100644 Documentation/vfio.txt
diff --git a/Docum
n and the VFIO PCI bus driver, for exposing PCI devices
through VFIO, can be found here:
git://github.com/awilliam/linux-vfio.git vfio-next-20111221
A development version of qemu which includes a full working
vfio-pci driver, indepdendent of KVM support, can be found here:
git://github.com/awi
On Wed, 21 Dec 2011, Wanlong Gao wrote:
Remove useless arguments kvm_dev and name.
Signed-off-by: Wanlong Gao
This doesn't apply on top of master. Can you please resend?
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
Mor
On Wed, 21 Dec 2011, Asias He wrote:
This is useful when client submiting multiple IPC cmd in one socket
connection.
Signed-off-by: Asias He
This patch doesn't apply on top of master after Lai's patches. Can you
please resend?
Pekka
--
To unsubscribe from this list:
On Thu, Dec 15, 2011 at 12:38:40PM +0200, Gleb Natapov wrote:
> Drop bsp_vcpu pointer from kvm struct since its only use is incorrect
> anyway.
>
> Signed-off-by: Gleb Natapov
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord..
On Wed, Dec 14, 2011 at 07:25:33PM +0100, Jan Kiszka wrote:
> Move the test for KVM_PIT_FLAGS_HPET_LEGACY into create_pit_timer
> instead of replicating it on the caller site.
>
> Signed-off-by: Jan Kiszka
> ---
> arch/x86/kvm/i8254.c | 10 +++---
> 1 files changed, 3 insertions(+), 7 dele
On Wed, Dec 21, 2011 at 10:00:48AM +0900, Minchan Kim wrote:
> This patch is follow-up of Christohp Hellwig's work
> [RFC: ->make_request support for virtio-blk].
> http://thread.gmane.org/gmane.linux.kernel/1199763
>
> Quote from hch
> "This patchset allows the virtio-blk driver to support much h
* Chris Wright (chr...@sous-sol.org) wrote:
> * Hao, Xudong (xudong@intel.com) wrote:
> > Yes, Chris, thanks your comments.
> > How about this one?
>
> Yes, it gets the locking right.
Sorry, I missed one other problem on the error path. You need to also
update pdev->dev.archdata.iommu to NUL
* Hao, Xudong (xudong@intel.com) wrote:
> Yes, Chris, thanks your comments.
> How about this one?
Yes, it gets the locking right.
Also makes host device and guest assigned device go through the same order:
alloc_devinfo and init
lock; place info on lists; unlock
domain_context_mapping()
The
On Thu, Dec 15, 2011 at 02:42:02PM -0500, Eric B Munson wrote:
> Changes from V6:
> Use __this_cpu_and when clearing the PVCLOCK_GUEST_STOPPED flag
Avi,
Do you have any further comments on how this should be done?
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of
Remove useless arguments kvm_dev and name.
Signed-off-by: Wanlong Gao
---
tools/kvm/include/kvm/kvm.h |2 +-
tools/kvm/kvm.c |2 +-
tools/kvm/powerpc/kvm.c |2 +-
tools/kvm/x86/kvm.c |2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/too
The commit 2b6c246215e8f936d2366ef8c4a6eec730b819b2
(kvm tools: Use compat message per device instead of per instance)
prevents multiple messages for the same type of device.
The variable compat_id is expected to be initialized to -1.
Signed-off-by: Asias He
---
tools/kvm/virtio/blk.c |2 +-
This is to inform you that your recent unsubscribe request was unsuccessful.
This is probably because we could find no current subscription in your name.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at
If the guest is stopped, there is no need to run it.
This patch fixes this when running 'lkvm stop'.
KVM_RUN failed: Bad address
Signed-off-by: Asias He
---
tools/kvm/kvm-cpu.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/tools/kvm/kvm-cpu.c b/tools/kvm/kvm-cpu.
Currently, 'lkvm stop' can not stop a pasued guest becasue
guest is blocked on the pause_lock.
This patch fixes it by un-pausing the guest before stops it.
The pthread_kill() call is not needed.
Signed-off-by: Asias He
---
tools/kvm/kvm-cpu.c |2 +-
1 files changed, 1 insertions(+), 1 dele
'./lkvm list' does not give the correct vm status if the vm is pasued
currently.
This patch fixes this by using KVM_IPC_VMSTATE IPC cmd to query the vm
status.
Suppose the guest is paused by ./lkvm pause:
Before:
asias@hj$ ./lkvm list
PID NAME STATE
--
This can be used to get vm status information:
vm is running or pasued.
Signed-off-by: Asias He
---
tools/kvm/builtin-run.c | 20 +---
tools/kvm/include/kvm/kvm-ipc.h |1 +
tools/kvm/include/kvm/kvm.h |5 +
tools/kvm/powerpc/i
This is useful when client submiting multiple IPC cmd in one socket
connection.
Signed-off-by: Asias He
---
tools/kvm/kvm-ipc.c | 15 ---
1 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/tools/kvm/kvm-ipc.c b/tools/kvm/kvm-ipc.c
index 68c2565..f2c3523 100644
--- a/to
On 12/21/2011 01:58 PM, Jan Kiszka wrote:
> > +1
>
> Ah, the adult crowd that trusts the other side.
It's 80% gnome-shell rants and similar, don't bother.
> OK, will see that I can cook up a proper patch later.
>
Thanks.
--
error compiling committee.c: too many arguments to function
--
To uns
On 2011-12-21 12:45, Avi Kivity wrote:
> On 12/21/2011 01:28 PM, Jan Kiszka wrote:
>>>
>>> timer_mode_mask can just be slaved to the bit (as received by
>>> KVM_SET_CPUID); that's exactly kvm_update_cpuid()'s role.
>>
>> Like
>
> Ugh, the facebook generation invades lkml.
I'm not on facebook. Doe
On 12/21/2011 01:28 PM, Jan Kiszka wrote:
> >
> > timer_mode_mask can just be slaved to the bit (as received by
> > KVM_SET_CPUID); that's exactly kvm_update_cpuid()'s role.
>
> Like
Ugh, the facebook generation invades lkml.
> diff --git a/arch/x86/kvm/cpuid.c b/arch/x86/kvm/cpuid.c
> index 230
On 2011-12-21 11:44, Avi Kivity wrote:
> On 12/21/2011 12:41 PM, Jan Kiszka wrote:
>> On 2011-12-21 11:35, Avi Kivity wrote:
>>> On 12/21/2011 12:25 PM, Jan Kiszka wrote:
We must not report the TSC deadline timer feature on our own when user
space provides the APIC as we have no clue abou
On 12/21/2011 12:41 PM, Jan Kiszka wrote:
> On 2011-12-21 11:35, Avi Kivity wrote:
> > On 12/21/2011 12:25 PM, Jan Kiszka wrote:
> >> We must not report the TSC deadline timer feature on our own when user
> >> space provides the APIC as we have no clue about its features.
> >
> > We must not repor
On Wed, Dec 21, 2011 at 12:25, Jan Kiszka wrote:
> We must not report the TSC deadline timer feature on our own when user
> space provides the APIC as we have no clue about its features.
>
> Signed-off-by: Jan Kiszka
> ---
>
> Changes in v2:
> - reorganized code to make the flow clearer
I've se
On 2011-12-21 11:35, Avi Kivity wrote:
> On 12/21/2011 12:25 PM, Jan Kiszka wrote:
>> We must not report the TSC deadline timer feature on our own when user
>> space provides the APIC as we have no clue about its features.
>
> We must not report the TSC deadline timer feature on our own, period.
On 12/21/2011 12:25 PM, Jan Kiszka wrote:
> We must not report the TSC deadline timer feature on our own when user
> space provides the APIC as we have no clue about its features.
We must not report the TSC deadline timer feature on our own, period.
We should just update the timer mode mask there
On 2011-12-21 05:59, Alex Williamson wrote:
> Only allow KVM device assignment to attach to devices which:
>
> - Are not bridges
> - Have BAR resources (assume others are special devices)
> - The user has permissions to use
>
> Assigning a bridge is a configuration error, it's not supported, a
We must not report the TSC deadline timer feature on our own when user
space provides the APIC as we have no clue about its features.
Signed-off-by: Jan Kiszka
---
Changes in v2:
- reorganized code to make the flow clearer
arch/x86/kvm/cpuid.c | 18 --
1 files changed, 8 ins
On Wed, 2011-12-21 at 14:15 +0400, Konstantin Khlebnikov wrote:
> Sasha Levin wrote:
> > On Wed, 2011-12-21 at 13:49 +0400, Konstantin Khlebnikov wrote:
> >> Sasha Levin wrote:
> >>> On Wed, 2011-12-21 at 13:15 +0300, Konstantin Khlebnikov wrote:
> fix permanent failing with message "Fatal: Fa
Pick arch/$ARCH/boot/bzImage and vmlinux if we are in linux source tree root
directory.
Signed-off-by: Konstantin Khlebnikov
---
tools/kvm/builtin-run.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-run.c
index 094c0fa2..50
Sasha Levin wrote:
On Wed, 2011-12-21 at 13:49 +0400, Konstantin Khlebnikov wrote:
Sasha Levin wrote:
On Wed, 2011-12-21 at 13:15 +0300, Konstantin Khlebnikov wrote:
fix permanent failing with message "Fatal: Failed linking stage 2 of init."
if there is no custom init_stage2.
Why would it ha
On Wed, 2011-12-21 at 13:49 +0400, Konstantin Khlebnikov wrote:
> Sasha Levin wrote:
> > On Wed, 2011-12-21 at 13:15 +0300, Konstantin Khlebnikov wrote:
> >> fix permanent failing with message "Fatal: Failed linking stage 2 of init."
> >> if there is no custom init_stage2.
> >
> > Why would it happ
Sasha Levin wrote:
On Wed, 2011-12-21 at 13:15 +0300, Konstantin Khlebnikov wrote:
fix permanent failing with message "Fatal: Failed linking stage 2 of init."
if there is no custom init_stage2.
Why would it happen? whats the usecase?
run "lkvm run" in dir there is no "guest" subdirectory
--
On Wed, 2011-12-21 at 13:15 +0300, Konstantin Khlebnikov wrote:
> fix permanent failing with message "Fatal: Failed linking stage 2 of init."
> if there is no custom init_stage2.
Why would it happen? whats the usecase?
--
Sasha.
--
To unsubscribe from this list: send the line "unsubscribe kvm"
On 12/21/2011 04:07 PM, Asias He wrote:
> lkvm alias vm is not removed in 'make clean'.
> This patch fixes this.
>
> Signed-off-by: Asias He
Reviewed-by: Wanlong Gao
> ---
> tools/kvm/Makefile |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tools/kvm/Makefile
On 12/21/2011 04:07 PM, Asias He wrote:
> ./lkvm {pause,resume} do not give any feedback to user who uses these
> commands in the console where the command run.
>
> This patch makes the command output in the command console instead of
> guest console.
I's like notice users in both sides, so don
fix permanent failing with message "Fatal: Failed linking stage 2 of init."
if there is no custom init_stage2.
Signed-off-by: Konstantin Khlebnikov
---
tools/kvm/builtin-run.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-ru
- Original Message -
> On 07/19/2011 02:05 PM, Sasha Levin wrote:
> > On Tue, 2011-07-19 at 13:57 +0300, Avi Kivity wrote:
> > > On 07/19/2011 01:31 PM, Sasha Levin wrote:
> > > > This patch changes coalesced mmio to create one mmio device
> > > > per
> > > > zone instead of handling
Hi Sasha!
On Wed, Dec 21, 2011 at 10:28:52AM +0200, Sasha Levin wrote:
> On Wed, 2011-12-21 at 10:00 +0900, Minchan Kim wrote:
> > This patch is follow-up of Christohp Hellwig's work
> > [RFC: ->make_request support for virtio-blk].
> > http://thread.gmane.org/gmane.linux.kernel/1199763
> >
> > Q
./lkvm {pause,resume} do not give any feedback to user who uses these
commands in the console where the command run.
This patch makes the command output in the command console instead of
guest console.
Signed-off-by: Asias He
---
tools/kvm/builtin-pause.c |2 ++
tools/kvm/builtin-resume.c
lkvm alias vm is not removed in 'make clean'.
This patch fixes this.
Signed-off-by: Asias He
---
tools/kvm/Makefile |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/kvm/Makefile b/tools/kvm/Makefile
index ce3df65..d54fa47 100644
--- a/tools/kvm/Makefile
+++ b/tools
57 matches
Mail list logo