On Fri, Aug 14, 2015 at 01:57:42PM -0700, Andy Lutomirski wrote:
Don't bother testing yet. I'm waffling between trying something like
this and adding SA_SAVE_SS. I have partially written patches for the
latter.
ok, ping me if anything
--
To unsubscribe from this list: send the line
a
trampoline.
Cc: Stas Sergeev s...@list.ru
Cc: Linus Torvalds torva...@linux-foundation.org
Cc: Cyrill Gorcunov gorcu...@gmail.com
Cc: Pavel Emelyanov xe...@parallels.com
Signed-off-by: Andy Lutomirski l...@kernel.org
Looks reasonable to me. Andy, Linus, what the final conclusion --
are we about
On Thu, Jun 06, 2013 at 03:03:03PM +0300, Pekka Enberg wrote:
/* Set X86_FEATURE_HYPERVISOR */
if (entry-index == 0)
Ping! Is there someone out there who has a AMD box they could test this on?
I don't have it, sorry :-(
--
To unsubscribe from
Hi guys, here are a couple of fixes for make clean and cpuid filtering.
Please give them a shot.
--
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 http://vger.kernel.org/majordomo-info.html
This is redundant since we're to remove them right after
being generated.
CC: Ingo Molnar mi...@kernel.org
CC: Pekka Enberg penb...@kernel.org
CC: Sasha Levin sasha.le...@oracle.com
CC: Asias He as...@redhat.com
Signed-off-by: Cyrill Gorcunov gorcu...@openvz.org
---
tools/kvm/Makefile | 13
own string and kernel will use generic cpu init.
Reported-by: Ingo Molnar mi...@kernel.org
CC: Pekka Enberg penb...@kernel.org
CC: Sasha Levin sasha.le...@oracle.com
CC: Asias He as...@redhat.com
Signed-off-by: Cyrill Gorcunov gorcu...@openvz.org
---
tools/kvm/x86/cpuid.c |8
1 file
On Fri, Apr 12, 2013 at 12:17:58PM +0300, Pekka Enberg wrote:
On Fri, Apr 12, 2013 at 10:42 AM, Lin Ming min...@gmail.com wrote:
I run poweroff or halt in SDL window, but the window does not exit
although guest is already halted. But qemu can exit properly.
Is it because hlt instruction
On Fri, Oct 26, 2012 at 06:31:00PM +0300, Pekka Enberg wrote:
On Thu, 25 Oct 2012, Sasha Levin wrote:
I think we're seeing that because we don't handle VIRTIO_MSI_NO_VECTOR
properly.
We need to deal with the ability to remove GSI friends as well. I've
added it to my workqueue (unless
On Wed, Sep 12, 2012 at 11:03:36PM +0800, Asias He wrote:
We have this currently:
kvm__init()
kvm__arch_setup_firmware()
mptable__init()
using kvm-nrcpus
kvm_cpu__init()
kvm-nrcpus = kvm-cfg.nrcpus
kvm-nrcpus is used in mptable__init() before it
On Thu, Aug 30, 2012 at 09:36:37AM +0200, Sasha Levin wrote:
+ char *new_env[] = { TERM=linux, DISPLAY=192.168.33.1:0,
+ HOME=/virt/home, NULL };
+
+ mkdir(/virt/home, 0755);
Please add check for mkdir error code. Frankly, this is a bad habbit
to assume
On Thu, Aug 30, 2012 at 10:16:54AM +0200, Ingo Molnar wrote:
* Pekka Enberg penb...@kernel.org wrote:
On Thu, Aug 30, 2012 at 10:36 AM, Sasha Levin levinsasha...@gmail.com
wrote:
Build with -flto set, which should enable link-time-optimizations.
I'm not sure if it provides a
On Thu, Aug 30, 2012 at 10:33:21AM +0200, Sasha Levin wrote:
Ingo, any objections to this?
No objections if you can live with a 2x-4x increase in build
time - at worst it might cause funnies with the BIOS linker
script and such.
Maybe we could enable it via some make option?
On Thu, Aug 16, 2012 at 10:01:58AM +0300, Pekka Enberg wrote:
Hello,
[0.248962] Pid: 0, comm: swapper/0 Not tainted 3.6.0-rc1+ #24
[penberg@tux ~]$ cat perf-kvmtool-issue
Hello,
Has anyone seen this? It's kvmtool/next with 3.6.0-rc1. Looks like we
are doing uncore_init() on virtualized
On Thu, Aug 16, 2012 at 11:07:43AM +0400, Cyrill Gorcunov wrote:
On Thu, Aug 16, 2012 at 10:01:58AM +0300, Pekka Enberg wrote:
Hello,
[0.248962] Pid: 0, comm: swapper/0 Not tainted 3.6.0-rc1+ #24
[penberg@tux ~]$ cat perf-kvmtool-issue
Hello,
Has anyone seen this? It's kvmtool
On Thu, Aug 16, 2012 at 10:41:53AM +0300, Pekka Enberg wrote:
On 08/16/2012 03:19 PM, Peter Zijlstra wrote:
On Thu, 2012-08-16 at 10:01 +0300, Pekka Enberg wrote:
Has anyone seen this? It's kvmtool/next with 3.6.0-rc1. Looks like we
are doing uncore_init() on virtualized CPU which breaks
On Thu, Aug 16, 2012 at 11:45:52AM +0300, Avi Kivity wrote:
On Thu, Aug 16, 2012 at 10:38 AM, Yan, Zheng
zheng.z@linux.intel.com wrote:
The Intel uncore doc does not specify how to check if uncore exist.
How about disabling uncore on virtualized CPU?
(CC'ing Avi.)
Why not
On Wed, Jun 20, 2012 at 10:23:06AM +0300, Pekka Enberg wrote:
On Fri, 15 Jun 2012, Cyrill Gorcunov wrote:
On Fri, Jun 15, 2012 at 01:34:16PM +0200, Sasha Levin wrote:
We need to set the HYPERVISOR flag to let the kernel know we're running
under a hypervisor.
This makes the kernel
On Fri, Jun 15, 2012 at 01:34:16PM +0200, Sasha Levin wrote:
We need to set the HYPERVISOR flag to let the kernel know we're running
under a hypervisor.
This makes the kernel enable all sorts of para-virtualization options
such as kvm-clock.
Signed-off-by: Sasha Levin
From: Cyrill Gorcunov gorcu...@openvz.org
Use str[n|l] functions to make sure destination is
not overflowed.
Seems socket path generation should be moved into
a separate helper, but it's for another patch.
Signed-off-by: Cyrill Gorcunov gorcu...@openvz.org
---
tools/kvm/include/kvm/strbuf.h
On Tue, Jun 05, 2012 at 08:47:17AM +0800, Asias He wrote:
I must admit I don't understand this code ;) The data get read into
stack variable forever?
The data we read itself is not interesting at all. virtio_blk_thread()
sleeps on the eventfd bdev-io_efd until notify_vq() writes something
On Mon, Jun 04, 2012 at 11:40:53PM +0800, Asias He wrote:
+static void *virtio_blk_thread(void *dev)
+{
+ struct blk_dev *bdev = dev;
+ u64 data;
+
+ while (1) {
+ read(bdev-io_efd, data, sizeof(u64));
+ virtio_blk_do_io(bdev-kvm, bdev-vqs[0], bdev);
Otherwise I'm getting the following compile
problem on my Fedora machine. The helper is
rather taken from linux kernel.
| [cyrill@moon kvm]$ make tags
| x86/include/kvm/barrier.h:11:25: fatal error: asm/barrier.h: No such file or
directory compilation terminated.
Signed-off-by: Cyrill
On Thu, Apr 12, 2012 at 09:39:59PM +0800, 'Asias He wrote:
+static bool is_mounted(struct stat *st)
+{
+ struct stat st_buf;
+ struct mntent *mnt;
+ FILE *f;
+
+ f = setmntent(/proc/mounts, r);
+ if (!f)
+ return false;
+
+ while ((mnt = getmntent(f))
On Thu, Mar 08, 2012 at 05:11:16PM +0200, Pekka Enberg wrote:
On Thu, Mar 8, 2012 at 4:32 PM, Daniele Carollo carollo.d...@gmail.com
wrote:
for an university study, I'm doing some network test between two vm
using native linux kvm tool and connected via tap/virtio/vhost.
When I run my
On Thu, Mar 08, 2012 at 04:31:17PM +0100, Daniele Carollo wrote:
I don't really use the tap interface so lets CC Asias. Which guest
kernel are you using, btw?
Yup, Asias was using it, if my memory doesn't betray me. Also both -- host
and guest kernel versions might be useful to
asias.he...@gmail.com
Cc: Cyrill Gorcunov gorcu...@openvz.org
Cc: Ingo Molnar mi...@elte.hu
Cc: Sasha Levin levinsasha...@gmail.com
Signed-off-by: Pekka Enberg penb...@kernel.org
---
tools/kvm/builtin-run.c | 29 ++---
1 files changed, 26 insertions(+), 3 deletions
On Fri, Feb 24, 2012 at 05:05:29PM +0200, Pekka Enberg wrote:
This patch adds a --bios command line option to vm run. You can use this
to
try to boot with SeaBIOS, for example:
./vm run --bios=/usr/share/seabios/bios.bin \
--disk $HOME/images/debian_lenny_amd64_standard.qcow2
On Tue, Feb 14, 2012 at 11:07:08PM -0500, Kevin O'Connor wrote:
...
hardware. Maybe we could poke someone from KVM camp for a hint?
SeaBIOS has two ways to be deployed - first is to copy the image to
the top of the first 1MB (eg, 0xe-0xf) and jump to
0xf000:0xfff0 in 16bit mode.
On Tue, Feb 14, 2012 at 01:10:59PM +0200, Pekka Enberg wrote:
On Tue, Feb 14, 2012 at 1:03 PM, Yang Bai hamo...@gmail.com wrote:
Since on X86, bios is always at the end of the address space, so I
have some thought about how to implement the seabios support for kvm
tool.
1. using
On Tue, Feb 14, 2012 at 09:20:18PM +0800, Yang Bai wrote:
And will seabios replace the present bios implement or co-exsit?
Ideally we should get rid of our minibios completely and only have
seabios here instead.
Cyrill
--
To unsubscribe from this list: send the line unsubscribe kvm in
On Tue, Feb 14, 2012 at 05:35:47PM +0200, Pekka Enberg wrote:
On Tue, Feb 14, 2012 at 09:20:18PM +0800, Yang Bai wrote:
And will seabios replace the present bios implement or co-exsit?
On Tue, Feb 14, 2012 at 3:32 PM, Cyrill Gorcunov gorcu...@openvz.org wrote:
Ideally we should get rid
On Mon, Feb 13, 2012 at 08:14:22PM +0800, Yang Bai wrote:
Hi all,
As I know, native tool does not support loading BIOS so it does not
support Windows. Is this supporting now?
If not, I may try to implement it.
Nope yet. There was a plan to implement seabios support,
but nothing is done
On Thu, Feb 09, 2012 at 03:01:26PM +0200, Pekka Enberg wrote:
On Thu, Feb 9, 2012 at 7:40 AM, Yang Bai hamo...@gmail.com wrote:
Since the different issues have been handled in the
internal of kvm__init, it can only return NULL if error
happened.
Signed-off-by: Yang Bai hamo...@gmail.com
On Mon, Feb 06, 2012 at 12:22:04PM +0200, Pekka Enberg wrote:
Hi Michael,
On Mon, 6 Feb 2012, Michael Ellerman wrote:
Signed-off-by: Michael Ellerman mich...@ellerman.id.au
---
tools/kvm/builtin-run.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git
On Sat, Feb 04, 2012 at 10:02:19PM +0400, Cyrill Gorcunov wrote:
Strictly speaking, kvm__init need more serious rewrite together with
kvm__arch_init/kvm_ipc__start/kvm_ipc__register_handler ret. vals tests,
i'll do this a bit late.
Sorry for delay, was busy. Anyway, here is a quickfix
On Sat, Feb 04, 2012 at 02:15:36PM +0200, Pekka Enberg wrote:
On Fri, 3 Feb 2012, Cyrill Gorcunov wrote:
If guest name is used (which is default case) the kvm might end
up carrying the pointer to name which is allocated on stack.
kvm_cmd_run_init
(on stack) default_name
kvm__init
On Sat, Feb 04, 2012 at 04:20:05PM +0400, Cyrill Gorcunov wrote:
On Sat, Feb 04, 2012 at 02:15:36PM +0200, Pekka Enberg wrote:
On Fri, 3 Feb 2012, Cyrill Gorcunov wrote:
If guest name is used (which is default case) the kvm might end
up carrying the pointer to name which is allocated
On Sat, Feb 04, 2012 at 02:57:23PM +0200, Pekka Enberg wrote:
On Sat, 4 Feb 2012, Cyrill Gorcunov wrote:
Index: linux-2.6.git/tools/kvm/kvm.c
===
--- linux-2.6.git.orig/tools/kvm/kvm.c
+++ linux-2.6.git/tools/kvm/kvm.c
@@ -254,6
' not found. Please make sure your kernel has CONFIG_KVM
enabled and that the KVM modules are loaded.
| Segmentation fault (core dumped)
| [cyrill@moon kvm]$
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/builtin-run.c |4
tools/kvm/kvm.c | 11 ---
2 files
On Sat, Feb 04, 2012 at 07:38:41PM +0200, Pekka Enberg wrote:
On Sat, 4 Feb 2012, Cyrill Gorcunov wrote:
In case if there error happened in kvm__init and we have
no files opened -- we should not try to close them.
Also once kvm failed to init the caller should not try
to dereference
On Sat, Feb 04, 2012 at 07:54:38PM +0200, Pekka Enberg wrote:
On Sat, 4 Feb 2012, Cyrill Gorcunov wrote:
Index: linux-2.6.git/tools/kvm/kvm.c
===
--- linux-2.6.git.orig/tools/kvm/kvm.c
+++ linux-2.6.git/tools/kvm/kvm.c
On error mmap returns MAP_FAILED so we
need a proper test here.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/hw/pci-shmem.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Index: linux-2.6.git/tools/kvm/hw/pci-shmem.c
On Fri, Feb 03, 2012 at 11:15:41PM +0400, Cyrill Gorcunov wrote:
On error mmap returns MAP_FAILED so we
need a proper test here.
Pekka, pick this one instead -- a caller is expecting null/not-null
only.
Cyrill
---
kvm tools: Fix test for mmap failure
On error mmap returns MAP_FAILED
symbols should be more than enough.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
I hope I didn't miss anything?
tools/kvm/kvm.c |2 +-
tools/kvm/powerpc/include/kvm/kvm-arch.h |2 +-
tools/kvm/x86/include/kvm/kvm-arch.h |2 +-
3 files changed, 3
On Wed, Feb 01, 2012 at 09:05:34AM +0200, Pekka Enberg wrote:
On Tue, 31 Jan 2012, Sasha Levin wrote:
Might happen when hardware virtualization is not supported.
Reported-by: Ingo Molnar mi...@elte.hu
Signed-off-by: Sasha Levin levinsasha...@gmail.com
---
tools/kvm/builtin-run.c |4
On Wed, Feb 01, 2012 at 09:26:00AM +0200, Pekka Enberg wrote:
On Wed, 1 Feb 2012, Cyrill Gorcunov wrote:
I suspect we need something like
---
tools/kvm/builtin-run.c |5 +
tools/kvm/kvm.c |2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
Index: linux-2.6.git/tools
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/Makefile |2 ++
1 file changed, 2 insertions(+)
Index: linux-2.6.git/tools/kvm/Makefile
===
--- linux-2.6.git.orig/tools/kvm/Makefile
+++ linux-2.6.git/tools/kvm
Raise SIGABRT in case if run-time crtitical
problem found.
Proposed-by: Ingo Molnar mi...@elte.hu
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
Ingo, you meant something like below?
tools/kvm/include/kvm/util.h | 17 ++---
1 file changed, 14 insertions(+), 3 deletions
On Mon, Dec 19, 2011 at 10:19:43AM +0200, Pekka Enberg wrote:
On Mon, Dec 19, 2011 at 09:13:28AM +0200, Pekka Enberg wrote:
- BUILD_BUG_ON(i E820_X_MAX);
+ assert(i = E820_X_MAX);
We should use BUG_ON() like tools/perf does.
On Mon, Dec 19, 2011 at 9:57 AM, Cyrill Gorcunov
On Mon, Dec 19, 2011 at 11:40:09AM +0100, Ingo Molnar wrote:
GDB will catch that signal.
Yeah, good point! Pekka, drop this patch please, I'll make new one at evening.
Cyrill
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to
On Mon, Dec 19, 2011 at 11:50:31AM +0100, Ingo Molnar wrote:
...
The tool-specific BUG() implementation can be added as a delta
on top of that. It's in fact better to keep those two steps
separate.
OK, will do on top.
Cyrill
--
To unsubscribe from this list: send the line
for
‘__xchg_wrong_size’ [-Werror=missing-prototypes]
| ../../arch/x86/include/asm/cmpxchg.h: In function ‘__xchg_wrong_size’:
| ../../arch/x86/include/asm/cmpxchg.h:12:2: error: expected declaration
specifiers before ‘__compiletime_error’
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
The kernel already has pr_err helper lets do the same.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/builtin-stat.c |2 +-
tools/kvm/disk/core.c |2 +-
tools/kvm/include/kvm/util.h |2 +-
tools/kvm/kvm.c|2 +-
tools/kvm/util
Index is u8 value so array size should be 256.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/ui/sdl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.git/tools/kvm/ui/sdl.c
===
--- linux
If some of subsequent calls fails we better to return error
code instead of dying with a message. This is a first step
in getting rid of number of die() calls we have in code.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/builtin-run.c |5 -
tools/kvm
BUILD_BUG_ON is unable to catch errors on expression which
can't be evaluated at compile time.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/x86/bios.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Index: linux-2.6.git/tools/kvm/x86/bios.c
On Mon, Dec 19, 2011 at 09:13:28AM +0200, Pekka Enberg wrote:
-BUILD_BUG_ON(i E820_X_MAX);
+assert(i = E820_X_MAX);
We should use BUG_ON() like tools/perf does.
We dont have it yet. So I'll introduce this helper later,
but note that we will have to cover _all_ assert() calls
On Fri, Dec 16, 2011 at 10:40:06AM +0200, Sasha Levin wrote:
Signed-off-by: Sasha Levin levinsasha...@gmail.com
---
Thanks, Sasha!
Cyrill
--
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
On Wed, Dec 07, 2011 at 12:21:52PM +0200, Gleb Natapov wrote:
On Tue, Dec 06, 2011 at 10:42:55PM +0200, Sasha Levin wrote:
+static void handle_nmi(int fd, u32 type, u32 len, u8 *msg)
+{
+ u32 vcpu = *(u32 *)msg;
+
+ ioctl(kvm_cpus[vcpu]-vcpu_fd, KVM_NMI);
You need to check that
On Wed, Dec 07, 2011 at 12:33:05PM +0200, Gleb Natapov wrote:
On Wed, Dec 07, 2011 at 02:31:11PM +0400, Cyrill Gorcunov wrote:
On Wed, Dec 07, 2011 at 12:21:52PM +0200, Gleb Natapov wrote:
On Tue, Dec 06, 2011 at 10:42:55PM +0200, Sasha Levin wrote:
+static void handle_nmi(int fd, u32
On Wed, Dec 07, 2011 at 12:41:50PM +0200, Gleb Natapov wrote:
Yup, but while we support linux kernels only it should be fine. Still
of course on long term we need a check.
Tomorrow someone will send a patch to change how Linux behaves and
slightly older kvmtool will not be able to run
On Tue, Dec 06, 2011 at 12:25:29PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 5:42 AM, Matt Evans m...@ozlabs.org wrote:
vesa, pci-shmem and virtio-pci devices need to set up config space with
little-endian conversions (as config space is LE). The pci_config_address
bitfield also
On Tue, Dec 06, 2011 at 01:41:56PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 12:28 PM, Cyrill Gorcunov gorcu...@gmail.com wrote:
On Tue, Dec 06, 2011 at 12:25:29PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 5:42 AM, Matt Evans m...@ozlabs.org wrote:
vesa, pci-shmem
On Tue, Dec 06, 2011 at 12:29:48PM +0200, Pekka Enberg wrote:
...
Otherwise looks OK to me. Cyrill?
It might be not seen from patch (or my local kvm repo
is not yet updated well) but I somehow miss who will be
reading initrd in case of loading flat image? If noone,
then what's the point to
On Tue, Dec 06, 2011 at 01:58:24PM +0200, Pekka Enberg wrote:
On Tue, 2011-12-06 at 15:47 +0400, Cyrill Gorcunov wrote:
On Tue, Dec 06, 2011 at 01:41:56PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 12:28 PM, Cyrill Gorcunov gorcu...@gmail.com
wrote:
On Tue, Dec 06, 2011 at 12
On Tue, Dec 06, 2011 at 03:29:00PM +0200, Pekka Enberg wrote:
Hehe, this is because it should be rtaher defined as
union pci_config_address {
struct {
#if __BYTE_ORDER == __LITTLE_ENDIAN
unsignedzeros : 2;
unsignedregister_number : 6;
#else
...
On Wed, Dec 07, 2011 at 11:42:27AM +1100, Matt Evans wrote:
Hi Cyrill,
On 06/12/11 23:04, Cyrill Gorcunov wrote:
On Tue, Dec 06, 2011 at 12:29:48PM +0200, Pekka Enberg wrote:
...
Otherwise looks OK to me. Cyrill?
It might be not seen from patch (or my local kvm repo
is not yet
On Tue, Dec 06, 2011 at 12:25:29PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 5:42 AM, Matt Evans m...@ozlabs.org wrote:
vesa, pci-shmem and virtio-pci devices need to set up config space with
little-endian conversions (as config space is LE). The pci_config_address
bitfield also
On Tue, Dec 06, 2011 at 01:41:56PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 12:28 PM, Cyrill Gorcunov gorcu...@gmail.com wrote:
On Tue, Dec 06, 2011 at 12:25:29PM +0200, Pekka Enberg wrote:
On Tue, Dec 6, 2011 at 5:42 AM, Matt Evans m...@ozlabs.org wrote:
vesa, pci-shmem
On Tue, Dec 06, 2011 at 03:29:00PM +0200, Pekka Enberg wrote:
Hehe, this is because it should be rtaher defined as
union pci_config_address {
struct {
#if __BYTE_ORDER == __LITTLE_ENDIAN
unsignedzeros : 2;
unsignedregister_number : 6;
#else
...
On Wed, Dec 07, 2011 at 11:42:27AM +1100, Matt Evans wrote:
Hi Cyrill,
On 06/12/11 23:04, Cyrill Gorcunov wrote:
On Tue, Dec 06, 2011 at 12:29:48PM +0200, Pekka Enberg wrote:
...
Otherwise looks OK to me. Cyrill?
It might be not seen from patch (or my local kvm repo
is not yet
On Mon, Dec 05, 2011 at 11:22:11AM +0200, Sasha Levin wrote:
+static int kvm_custom_stage2(void)
+{
+ char tmp[PATH_MAX], dst[PATH_MAX], *src;
+ const char *rootfs;
+ int r;
+
+ src = realpath(guest/init_stage2, NULL);
+ if (src == NULL)
+ return
On Tue, Nov 29, 2011 at 03:01:59PM +0200, Pekka Enberg wrote:
Hi Sasha, where the rmb() then? Or maybe you wanted plain barrier() here?
On the kernel side.
Theres a mb there which happens there during the kick.
I guess we need to improve the comment in next_desc()?
Kernel's code has
On Mon, Nov 28, 2011 at 07:54:27PM +0200, Sasha Levin wrote:
+/*
+ * Each buffer in the virtqueues is actually a chain of descriptors. This
+ * function returns the next descriptor in the chain, or vq-vring.num if
we're
+ * at the end.
+ */
+static unsigned next_desc(struct vring_desc
On Wed, Nov 09, 2011 at 05:49:53PM +0200, Sasha Levin wrote:
On Wed, 2011-11-09 at 17:42 +0200, Richard Weinberger wrote:
On Wed, 09 Nov 2011 16:49:51 +0200, Sasha Levin
levinsasha...@gmail.com wrote:
We'll do kvm_clock as well if you compile it in the kernel.
CONFIG_HIGH_RES_TIMERS
On Wed, Nov 09, 2011 at 08:00:06PM +0400, Cyrill Gorcunov wrote:
...
You'll need CONFIG_KVM_CLOCK.
I'm not actually sure how close our implementation is to having tsc
working so far, Cyrill knows more about that than me.
We dropped tsc while were debuggin timer interrupts
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/builtin-run.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.git/tools/kvm/builtin-run.c
===
--- linux-2.6.git.orig/tools/kvm/builtin-run.c
On Tue, Sep 06, 2011 at 02:23:54AM +0300, Sasha Levin wrote:
This patch allows to run previously created guestfs by simply specifying it
with the '-d' parameter.
This allows running guestfs which were created before using:
kvm setup -n [name]
Signed-off-by: Sasha Levin
On Thu, Jul 28, 2011 at 12:31:51PM +0300, Pekka Enberg wrote:
On Thu, Jul 28, 2011 at 12:01 PM, Sasha Levin levinsasha...@gmail.com wrote:
PCI BAR probing is done in four steps:
1. Read address (and flags).
2. Mask BAR.
3. Read BAR again - Now the expected result is the size of the
On Thu, Jul 28, 2011 at 12:01:54PM +0300, Sasha Levin wrote:
...
struct mmio_mapping {
struct rb_int_node node;
- void(*kvm_mmio_callback_fn)(u64 addr, u8 *data, u32
len, u8 is_write);
+ void(*kvm_mmio_callback_fn)(u64 addr, u8
On Thu, Jul 28, 2011 at 12:01:52PM +0300, Sasha Levin wrote:
Map GSIs manually when starting the guest.
This will allow us mapping new GSIs for MSIX in the future.
Signed-off-by: Sasha Levin levinsasha...@gmail.com
---
Other than a few nits the series looks good to me, thanks Sasha!
--
To
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/bios/rom.ld.S |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Index: linux-2.6.git/tools/kvm/bios/rom.ld.S
===
--- linux-2.6.git.orig/tools/kvm/bios
Put bios code into bios.s and adjust makefile
rules accordingly. It's more natural than bios-rom.S
(which is now simply a container over real bios code).
Also improve bios deps in Makefile.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/Makefile| 29
Some kernel parameters are convenient if passed in
hex form so our options parser should handle even
such form of input.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/util/parse-options.c | 102 -
1 file changed, 82 insertions(+), 20
Noone use it anymore. Also cleanup comment on
int10 as well, int10_handler routine do all
the hard work.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/bios/bios-rom.S | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
Index: linux-2.6.git/tools/kvm/bios
Usually this might be set by loader but since
we're the loader lets allow to specify vesa
mode as well.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
---
tools/kvm/kvm-run.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
Index: linux-2.6.git/tools/kvm/kvm-run.c
Nothing serious, please review. Thanks.
Cyrill
--
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 http://vger.kernel.org/majordomo-info.html
On Tue, Jun 07, 2011 at 10:53:28PM +0300, Pekka Enberg wrote:
On Tue, 7 Jun 2011, Cyrill Gorcunov wrote:
Usually this might be set by loader but since
we're the loader lets allow to specify vesa
mode as well.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
This patch causes 'make check
On Tue, Jun 07, 2011 at 10:53:28PM +0300, Pekka Enberg wrote:
On Tue, 7 Jun 2011, Cyrill Gorcunov wrote:
Usually this might be set by loader but since
we're the loader lets allow to specify vesa
mode as well.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
This patch causes 'make check
On Wed, Jun 08, 2011 at 12:10:30AM +0400, Cyrill Gorcunov wrote:
On Tue, Jun 07, 2011 at 10:53:28PM +0300, Pekka Enberg wrote:
On Tue, 7 Jun 2011, Cyrill Gorcunov wrote:
Usually this might be set by loader but since
we're the loader lets allow to specify vesa
mode as well.
Signed-off
On Mon, Jun 06, 2011 at 11:28:56AM +0300, Avi Kivity wrote:
On 06/03/2011 10:37 PM, Pekka Enberg wrote:
This patch fixes access to 'ah' in int10_vesa() by masking the high bits.
@@ -131,7 +131,7 @@ static void int10_vesa(struct int10_args *args)
{
u8 al;
-al = args-eax;
+
On Mon, Jun 06, 2011 at 12:09:25PM +0300, Pekka Enberg wrote:
...
Type conversion will do the work but having explicit masking is
a way better I believe, at least it makes this code snippet notable.
True but the patch description is bogus as it really doesn't _fix_ anything.
On Fri, Jun 03, 2011 at 10:37:04PM +0300, Pekka Enberg wrote:
This patch fixes access to 'ah' in int10_vesa() by masking the high bits.
Cc: Ingo Molnar mi...@elte.hu
Cc: Cyrill Gorcunov gorcu...@gmail.com
Cc: John Floren j...@jfloren.net
Cc: Sasha Levin levinsasha...@gmail.com
Signed-off
On Fri, Jun 03, 2011 at 10:37:03PM +0300, Pekka Enberg wrote:
This patch cleans up the code in bios/int10.c without changing functionality.
Cc: Ingo Molnar mi...@elte.hu
Cc: Cyrill Gorcunov gorcu...@gmail.com
Cc: John Floren j...@jfloren.net
Cc: Sasha Levin levinsasha...@gmail.com
Signed
On 05/24/2011 12:37 PM, Paolo Bonzini wrote:
On 05/23/2011 01:38 PM, Ingo Molnar wrote:
Later on even this could be removed: using section
tricks we can put init functions into a section
This is not kernel space, the C library provides a way to do that with
__attribute__((constructor))...
-variable]
bios/int10.c: In function ‘int10_vesa’:
bios/int10.c:96:9: error: variable ‘ah’ set but not used
[-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors
so get rid of them.
Signed-off-by: Cyrill Gorcunov gorcu...@gmail.com
CC: Sasha Levin levinsasha...@gmail.com
On 05/22/2011 02:58 PM, Ingo Molnar wrote:
* tip-bot for Cyrill Gorcunov gorcu...@gmail.com wrote:
diff --git a/tools/perf/feature-tests.mak
b/tools/kvm/config/feature-tests.mak
similarity index 83%
copy from tools/perf/feature-tests.mak
copy to tools/kvm/config/feature-tests.mak
Btw
On 05/22/2011 03:00 PM, Ingo Molnar wrote:
* Ingo Molnar mi...@elte.hu wrote:
* tip-bot for Cyrill Gorcunov gorcu...@gmail.com wrote:
diff --git a/tools/perf/feature-tests.mak
b/tools/kvm/config/feature-tests.mak
similarity index 83%
copy from tools/perf/feature-tests.mak
copy
On 05/21/2011 12:51 PM, Sasha Levin wrote:
Currently the ioport implementation is based on a USHRT_MAX length
array of ptrs to ioport_operations.
Instead, use an interval rbtree to map the ioports to
ioport_operations.
Signed-off-by: Sasha Levin levinsasha...@gmail.com
---
...
-static
On 05/21/2011 02:55 PM, Sasha Levin wrote:
...
void ioport__register(u16 port, struct ioport_operations *ops, int count)
{
- int i;
+ struct ioport_entry *entry;
- for (i = 0; i count; i++)
- ioport_ops[port + i]= ops;
+ entry = ioport_search(ioport_tree, port);
1 - 100 of 154 matches
Mail list logo