Hi all,
This is a series for kvmtool that uses a newish kernel API to get
MMU info, which is then fed to the guest.
Currently we just make a good guess based on the PVR, but that is
potentially flakey in a few ways. The most notable is that if you don't
specify hugepages we don't boot - because t
off-by: Michael Ellerman
---
tools/kvm/include/kvm/util.h |4 +++-
tools/kvm/powerpc/include/kvm/kvm-arch.h |1 +
tools/kvm/powerpc/kvm.c |2 +-
tools/kvm/util/util.c| 13 +
tools/kvm/x86/include/kvm/kvm-arch.h |
ist of page sizes against the page size
of the memory backing guest RAM - this accounts for the unfortunate amount
of code in setup_mmu_info().
Finally we need to turn the structure as returned by the kernel into the
format expected in the device tree.
Signed-off-by: Michael Ellerman
---
tools/
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c |7 ---
tools/kvm/powerpc/cpu_info.h |2 --
tools/kvm/powerpc/kvm.c |7 ---
3 files changed, 4 insertions(+), 12 deletions(-)
diff --git a/tools/kvm/powerpc/cpu_info.c b/tools/kvm/powerpc/cpu_info.c
index
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c |3 +--
tools/kvm/powerpc/cpu_info.h |1 -
tools/kvm/powerpc/kvm.c |5 +++--
3 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/tools/kvm/powerpc/cpu_info.c b/tools/kvm/powerpc/cpu_info.c
index 82a9d4f
So we can use it on powerpc.
Signed-off-by: Michael Ellerman
---
tools/kvm/include/kvm/util.h |2 +-
tools/kvm/util/util.c| 13 +
tools/kvm/x86/kvm.c | 13 -
3 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/tools/kvm/include/kvm
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/kvm/powerpc/cpu_info.c b/tools/kvm/powerpc/cpu_info.c
index ad27451..7326f5b 100644
--- a/tools/kvm/powerpc/cpu_info.c
+++ b/tools/kvm/powerpc/cpu_info.c
We are about to add more logic to find_cpu_info(). To support this we
need to pass kvm through to it, and also restructure the return flow
so we can operate on info before it is returned.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c | 16 +++-
tools/kvm/powerpc
Matt's enter key was broken when he wrote this ;)
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools/kvm/powerpc/cpu_info.c b/tools/kvm/powerpc/cpu_info.c
index 7326f5b..586b232 100644
--- a/
Using designated initializers for structs is preferable because it
is self documenting, and more robust against changes to the structure
layout.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c | 38 +-
tools/kvm/powerpc/cpu_info.h |6
It implements essentially the same logic. The one difference is it sets
MAP_NORESERVE when using anonymous mmap, but I think that is OK.
Reword the comment about hugetblfs, we are no longer required to use
hugepages to back the guest.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/kvm.c
On Tue, 2012-07-17 at 10:33 +0100, Matt Evans wrote:
> Hi Michael,
>
> On 2012-07-17 06:00, Michael Ellerman wrote:
>
> > This is a series for kvmtool that uses a newish kernel API to get
> > MMU info, which is then fed to the guest.
> >
> > Currently we jus
On Tue, 2012-07-17 at 17:09 +0300, Pekka Enberg wrote:
> On Tue, Jul 17, 2012 at 12:33 PM, Matt Evans wrote:
> > Just had a look, all good. Thanks for tidying some old FIXMEs, especially
> > the page/segment DT props encoding grot -- and the designated inits in the
> > cpuinfo struct, whew, I hea
In uip_dhcp_get_dns() we try to open /etc/resolv.conf. If we fail to
open it we then SEGV trying to fclose() it.
Fix the code to just return directly if we can't open it.
Signed-off-by: Michael Ellerman
---
tools/kvm/net/uip/dhcp.c |3 +--
1 file changed, 1 insertion(+), 2 dele
on't split the line to improve grepability.
Signed-off-by: Michael Ellerman
---
tools/kvm/virtio/net.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tools/kvm/virtio/net.c b/tools/kvm/virtio/net.c
index 10420ae..8f3735b 100644
--- a/tools/kvm/virtio/net.c
+++ b/tools/kvm/vi
Currently the parsing of network command line parameters doesn't check
for unknown parameters.
For example "-n mod=none" will cause kvmtool to setup TAP networking.
Save users from their typos by checking for unknown parameters and bailing.
Signed-off-by: Michael Ellerman
On Sat, 2012-08-11 at 08:21 +0800, Asias He wrote:
> On Fri, Aug 10, 2012 at 1:38 PM, Michael Ellerman
> wrote:
> > Currently the parsing of network command line parameters doesn't check
> > for unknown parameters.
> >
> > For example "-n mod=none&quo
Commit 141124c deleted system.h, we really want barrier.h
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/include/kvm/barrier.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/kvm/powerpc/include/kvm/barrier.h
b/tools/kvm/powerpc/include/kvm/barrier.h
index
idea, or whether it works at all.
Signed-off-by: Michael Ellerman
---
I don't see any logic in pci_get_io_space_block() to stop it running into
the VIRTIO area? But I haven't dug any further.
---
tools/kvm/powerpc/include/kvm/kvm-arch.h |1 +
1 file changed, 1 insertion(+)
: Michael Ellerman
---
Tested on x86_64 and powerpc, and cross compiling powerpc. I don't have an
i386 box around sorry.
---
tools/kvm/Makefile |8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tools/kvm/Makefile b/tools/kvm/Makefile
index 7a9d3b6..501387b 1
The sed expression for ARCH seems to have been cribbed from the top-level
kernel Makefile, and includes lots of architectures kvmtool doesn't
support - strip it down.
Also call uname -m directly there and get rid of uname_M.
Signed-off-by: Michael Ellerman
---
tools/kvm/Makefile |
I think this code was based on an earlier version of the KVM_SET_ONE_REG
API, which at the time was in agraf's tree but not mainline?
Either way it doesn't compile as is, so fix it up.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/kvm-cpu.c |6 --
1 file changed, 4
Signed-off-by: Michael Ellerman
---
tools/kvm/builtin-run.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-run.c
index 95d35a5..569246e 100644
--- a/tools/kvm/builtin-run.c
+++ b/tools/kvm/builtin-run.c
@@ -997,6 +997,11
we have closed stdin term_cleanup() doesn't work and the users terminal
is fubared.
Set both fd's to -1 in kvm__new() to avoid the problem.
Signed-off-by: Michael Ellerman
---
tools/kvm/kvm.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/tools/kvm/kvm.c b
nd
do nothing if it is false.
Signed-off-by: Michael Ellerman
---
tools/kvm/ioeventfd.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/tools/kvm/ioeventfd.c b/tools/kvm/ioeventfd.c
index f2d5a30..9b328cd 100644
--- a/tools/kvm/ioeventfd.c
+++ b/tools/kvm/ioeven
On Mon, 2012-02-06 at 12:22 +0200, Pekka Enberg wrote:
> On Mon, 6 Feb 2012, Michael Ellerman wrote:
> > diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-run.c
> > index 95d35a5..569246e 100644
> > --- a/tools/kvm/builtin-run.c
> > +++ b/tools/kvm/builtin-
t;sets up future vcpus to have a local APIC".
So the simplest fix seems to be to enforce that ordering in the code.
Signed-off-by: Michael Ellerman
---
arch/x86/kvm/x86.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
i
On Mon, 2012-02-06 at 13:46 -0600, Scott Wood wrote:
> On 02/03/2012 04:52 PM, Anthony Liguori wrote:
> > On 02/03/2012 12:07 PM, Eric Northup wrote:
> >> On Thu, Feb 2, 2012 at 8:09 AM, Avi Kivity wrote:
> >> [...]
> >>>
> >>> Moving to syscalls avoids these problems, but introduces new ones:
> >
On Tue, 2012-02-07 at 17:38 -0200, Marcelo Tosatti wrote:
> On Tue, Feb 07, 2012 at 05:32:07PM +1100, Michael Ellerman wrote:
> > A test case which does the following:
> >
> > ioctl(vmfd, KVM_CREATE_VCPU, 0);
> > ioctl(vmfd, KVM_CREATE_IRQCHIP);
> > ioctl(cpuf
On Wed, 2012-02-08 at 21:41 +1100, Michael Ellerman wrote:
> On Tue, 2012-02-07 at 17:38 -0200, Marcelo Tosatti wrote:
> > On Tue, Feb 07, 2012 at 05:32:07PM +1100, Michael Ellerman wrote:
> > > A test case which does the following:
> > >
> > > ioctl(vmfd, KV
On Wed, 2012-02-15 at 22:21 +, Arnd Bergmann wrote:
> On Tuesday 07 February 2012, Alexander Graf wrote:
> > On 07.02.2012, at 07:58, Michael Ellerman wrote:
> >
> > > On Mon, 2012-02-06 at 13:46 -0600, Scott Wood wrote:
> > >> You're exposing a large
On Thu, 2012-02-16 at 21:28 +0200, Avi Kivity wrote:
> On 02/16/2012 03:04 AM, Michael Ellerman wrote:
> > >
> > > ioctl is good for hardware devices and stuff that you want to enumerate
> > > and/or control permissions on. For something like KVM that is really
On Mon, 2012-03-05 at 14:29 +0200, Avi Kivity wrote:
> If some vcpus are created before KVM_CREATE_IRQCHIP, then
> irqchip_in_kernel() and vcpu->arch.apic will be inconsistent, leading
> to potential NULL pointer dereferences.
>
> Fix by:
> - ensuring that no vcpus are installed when KVM_CREATE_IR
Commit 21692d1 (Beautify debug output) broke the powerpc build because
it changed the signature for kvm__dump_mem() but didn't update all callers.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/kvm-cpu.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tool
, but does work.
Note that we skip registering the cleanup routines, so we don't need to
modify them.
Signed-off-by: Michael Ellerman
---
tools/kvm/term.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/tools/kvm/term.c b/tools/kvm/term.c
index 4413450..
Add perror() calls to a couple of exit paths, to ease debugging.
There are also two places where we print "Failed starting IPC thread",
but one is really an epoll failure, so make that obvious.
Signed-off-by: Michael Ellerman
---
tools/kvm/kvm-ipc.c | 17 +
1 file c
In xics_init() we set the maximum server to kvm->nrcpus, and then set
the nr_servers using maximum server + 1.
That is off by one, in the harmless direction.
Simplify it to just set nr_servers = kvm->nrcpus.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/xics.c |5 +
Currently list always returns 0, even if there was an error. Instead
have it accumulate any errors and return that.
Signed-off-by: Michael Ellerman
---
tools/kvm/builtin-list.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tools/kvm/builtin-list.c b/tools/kvm
The kernel can handle a missing timebase-frequency property much better
than one that claims zero.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/kvm.c |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/kvm/powerpc/kvm.c b/tools/kvm/powerpc/kvm.c
index
We should hard-code less of this stuff, but for now this works.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/cpu_info.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/tools/kvm/powerpc/cpu_info.c b/tools/kvm/powerpc/cpu_info.c
index 11ca14e..a9dfe39 100644
--- a
On Tue, 2013-04-23 at 10:57 -0400, Sasha Levin wrote:
> We don't really need 64MB of RAM to boot, it's a nice default if we don't
> have anything else - but it's not actually required for anything:
Nice, I am carrying something similar locally so I can boot in 4K :)
cheers
--
To unsubscribe from
On Wed, 2013-05-01 at 16:50 +0100, Will Deacon wrote:
> Hi guys,
>
> This RFC series does a number of things, but the main goal is to allow
> re-use of the emulation code under hw/ on architectures other than x86.
> It also comes about after discussions concerning virtio-console for
> earlyprintk:
On Sun, 2013-06-23 at 21:23 -0400, Sasha Levin wrote:
> Commit "kvm tools: virtio: remove hardcoded assumptions
> about guest page size" has introduced a bug that prevented
> guests with more than 4gb of ram from booting.
>
> The issue is that 'pfn' is a 32bit integer, so when multiplying
> it by
On Tue, Jul 16, 2013 at 11:09:30AM +0800, Tiejun Chen wrote:
> The SOFT_DISABLE_INTS seems an odd name for something that updates the
> software state to be consistent with interrupts being hard disabled, so
> rename SOFT_DISABLE_INTS with RECONCILE_IRQ_STATE to avoid this confusion.
Yes!
I have
and x86-64.
Signed-off-by: Michael Ellerman
---
tools/kvm/Makefile | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/kvm/Makefile b/tools/kvm/Makefile
index 8f996cd..b614aab 100644
--- a/tools/kvm/Makefile
+++ b/tools/kvm/Makefile
@@ -150,6 +150,7 @@ ifeq ($(ARCH
oot, and end up spinning in the guest.
We can't implement reboot properly, ie. causing a reboot, but it's still
preferable to implement it as halt rather than not implementing it at
all.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/spapr_rtas.c | 14 ++
1 fi
Some powernv systems include a hwrng. Guests can access it via the
H_RANDOM hcall.
We add a real mode implementation of H_RANDOM when a hwrng is found.
Userspace can detect the presence of the hwrng by quering the
KVM_CAP_PPC_HWRNG capability.
Signed-off-by: Michael Ellerman
---
arch/powerpc
l
the dieharder tests.
Signed-off-by: Michael Ellerman
---
arch/powerpc/Kconfig| 3 +
arch/powerpc/include/asm/archrandom.h | 32 +
arch/powerpc/include/asm/machdep.h | 4 ++
arch/powerpc/platforms/powernv/Kconfig | 1 +
arch/powerpc/platforms/powernv/Makefile
e, but isn't
in this case.
We'd also like to add another platform, powerpc, which will just make it
worse. So clean it up in preparation.
Signed-off-by: Michael Ellerman
---
scripts/kvm/kvm_stat | 44 +---
1 file changed, 29 insertions(+), 15 del
Unfortunately ioctl numbers are platform specific, so abstract them out
of the code so they can be overridden. As it happens x86 and s390 share
the same values, so nothing needs to change yet.
Signed-off-by: Michael Ellerman
---
scripts/kvm/kvm_stat | 12 +---
1 file changed, 9
the filter parameter is
'reason'.
So invert the way we setup filters, define it by default for the generic
tracepoint 'kvm_userspace_exit', and let x86 override it. Doing it this
way will also work for powerpc when we add it.
Signed-off-by: Michael Ellerman
---
scripts/
, the online file will not be
present at all.
Signed-off-by: Michael Ellerman
---
scripts/kvm/kvm_stat | 24 ++--
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
index d7e97e7..2a788bc 100755
--- a/scripts/kvm/kvm_stat
are incorrect.
As far as I can tell this does not matter in practice because s390 does
not define a kvm_exit trace point.
While we're there, fix the whitespace to match the rest of the file.
Signed-off-by: Michael Ellerman
---
scripts/kvm/kvm_stat
Add support for powerpc platforms. We use uname -m, which allows us to
detect ppc, ppc64 and ppc64le/el.
Signed-off-by: Michael Ellerman
---
scripts/kvm/kvm_stat | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/scripts/kvm/kvm_stat b/scripts/kvm/kvm_stat
On Tue, 2014-06-17 at 10:27 +0200, Alexander Graf wrote:
> On 17.06.14 09:54, Michael Ellerman wrote:
> > Add support for powerpc platforms. We use uname -m, which allows us to
> > detect ppc, ppc64 and ppc64le/el.
> >
> > Signed-off-by: Michael Ellerman
>
> Co
On Wed, 2014-06-18 at 02:59 +0200, Alexander Graf wrote:
> On 18.06.14 02:50, Michael Ellerman wrote:
> > On Tue, 2014-06-17 at 10:27 +0200, Alexander Graf wrote:
> >> On 17.06.14 09:54, Michael Ellerman wrote:
> >>> Add support for powerpc platforms. We us
Commit e58e263 "PPC, KVM, CMA: use general CMA reserved area management
framework" in next-20140624 removed arch/powerpc/kvm/book3s_hv_cma.c but
neglected to update the Makefile, thus breaking the build.
Signed-off-by: Michael Ellerman
---
Hi Andrew,
This is in your akpm-curr
On Tue, 2014-06-24 at 15:51 +0900, Joonsoo Kim wrote:
> On Tue, Jun 24, 2014 at 04:36:47PM +1000, Michael Ellerman wrote:
> > Commit e58e263 "PPC, KVM, CMA: use general CMA reserved area management
> > framework" in next-20140624 removed arch/powerpc/kvm/book3s_hv_cma.c bu
, particularly for the MMCRx/SIxR/SDAR fields, and
might have helped us notice the recent double restore bug we had in this
code.
Signed-off-by: Michael Ellerman
---
This is on top of "powerpc/kvm: Remove redundant save of SIER AND MMCR2".
arch/powerpc/kernel/asm-offsets.c
On Thu, 2014-07-10 at 12:16 +0200, Alexander Graf wrote:
> On 10.07.14 11:34, Michael Ellerman wrote:
> > We have two arrays in kvm_host_state that contain register values for
> > the PMU. Currently we only create an asm-offsets symbol for the base of
> > the arrays, and do th
On Fri, 2014-10-31 at 16:36 +0100, Paolo Bonzini wrote:
> Thanks, applied the series at last.
Thanks.
cheers
--
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
"closest" device, avoiding inter-chip
memory traffic.
Signed-off-by: Guo Chao
Signed-off-by: Michael Ellerman
---
drivers/char/hw_random/Kconfig | 13 ++
drivers/char/hw_random/Makefile | 1 +
drivers/char/hw_random/powernv-rng.c | 81 +++
On Thu, Sep 26, 2013 at 06:01:27PM +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2013-09-26 at 16:31 +1000, Michael Ellerman wrote:
>
> > + pr_info("registered powernv hwrng.\n");
>
> First letter of a line should get a capital :-) Also since
> it's
On Thu, Sep 26, 2013 at 11:06:59AM +0200, Paolo Bonzini wrote:
> Il 26/09/2013 08:31, Michael Ellerman ha scritto:
> > Some powernv systems include a hwrng. Guests can access it via the
> > H_RANDOM hcall.
>
> Is there any reason to do this in the kernel?
It's
On Fri, Sep 27, 2013 at 07:45:45PM +0530, Anshuman Khandual wrote:
> On 09/26/2013 12:01 PM, Michael Ellerman wrote:
> > +int powernv_hwrng_present(void)
> > +{
> > + return __raw_get_cpu_var(powernv_rng) != NULL;
> > +}
> > +
> > static unsigned
On Wed, 2013-10-02 at 13:02 +0300, Gleb Natapov wrote:
> On Wed, Oct 02, 2013 at 11:50:50AM +0200, Alexander Graf wrote:
> >
> > On 02.10.2013, at 11:11, Alexander Graf wrote:
> >
> > So how do you solve live migration between a kernel that has this patch and
> > one that doesn't?
> >
> Yes, I
char() & h_get_term_char() of
kvm->cfg.active_console but needs to be vcpu->kvm->cfg.active_console.
That commit also missed updates to term_putc() & term_getc() in
spapr_rtas.c, and I'm guessing that we need similar checks of
active_console in rtas_put_term_char() & rtas
Finally drop irq__exit(), it does nothing and is never called.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/irq.c | 19 ---
tools/kvm/powerpc/kvm-cpu.c |3 ---
tools/kvm/powerpc/xics.c| 38 +++---
tools/kvm/powerpc/xics.h
_fdt() out of kvm__arch_setup_firmware() and make it a
firmware_init() call of its own.
With this patch I am able to boot guests again on HV KVM.
Signed-off-by: Michael Ellerman
---
tools/kvm/powerpc/kvm.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/kvm/powerpc/kvm.c b/tool
On Fri, 2012-10-05 at 09:30 +0300, Pekka Enberg wrote:
> Applied all three patches, thanks Michael!
Thanks Pekka.
cheers
--
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/majord
Signed-off-by: Michael Ellerman
---
MAINTAINERS |1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index e73060f..32dc107 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4244,6 +4244,7 @@ KERNEL VIRTUAL MACHINE (KVM) FOR POWERPC
M: Alexander Graf
L: kvm
On Thu, 2014-05-29 at 17:45 +1000, Michael Neuling wrote:
> > > +/* Values for 2nd argument to H_SET_MODE */
> > > +#define H_SET_MODE_RESOURCE_SET_CIABR1
> > > +#define H_SET_MODE_RESOURCE_SET_DAWR2
> > > +#define H_SET_MODE_RESOURCE_ADDR_TRANS_MODE3
> > > +#define H_SET_MODE_R
On Wed, 2009-08-26 at 21:15 +0530, Amit Shah wrote:
> [cc'ing some people who have made some commits in hvc_console.c]
>
> On (Wed) Aug 26 2009 [16:57:18], Amit Shah wrote:
> > On (Tue) Aug 25 2009 [11:47:20], Amit Shah wrote:
> > >
> > > Hello all,
> > >
> > > Here is a new iteration of the pat
On Thu, 2009-05-07 at 03:53 -0600, Matthew Wilcox wrote:
> On Thu, May 07, 2009 at 05:40:15PM +0800, Sheng Yang wrote:
> > It's indeed weird. Why the semantic of pci_enable_msix can be changed to
> > "enable msix, or tell me how many vector do you have"? You can simply call
> > pci_msix_table_siz
On Fri, 2009-05-08 at 09:25 +0930, Rusty Russell wrote:
> On Thu, 7 May 2009 07:49:53 pm Sheng Yang wrote:
> > On Thursday 07 May 2009 17:53:02 Matthew Wilcox wrote:
> > > Here's a good example. Let's suppose you have a driver which supports
> > > two different models of cards, one has 16 MSI-X in
On Mon, 2015-11-02 at 16:10 -0600, Suresh E. Warrier wrote:
> Hi Fengguang,
>
> I understand the problem.
>
> I will send you the URL for the powerpc git tree once my patches
> are pulled in so that we can then pass that to the robot.
The robot already knows about the powerpc next tree.
It cho
On Wed, 2015-25-11 at 23:44:49 UTC, "Suresh E. Warrier" wrote:
> smp_muxed_ipi_message_pass() invokes smp_ops->cause_ipi, which
> updates the MFFR through an ioremapped address, to cause the
> IPI. Because of this real mode callers cannot call
> smp_muxed_ipi_message_pass() for IPI messaging.
You'
Subject would be better as "powerpc/xics".
On Wed, 2015-25-11 at 23:44:50 UTC, "Suresh E. Warrier" wrote:
> Function to cause an IPI. Requires kvm_hstate.xics_phys
> to be initialized with physical address of XICS.
Please expand the change log a bit, this is a bit terse.
> diff --git a/arch/powe
On Wed, 2015-06-17 at 11:15 +0100, Will Deacon wrote:
> On Wed, Jun 17, 2015 at 10:43:48AM +0100, Andre Przywara wrote:
> > Instead of referring to the Linux header including the barrier
> > macros, copy over the rather simple implementation for the PowerPC
> > barrier instructions kvmtool uses. Th
On Wed, 2015-06-17 at 11:13 +0100, Will Deacon wrote:
> On Wed, Jun 17, 2015 at 10:43:50AM +0100, Andre Przywara wrote:
> > The powerpc code uses some PAPR hypercalls, of which we need the
> > hypercall number. Copy the macro definition parts from the kernel's
> > (private) hvcall.h file and remove
On Thu, 2015-06-18 at 15:52 +0100, Andre Przywara wrote:
> Hi,
>
> On 06/17/2015 10:43 AM, Andre Przywara wrote:
> > For converting the guest/init binary into an object file, we call
> > the linker binary, setting the endianness to big endian explicitly
> > when compiling kvmtool for powerpc.
> >
On Thu, 2015-06-18 at 16:50 +0100, Andre Przywara wrote:
> Currently we set CC unconditionally to ${CROSS_COMPILE}gcc, the same
> for LD.
> Allow people to override the compiler name by specifying it explicitly
> on the command line or via the environment.
> Beside calling a certain compiler binary
Now that we don't have the kernel header on hand, just define the
minimum set of hcall opcodes and return values we need in order to
build.
Signed-off-by: Michael Ellerman
---
powerpc/spapr.h | 25 -
1 file changed, 20 insertions(+), 5 deletions(-)
diff --git a/po
We have always built kvmtool as 64-bit on powerpc, but mainly just out
of habit. There's not AFAIK any reason we *can't* build 32-bit.
So fix up a few places where we were assuming 64-bit, and drop the
Makefile logic that forces 64-bit.
Signed-off-by: Michael Ellerman
---
On Fri, 2015-06-19 at 17:21 +1000, Michael Ellerman wrote:
> Now that we don't have the kernel header on hand, just define the
> minimum set of hcall opcodes and return values we need in order to
> build.
Hi Andre,
I hope you don't mind me jumping in here but I thought I sho
On Fri, 2015-06-19 at 17:15 +0100, Andre Przywara wrote:
>
> What works though is using xxd to convert the binary guest/init into a C
> array:
> $ xxd -i guest/init | $(CC) -x c -c - -o guest/guest_init.o
> This has the nice property of using the same compiler that generates the
> other object file
On Tue, 2015-06-23 at 11:33 +0200, Paolo Bonzini wrote:
> On 19/06/2015 09:21, Michael Ellerman wrote:
> > diff --git a/powerpc/spapr.h b/powerpc/spapr.h
> > index 0537f881c0e4..7a377d093ef4 100644
> > --- a/powerpc/spapr.h
> > +++ b/powerpc/spapr.h
> > @@ -16
_node(area->nid,
> GFP_KERNEL|__GFP_THISNODE,
> area->order);
Yeah that looks right to me.
This code enables some firmware memory calibration so I think it really does
want to get memory on the
On Wed, 2015-08-12 at 21:06 +0200, Alexander Graf wrote:
>
> On 10.08.15 17:27, Nicholas Krause wrote:
> > This fixes the wrapper functions kvm_umap_hva_hv and the function
> > kvm_unmap_hav_range_hv to return the return value of the function
> > kvm_handle_hva or kvm_handle_hva_range that they ar
On Wed, 2015-08-12 at 10:53 +0530, Bharata B Rao wrote:
> On Tue, Aug 11, 2015 at 03:48:26PM +0200, Andrea Arcangeli wrote:
> > Hello Bharata,
> >
> > On Tue, Aug 11, 2015 at 03:37:29PM +0530, Bharata B Rao wrote:
> > > May be it is a bit late to bring this up, but I needed the following fix
> > >
On Tue, 2015-09-08 at 12:09 +0530, Bharata B Rao wrote:
> On Tue, Sep 08, 2015 at 04:08:06PM +1000, Michael Ellerman wrote:
> > On Wed, 2015-08-12 at 10:53 +0530, Bharata B Rao wrote:
> > > On Tue, Aug 11, 2015 at 03:48:26PM +0200, Andrea Arcangeli wrote:
> > > > Hell
On Tue, 2015-09-08 at 17:14 +1000, Michael Ellerman wrote:
> On Tue, 2015-09-08 at 12:09 +0530, Bharata B Rao wrote:
> > On Tue, Sep 08, 2015 at 04:08:06PM +1000, Michael Ellerman wrote:
> > > Hmm, not for me. See below.
> > >
> > > What setup were you testing
On Sun, 2014-12-21 at 23:56 +0100, Alexander Graf wrote:
> On 21.12.14 15:13, Andreas Schwab wrote:
> > arch/powerpc/kvm/built-in.o: In function `kvm_no_guest':
> > arch/powerpc/kvm/book3s_hv_rmhandlers.o:(.text+0x724): undefined reference
> > to `power7_wakeup_loss'
>
> Ugh. We just removed supp
On Thu, 2015-01-15 at 09:58 +0100, Christian Borntraeger wrote:
> ACCESS_ONCE does not work reliably on non-scalar types. For
> example gcc 4.6 and 4.7 might remove the volatile tag for such
> accesses during the SRA (scalar replacement of aggregates) step
> (https://gcc.gnu.org/bugzilla/show_bug.c
.__spin_yield" [arch/powerpc/kvm/kvm.ko] undefined!
>
> Yes, this is the patch that depends on the "powerpc: Export
> __spin_yield" patch that Suresh posted to linuxppc-...@ozlabs.org and
> I acked.
>
> I think the best thing at this stage is probably for Alex to t
95 matches
Mail list logo
