Re: [Qemu-devel] [PATCHv2 2/2] MAINTAINERS: remove Blue Swirl as SPARC maintainer

2016-06-14 Thread Mark Cave-Ayland
On 14/06/16 09:33, Markus Armbruster wrote: > Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> writes: > >> Blue is no longer active in the QEMU project, so remove him from the list of >> SPARC maintainers. >> >> Signed-off-by: Mark Cave-Ayland <mark.cave

Re: [Qemu-devel] [Qemu-ppc] Determining interest in PPC e500spin, yield, and openpic patches

2016-06-14 Thread Mark Cave-Ayland
On 14/06/16 00:08, alar...@ddci.com wrote: > We've used older versions of QEMU for several years as a virtual > target for our OS. Many thanks to the community for providing this > platform. > > We've been working to get our OS running under QEMU 2.x and have > identified a few bugs in QEMU,

Re: [Qemu-devel] [PATCH] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-14 Thread Mark Cave-Ayland
On 14/06/16 12:04, Thomas Huth wrote: > Since the mac99 and g3beige PowerPC machines recently broke without > being noticed, it would be good to have a tester for "make check" > that detects such issues immediately. A simple way to test the firmware > of these machines is to use the "-prom-env"

Re: [Qemu-devel] git master build failure

2016-06-15 Thread Mark Cave-Ayland
On 15/06/16 10:48, Andrew Jones wrote: > On Wed, Jun 15, 2016 at 08:14:18AM +0100, Mark Cave-Ayland wrote: >> Hi Peter, >> >> Just to let you know that after pulling git master >> (49237b856ae58ee7955be0b959c504c51b014f20) I see the following build >&g

[Qemu-devel] [PATCHv3 1/2] MAINTAINERS: add Artyom Tarasenko as SPARC maintainer

2016-06-15 Thread Mark Cave-Ayland
to take on co-maintainership of SPARC with a focus on 64-bit architecture. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Acked-by: Artyom Tarasenko <atar4q...@gmail.com> --- MAINTAINERS |3 +++ 1 file changed, 3 insertions(+) diff --git a/MAINTAINERS b/MAINT

[Qemu-devel] git master build failure

2016-06-15 Thread Mark Cave-Ayland
Hi Peter, Just to let you know that after pulling git master (49237b856ae58ee7955be0b959c504c51b014f20) I see the following build failure here: cc -I/home/build/src/qemu/git/qemu/tcg -I/home/build/src/qemu/git/qemu/tcg/i386 -I/home/build/src/qemu/git/qemu/linux-headers

[Qemu-devel] [PATCHv3 2/2] MAINTAINERS: remove Blue Swirl as SPARC maintainer

2016-06-15 Thread Mark Cave-Ayland
Blue is no longer active in the QEMU project, so remove him from the list of SPARC maintainers. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> CC: Blue Swirl <blauwir...@gmail.com> --- MAINTAINERS |3 --- 1 file changed, 3 deletions(-) diff --git a/MAINTAINERS b

[Qemu-devel] [PATCHv3 0/2] MAINTAINERS: update entries for SPARC

2016-06-15 Thread Mark Cave-Ayland
Update MAINTAINERS for SPARC (and associated machines): add Artyom Tarasenko and remove Blue Swirl. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (2): MAINTAINERS: add Artyom Tarasenko as SPARC maintainer MAINTAINERS: remove Blue Swirl as SPARC main

Re: [Qemu-devel] git master build failure

2016-06-15 Thread Mark Cave-Ayland
On 15/06/16 12:26, Mark Cave-Ayland wrote: > On 15/06/16 11:23, Andrew Jones wrote: > >>> Is that from a clean clone of the repository? I wonder if a change to a >>> generated file isn't being picked up somewhere by "make distclean". My >>> standard bu

Re: [Qemu-devel] git master build failure

2016-06-15 Thread Mark Cave-Ayland
On 15/06/16 11:23, Andrew Jones wrote: >> Is that from a clean clone of the repository? I wonder if a change to a >> generated file isn't being picked up somewhere by "make distclean". My >> standard build script is this: > > Not a clean clone, but a clean build dir. I build in a separate build

Re: [Qemu-devel] [PATCH] MAINTAINERS: add Artyom Tarasenko as SPARC maintainer

2016-06-13 Thread Mark Cave-Ayland
On 08/06/16 17:15, Laurent Vivier wrote: > On 08/06/2016 17:02, Artyom Tarasenko wrote: >> On Wed, Jun 8, 2016 at 4:46 PM, Markus Armbruster <arm...@redhat.com> wrote: >>> Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> writes: >>> >>>>

[Qemu-devel] [PATCHv2 2/2] MAINTAINERS: remove Blue Swirl as SPARC maintainer

2016-06-13 Thread Mark Cave-Ayland
Blue is no longer active in the QEMU project, so remove him from the list of SPARC maintainers. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> CC: Blue Swirl <blauwir...@gmail.com> --- MAINTAINERS |1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b

[Qemu-devel] [PATCHv2 0/2] MAINTAINERS: update entries for SPARC

2016-06-13 Thread Mark Cave-Ayland
Update MAINTAINERS for SPARC: add Artyom Tarasenko and remove Blue Swirl. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (2): MAINTAINERS: add Artyom Tarasenko as SPARC maintainer MAINTAINERS: remove Blue Swirl as SPARC maintainer MAINTAINERS |2

[Qemu-devel] [PATCHv2 1/2] MAINTAINERS: add Artyom Tarasenko as SPARC maintainer

2016-06-13 Thread Mark Cave-Ayland
to take on co-maintainership of SPARC with a focus on 64-bit architecture. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Acked-by: Artyom Tarasenko <atar4q...@gmail.com> --- MAINTAINERS |1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS i

Re: [Qemu-devel] [PATCH 1/2] ppc: Properly tag the translation cache based on MMU mode

2016-06-06 Thread Mark Cave-Ayland
CM) | (1ULL << MSR_SF) | MSR_HVB; > hreg_compute_mem_idx(env); > env->hflags = env->msr & hflags_mask; > > > I've run through my complete set of OpenBIOS boot tests with both this and patch 2 applied on top of Cédric's recent fixes and I no longer see any regressions with g3beige/mac99 under TCG: Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> ATB, Mark.

Re: [Qemu-devel] [RFC/PATCH] ppc: Batch TLB flushes on 32-bit 6xx/7xx/7xxx in hash mode

2016-06-06 Thread Mark Cave-Ayland
== 2) || !(ctx->insns_flags & PPC_64B)) && !ctx->pr) { > gen_check_tlb_flush(ctx); > } > } > @@ -11491,6 +11501,11 @@ void gen_intermediate_code(CPUPPCState *env, struct > TranslationBlock *tb) > ctx.sf_mode = msr_is_64bit(env, env->msr); > ctx.has_cfar = !!(env->flags & POWERPC_FLAG_CFAR); > #endif > +if (env->mmu_model == POWERPC_MMU_32B || > +env->mmu_model == POWERPC_MMU_601 || > +(env->mmu_model & POWERPC_MMU_64B)) > +ctx.lazy_tlb_flush = true; > + > ctx.fpu_enabled = msr_fp; > if ((env->flags & POWERPC_FLAG_SPE) && msr_spe) > ctx.spe_enabled = msr_spe; > > After another run of the OpenBIOS tests with this patch applied on top of the previous 2 patches, I see no regressions introduced. Like Cédric I don't get the feeling that the Mac machines necessarily run faster, however the overall experience does feel smoother and more responsive. Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> ATB, Mark.

[Qemu-devel] [PATCH 2/2] dbdma: use DMA memory interface for memory accesses

2016-06-05 Thread Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/misc/macio/mac_dbdma.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/hw/misc/macio/mac_dbdma.c b/hw/misc/macio/mac_dbdma.c index 5632743..f116f9c 100644 --- a/hw/misc/macio/mac_d

Re: [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode

2016-06-05 Thread Mark Cave-Ayland
On 05/06/16 18:41, Cédric Le Goater wrote: > Hello Mark, > > On 06/03/2016 03:52 PM, Mark Cave-Ayland wrote: >> On 03/06/16 13:11, Cédric Le Goater wrote: >> >>> This is follow up to complete the serie "ppc: preparing pnv landing >>> (round 2)&

[Qemu-devel] [PATCH 1/2] macio: use DMA memory interface for non-block ATAPI transfers

2016-06-05 Thread Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/ide/macio.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index e4e567e..ad20e1b 100644 --- a/hw/ide/macio.c +++ b/hw/ide/macio.c @@ -92,7 +92,8 @@ stati

[Qemu-devel] [PATCH 0/2] macio/dbdma: use DMA memory access helpers

2016-06-05 Thread Mark Cave-Ayland
This is just a small patchset to move the macio/dbdma interfaces over from using cpu_physical_memory_read/write to dma_memory_read/write. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (2): macio: use DMA memory interface for non-block ATAPI transfers

[Qemu-devel] [PATCH] target-ppc: fixup bitrot in mmu_helper.c debug statements

2016-06-03 Thread Mark Cave-Ayland
This fixes compilation of mmu_helper.c when all of the debug #defines at the start of the file are enabled. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- target-ppc/mmu_helper.c | 38 -- 1 file changed, 24 insertions(+), 14 del

Re: [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode

2016-06-03 Thread Mark Cave-Ayland
" to the front of this patchset which will make the remaining patches bisectable for the Mac machines. With that: Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Does anyone know if Ben has any ideas as to why the MMU tlb_flush changes patch is causing such problems? ATB, Mark.

[Qemu-devel] [Bug 833658] Re: Qemu ppc does not boot Debian 3.1r8

2016-06-04 Thread Mark Cave-Ayland
FWIW the text becomes visible again if you switch QEMU to 8-bit mode rather than 32-bit mode, e.g. adding -g 800x600x8 to the command line. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/833658

[Qemu-devel] [Bug 833658] Re: Qemu ppc does not boot Debian 3.1r8

2016-06-04 Thread Mark Cave-Ayland
Just to add here: from my local tests with other older images, the text is there on the black screen but just extremely faint. My guess is that it's an endian-related bug in OpenBIOS programming the VGA registers but haven't had much time to dig into it. -- You received this bug notification

[Qemu-devel] [Bug 1588328] Re: Qemu 2.6 Solaris 9 Sparc Segmentation Fault

2016-06-04 Thread Mark Cave-Ayland
If you can verify that the media is correct and you still see problems, I'd be interested to take a look if you are able to provide me a copy of the media for debugging. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] [Qemu-ppc] [PATCH 0/3] ppc: complete the new HV mode

2016-06-06 Thread Mark Cave-Ayland
On 06/06/16 05:20, Benjamin Herrenschmidt wrote: > On Mon, 2016-06-06 at 13:55 +1000, Benjamin Herrenschmidt wrote: >> >> I'm not sure that 32-bit patch is correct. We shouldn't have to flush >> on IR/DR transitions at all, that's the whole point of the split I/D >> code. >> >> I think something

Re: [Qemu-devel] [PATCH 0/3] ppc: complete the new HV mode

2016-06-06 Thread Mark Cave-Ayland
On 06/06/16 07:30, Cedric Le Goater wrote: > On 06/06/2016 08:27 AM, Cédric Le Goater wrote: >> On 06/06/2016 12:26 AM, Mark Cave-Ayland wrote: >>> On 05/06/16 18:41, Cédric Le Goater wrote: >>> >>>> Hello Mark, >>>> >>>> On 06/03

[Qemu-devel] [PATCH 1/2] dma-helpers.c: [HACK] disable iovec truncation to nearest sector size

2016-05-27 Thread Mark Cave-Ayland
. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- dma-helpers.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/dma-helpers.c b/dma-helpers.c index b521d84..693394a 100644 --- a/dma-helpers.c +++ b/dma-helpers.c @@ -160,9 +160,11 @@ static void dma_blk_cb(void *opaqu

[Qemu-devel] [PATCH 0/2] macio: switch over to new byte-aligned DMA helpers

2016-05-27 Thread Mark Cave-Ayland
. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (2): dma-helpers.c: [HACK] disable iovec truncation to nearest sector size macio: switch over to new byte-aligned DMA helpers dma-helpers.c |2 + hw/ide/macio.c

Re: [Qemu-devel] [PATCH 0/2] macio: switch over to new byte-aligned DMA helpers

2016-05-31 Thread Mark Cave-Ayland
On 30/05/16 22:19, Aurelien Jarno wrote: > On 2016-05-27 09:48, Mark Cave-Ayland wrote: >> Here is a tidied up version of my patch to convert the macio controller over >> to >> using the new byte-aligned DMA helpers. >> >> The first patch is just a hack and tem

Re: [Qemu-devel] [PULL 03/31] hw/char: QOM'ify escc.c

2016-05-31 Thread Mark Cave-Ayland
On 27/05/16 11:06, Paolo Bonzini wrote: > From: xiaoqiang zhao > > * Drop the old SysBus init function and use instance_init > * Call qemu_chr_add_handlers in the realize callback > > Signed-off-by: xiaoqiang zhao > Message-Id:

Re: [Qemu-devel] [Qemu-ppc] [PULL 04/12] ppc: tlbie, tlbia and tlbisync are HV only

2016-05-31 Thread Mark Cave-Ayland
On 31/05/16 01:41, David Gibson wrote: > From: Benjamin Herrenschmidt > > Not that anything remotely recent supports tlbia but ... > > Signed-off-by: Benjamin Herrenschmidt > Signed-off-by: David Gibson > --- >

[Qemu-devel] [PATCH 2/2] macio: switch over to new byte-aligned DMA helpers

2016-05-27 Thread Mark Cave-Ayland
Now that the DMA helpers are byte-aligned they can be called directly from the macio routines rather than emulating byte-aligned accesses via multiple block-level accesses. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/ide/macio.c

Re: [Qemu-devel] [PATCH 2/2] macio: switch over to new byte-aligned DMA helpers

2016-05-27 Thread Mark Cave-Ayland
On 27/05/16 16:02, John Snow wrote: > On 05/27/2016 04:48 AM, Mark Cave-Ayland wrote: >> Now that the DMA helpers are byte-aligned they can be called directly from >> the macio routines rather than emulating byte-aligned accesses via multiple >> block-level accesses. >

Re: [Qemu-devel] [Qemu-ppc] [PULL 04/12] ppc: tlbie, tlbia and tlbisync are HV only

2016-06-02 Thread Mark Cave-Ayland
On 02/06/16 08:37, Cédric Le Goater wrote: > On 06/02/2016 05:17 AM, David Gibson wrote: >> On Wed, Jun 01, 2016 at 08:03:08AM +0100, Mark Cave-Ayland wrote: >>> On 01/06/16 03:15, David Gibson wrote: >>> >>>> On Tue, May 31, 2016 at 11:28:49PM +0100, Mark

Re: [Qemu-devel] [Qemu-ppc] [PULL 04/12] ppc: tlbie, tlbia and tlbisync are HV only

2016-06-02 Thread Mark Cave-Ayland
On 02/06/16 09:23, Cédric Le Goater wrote: > On 06/02/2016 09:45 AM, Mark Cave-Ayland wrote: >> On 02/06/16 08:37, Cédric Le Goater wrote: >>> On 06/02/2016 05:17 AM, David Gibson wrote: >>>> On Wed, Jun 01, 2016 at 08:03:08AM +0100, Mark Cave-Ayland wrote: >>

Re: [Qemu-devel] [Qemu-ppc] [PULL 04/12] ppc: tlbie, tlbia and tlbisync are HV only

2016-06-02 Thread Mark Cave-Ayland
On 02/06/16 09:47, Mark Cave-Ayland wrote: > On 02/06/16 09:23, Cédric Le Goater wrote: > >> On 06/02/2016 09:45 AM, Mark Cave-Ayland wrote: >>> On 02/06/16 08:37, Cédric Le Goater wrote: >>>> On 06/02/2016 05:17 AM, David Gibson wrote: >>>>> On

Re: [Qemu-devel] [Qemu-ppc] [PATCH] MAINTAINERS: Add David Gibson as ppc maintainer

2016-05-26 Thread Mark Cave-Ayland
/openrisc/ > F: tests/tcg/openrisc/ > > PowerPC > +M: David Gibson <da...@gibson.dropbear.id.au> > M: Alexander Graf <ag...@suse.de> > L: qemu-...@nongnu.org > S: Maintained Excellent news! Acked-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> ATB, Mark.

Re: [Qemu-devel] [PATCH 4/7] scsi-disk: introduce dma_readv and dma_writev

2016-06-01 Thread Mark Cave-Ayland
On 23/05/16 13:54, Paolo Bonzini wrote: > These are replacements for blk_aio_preadv and blk_aio_pwritev that allow > customization of the data path. They reuse the DMA helpers' DMAIOFunc > callback type, so that the same function can be used in either the > QEMUSGList or the bounce-buffered

Re: [Qemu-devel] [Qemu-ppc] [PULL 02/12] ppc: Use split I/D mmu modes to avoid flushes on interrupts

2016-06-01 Thread Mark Cave-Ayland
On 31/05/16 01:41, David Gibson wrote: > From: Benjamin Herrenschmidt > > We rework the way the MMU indices are calculated, providing separate > indices for I and D side based on MSR:IR and MSR:DR respectively, > and thus no longer need to flush the TLB on context

Re: [Qemu-devel] [PATCH] hw/char: QOM'ify escc.c (fix)

2016-06-01 Thread Mark Cave-Ayland
s->chn[i].clock = s->frequency / 2; > qemu_chr_add_handlers(s->chn[i].chr, serial_can_receive, >serial_receive1, serial_event, >chn[i]); > } > Thanks a lot for this patch, I can confirm that it fixes the problem under qemu-system-ppc for me: Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> ATB, Mark.

Re: [Qemu-devel] [Qemu-ppc] [PULL 02/12] ppc: Use split I/D mmu modes to avoid flushes on interrupts

2016-06-01 Thread Mark Cave-Ayland
On 02/06/16 04:15, David Gibson wrote: > On Wed, Jun 01, 2016 at 08:33:30PM +0100, Mark Cave-Ayland wrote: >> On 31/05/16 01:41, David Gibson wrote: >> >>> From: Benjamin Herrenschmidt <b...@kernel.crashing.org> >>> >>> We rework the way th

Re: [Qemu-devel] [PULL 03/31] hw/char: QOM'ify escc.c

2016-06-01 Thread Mark Cave-Ayland
On 01/06/16 04:06, xiaoqiang zhao wrote: >> 在 2016年6月1日,06:13,Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> 写道: >> >>> On 27/05/16 11:06, Paolo Bonzini wrote: >>> >>> From: xiaoqiang zhao <zxq_yx_...@163.com> >>> >>>

Re: [Qemu-devel] [Qemu-ppc] [PULL 04/12] ppc: tlbie, tlbia and tlbisync are HV only

2016-06-01 Thread Mark Cave-Ayland
On 01/06/16 03:15, David Gibson wrote: > On Tue, May 31, 2016 at 11:28:49PM +0100, Mark Cave-Ayland wrote: >> On 31/05/16 01:41, David Gibson wrote: >> >>> From: Benjamin Herrenschmidt <b...@kernel.crashing.org> >>> >>> Not that anything remotely

Re: [Qemu-devel] [PATCH] hw/char: QOM'ify escc.c (fix)

2016-06-01 Thread Mark Cave-Ayland
On 01/06/16 08:58, xiaoqiang zhao wrote: > The previous commit e7c9136977cb99c6eb52c9139f7b8d8b5fa87db9 > (hw/char: QOM'ify escc.c) cause qemu-system-ppc/ppc64 > OpenBIOS to freeze on startup, this commit fix it. > > Signed-off-by: xiaoqiang zhao > --- > hw/char/escc.c | 12

Re: [Qemu-devel] [PATCH v2 0/3] Second try at fixing sparc register allocation

2016-06-22 Thread Mark Cave-Ayland
On 22/06/16 07:52, Richard Henderson wrote: Attempting to fix the problem reported by Mark re i686 vs sparc64. Unsurprisingly, the problems tend to revolve around the 6 operand opcodes like sub2 or qemu_st64, where we use all, or all but one register. r~ Richard Henderson (3): tcg: Fix

[Qemu-devel] [Bug 1588328] Re: Qemu 2.6 Solaris 9 Sparc Segmentation Fault

2016-06-14 Thread Mark Cave-Ayland
Thanks for the test case. It appears that this is a regression that occurred somewhere between 2.5 and 2.6 - bisecting now. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1588328 Title: Qemu 2.6

Re: [Qemu-devel] [PULL 5/8] target-sparc: Use global registers for the register window

2016-06-14 Thread Mark Cave-Ayland
On 23/02/16 18:33, Richard Henderson wrote: > Via indirection off cpu_regwptr. > > Tested-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> > Signed-off-by: Richard Henderson <r...@twiddle.net> > --- > target-sparc/translate.c | 57 >

Re: [Qemu-devel] [PULL 5/8] target-sparc: Use global registers for the register window

2016-06-16 Thread Mark Cave-Ayland
On 16/06/16 21:26, Richard Henderson wrote: > On 06/14/2016 02:52 PM, Mark Cave-Ayland wrote: >> Following up the bug report at >> https://bugs.launchpad.net/qemu/+bug/1588328, I bisected the regression >> down to this particular commit. I can't see anything obviou

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 10:55, Paolo Bonzini wrote: > The implementation of SATN/STOP is completely busted. The idea > would be that the next DMA read is for a SCSI message and after > that the adapter would transition to the command phase. > > The recent fix to SATN/STOP broke migration, which is one more

Re: [Qemu-devel] [Bug 1588328] Re: Qemu 2.6 Solaris 9 Sparc Segmentation Fault

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 12:42, Artyom Tarasenko wrote: > Can you guys check if the problem persists when qemu is launched with > the -singlestep option? > I think it's in general a good idea always check TCG-related problems > with -singlestep , because it helps to find out whether a bug is in > the

Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 12:36, Artyom Tarasenko wrote: > Hi Mark, > > On Fri, Jun 17, 2016 at 1:27 PM, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> On 17/06/16 07:07, David Gibson wrote: >> >>> On Wed, Jun 15, 2016 at 01:10:18PM +1000, David Gibson wr

Re: [Qemu-devel] git master build failure

2016-06-17 Thread Mark Cave-Ayland
On 15/06/16 13:35, Laszlo Ersek wrote: > On 06/15/16 12:08, Mark Cave-Ayland wrote: >> On 15/06/16 10:48, Andrew Jones wrote: >> >>> On Wed, Jun 15, 2016 at 08:14:18AM +0100, Mark Cave-Ayland wrote: >>>> Hi Peter, >>>> >>

Re: [Qemu-devel] [RFC PATCH 1/1] OpenBIOS: Switch over to official OpenBIOS git repo

2016-06-21 Thread Mark Cave-Ayland
On 21/06/16 11:28, Stefan Hajnoczi wrote: > On Tue, Jun 21, 2016 at 01:40:42AM -0400, Jeff Cody wrote: >> This update should preserve git history, and allow seamless switching >> over to the official openbios git repo, rather than pulling from the >> svn mirror. All prior history from the svn

Re: [Qemu-devel] [Qemu-ppc] [PATCH 01/10] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-21 Thread Mark Cave-Ayland
On 20/06/16 10:32, Benjamin Herrenschmidt wrote: > On Mon, 2016-06-20 at 18:02 +1000, Benjamin Herrenschmidt wrote: >> On Mon, 2016-06-20 at 17:08 +1000, Benjamin Herrenschmidt wrote: >>> >>> That fixed, it dies elsewhere in something related to page faults, >>> still digging. >>> >> Next

[Qemu-devel] [PULL] qemu-sparc update

2016-06-20 Thread Mark Cave-Ayland
) qemu-sparc update Mark Cave-Ayland (2): MAINTAINERS: add Artyom Tarasenko as SPARC maintainer MAINTAINERS: remove Blue Swirl as SPARC maintainer MAINTAINERS |2 +- 1 file changed, 1 insertion

Re: [Qemu-devel] [PATCH] cputlb: don't cpu_abort() if guest tries to execute outside RAM or RAM

2016-06-20 Thread Mark Cave-Ayland
On 20/06/16 18:07, Peter Maydell wrote: > In get_page_addr_code(), if the guest program counter turns out not to > be in ROM or RAM, we can't handle executing from it, and we call > cpu_abort(). This results in the message > qemu: fatal: Trying to execute code outside RAM or ROM at 0x0800 >

Re: [Qemu-devel] [PATCH 0/2] tcg: Fix i686 booting sparc64 openbios

2016-06-20 Thread Mark Cave-Ayland
On 18/06/16 06:03, Richard Henderson wrote: > There's a minor typo here that affects dumping of 64-bit > registers on 32-bit hosts. Kind of embarrasing that this > hasn't been seen previously. > > The main change takes care of cases wherein there's overlap > between the indirect base register

Re: [Qemu-devel] [PULL 5/8] target-sparc: Use global registers for the register window

2016-06-24 Thread Mark Cave-Ayland
On 24/06/16 07:36, Paolo Bonzini wrote: On 24/06/2016 05:57, Richard Henderson wrote: Whatever happens, it happens after 10GB of logs, which is simply too much to sift through. I've tried to narrow it down, but the lack of a hardware tlb refill means that we get hundreds of thousands of Data

Re: [Qemu-devel] [PATCH v3 0/9] Third try at fixing sparc register allocation

2016-06-24 Thread Mark Cave-Ayland
On 24/06/16 04:48, Richard Henderson wrote: I was unhappy about the complexity of the second try. Better to convert to normal temps, allowing in rare occasions, spilling the "globals" to the stack in order to satisfy register allocation. I can no longer provoke an allocation failure on i686.

Re: [Qemu-devel] [PATCH] target-sparc: fix register corruption in ldstub if there is no write permission

2016-06-24 Thread Mark Cave-Ayland
you beat me to it - I can confirm that this fixes the issue here for me. Whilst testing I noticed another regression under qemu-system-sparc, however bisection reveals that this isn't caused by a SPARC-specific patch (and can be followed up separately) so: Tested-by: Mark Cave-Ayland <mark.cav

Re: [Qemu-devel] [PULL 17/28] hw/ptimer: Perform counter wrap around if timer already expired

2016-06-24 Thread Mark Cave-Ayland
On 06/06/16 15:47, Peter Maydell wrote: From: Dmitry Osipenko ptimer_get_count() might be called while QEMU timer already been expired. In that case ptimer would return counter = 0, which might be undesirable in case of polled timer. Do counter wrap around for periodic timer

Re: [Qemu-devel] [PATCH] target-sparc: fix register corruption in ldstub if there is no write permission

2016-06-24 Thread Mark Cave-Ayland
On 24/06/16 16:58, Richard Henderson wrote: On 06/24/2016 05:34 AM, Artyom Tarasenko wrote: Signed-off-by: Artyom Tarasenko --- target-sparc/translate.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/target-sparc/translate.c

Re: [Qemu-devel] [PATCH] target-sparc: fix register corruption in ldstub if there is no write permission

2016-06-24 Thread Mark Cave-Ayland
On 24/06/16 17:01, Artyom Tarasenko wrote: On Fri, Jun 24, 2016 at 5:51 PM, Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> wrote: On 24/06/16 13:34, Artyom Tarasenko wrote: Signed-off-by: Artyom Tarasenko <atar4q...@gmail.com> --- target-sparc/translate.c | 5 - 1 fil

[Qemu-devel] [PATCH] target-sparc: fix register corruption in ldstub if there is no write permission

2016-06-24 Thread Mark Cave-Ayland
From: Artyom Tarasenko <atar4q...@gmail.com> Signed-off-by: Artyom Tarasenko <atar4q...@gmail.com> Reviewed-by: Richard Henderson <r...@twiddle.net> Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- target-sparc/translate.c |5 - 1 file cha

[Qemu-devel] [PULL] qemu-sparc update

2016-06-24 Thread Mark Cave-Ayland
Hi Peter, This request just contains Artyom's fix for the Solaris 9 regression introduced by the previous register window changes. Please pull. ATB, Mark. The following changes since commit a01aef5d2f96c334d048f43f0d3573a1152b37ca: Merge remote-tracking branch

[Qemu-devel] [Bug 1588328] Re: Qemu 2.6 Solaris 9 Sparc Segmentation Fault

2016-06-24 Thread Mark Cave-Ayland
Artyom has located the regression and posted a patch here: https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg07226.html. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1588328 Title: Qemu

Re: [Qemu-devel] [PULL 17/28] hw/ptimer: Perform counter wrap around if timer already expired

2016-06-24 Thread Mark Cave-Ayland
On 24/06/16 19:19, Dmitry Osipenko wrote: On 24.06.2016 19:02, Peter Maydell wrote: On 24 June 2016 at 16:58, Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> wrote: On 06/06/16 15:47, Peter Maydell wrote: From: Dmitry Osipenko <dig...@gmail.com> ptimer_get_count() might be

Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 13:57, Artyom Tarasenko wrote: > On Fri, Jun 17, 2016 at 2:44 PM, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> On 17/06/16 12:36, Artyom Tarasenko wrote: >> >>> Hi Mark, >>> >>> On Fri, Jun 17, 2016 at 1:27 PM, Mark Ca

Re: [Qemu-devel] [PATCH v2] scsi: esp: remove handling of SATN/STOP

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 14:35, Paolo Bonzini wrote: > On 17/06/2016 15:13, Mark Cave-Ayland wrote: >> Unforunately this causes regressions on a few of my SPARC32 images: >> NextStep, Solaris 1.1.2, NetBSD and Debian Etch all hang whilst >> enumerating the SCSI bus with this patch a

Re: [Qemu-devel] [PATCH 1/3] sun4u: split out NPT and INT_DIS into separate CPUTimer fields

2016-01-08 Thread Mark Cave-Ayland
On 08/01/16 14:55, Peter Maydell wrote: > On 8 January 2016 at 14:34, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> I'm not particularly worried about sun4u for the moment as there are >> already other reasons why migration would fail, e.g. no >> VMSt

Re: [Qemu-devel] [PATCH v2 0/8] target-sparc: Update to use VMStateDescription

2016-01-15 Thread Mark Cave-Ayland
Hi Peter, Thanks for doing this! It's one of those things that has been on my list of things to do but never seems to get to the the top... I've just run through some SPARC32 tests with this patchset applied - testing a savevm/loadvm pair pre-patch and post-patch, and then a few random tests w

Re: [Qemu-devel] [PATCH 4/4] target-ppc: ensure we include the decrementer value during migration

2016-01-15 Thread Mark Cave-Ayland
On 12/01/16 02:44, David Gibson wrote: >>> In other words, isn't this just skipping the decrementer interrupts at >>> the qemu level rather than the guest level? >>> >>> It seems that instead we should be reconstructing the decrementer on >>> the destination based on an offset from the timebase.

Re: [Qemu-devel] [PATCH v2 0/8] target-sparc: Update to use VMStateDescription

2016-01-15 Thread Mark Cave-Ayland
On 15/01/16 17:07, Peter Maydell wrote: > On 15 January 2016 at 17:03, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> I've just run through some SPARC32 tests with this patchset applied - >> testing a savevm/loadvm pair pre-patch and post-patch, and th

Re: [Qemu-devel] [Qemu-ppc] [PATCH 2/4] target-ppc: use cpu_write_xer() helper in cpu_post_load

2016-01-18 Thread Mark Cave-Ayland
On 18/01/16 03:12, David Gibson wrote: > On Fri, Jan 08, 2016 at 01:25:32PM +1100, Alexey Kardashevskiy wrote: >> On 01/07/2016 05:22 AM, Mark Cave-Ayland wrote: >>> Otherwise some internal xer variables fail to get set post-migration. >>> >>> Signed-off

[Qemu-devel] [PULL] qemu-sparc update

2016-01-16 Thread Mark Cave-Ayland
Hi Peter, This is simply your VMStateDescription patchset for SPARC with appropriate SoBs added. Please pull. ATB, Mark. The following changes since commit 5a57acb66f19ee52723aa05b8afbbc41c3e9ec99: Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20160115' into staging

Re: [Qemu-devel] [PATCH 2/4] macio: add dma_active to VMStateDescription

2016-01-14 Thread Mark Cave-Ayland
On 14/01/16 16:51, John Snow wrote: > We can revisit this once the solution for the CPU interrupt status is > nailed down :) Sure :) > Broadly, you are right that this board has been pretty broken for a long > time, but it appears to be at least semi-functional in 2.5, so it might > be time to

Re: [Qemu-devel] [PATCH] hw/ptimer: Don't wrap around counter for expired timer that uses tick handler

2016-06-25 Thread Mark Cave-Ayland
dler setup, assuming it is IRQ handler. This fixes regression introduced by the commit 5a50307 ("hw/ptimer: Perform counter wrap around if timer already expired") on SPARC emulated machine as reported by Mark Cave-Ayland. Signed-off-by: Dmitry Osipenko <dig...@gmail.com> --- hw/cor

Re: [Qemu-devel] [PATCH v2] hw/ptimer: Don't wrap around counter for expired timer that uses tick handler

2016-06-25 Thread Mark Cave-Ayland
dler setup, assuming it drives timer IRQ. This fixes regression introduced by the commit 5a50307 ("hw/ptimer: Perform counter wrap around if timer already expired") on SPARC emulated machine as reported by Mark Cave-Ayland. Signed-off-by: Dmitry Osipenko <dig...@gmail.com>

Re: [Qemu-devel] [PULL 07/17] ppc: Fix rfi/rfid/hrfi/... emulation

2016-06-27 Thread Mark Cave-Ayland
On 27/06/16 05:42, Benjamin Herrenschmidt wrote: > On Thu, 2016-06-23 at 15:48 +1000, David Gibson wrote: >> From: Benjamin Herrenschmidt >> >> This reworks emulation of the various "rfi" variants. I removed >> some masking bits that I couldn't make sense of, the only

Re: [Qemu-devel] [RFC PATCH 1/1] OpenBIOS: Switch over to official OpenBIOS git repo

2016-06-27 Thread Mark Cave-Ayland
On 21/06/16 14:48, Mark Cave-Ayland wrote: > On 21/06/16 11:28, Stefan Hajnoczi wrote: > >> On Tue, Jun 21, 2016 at 01:40:42AM -0400, Jeff Cody wrote: >>> This update should preserve git history, and allow seamless switching >>> over to the official openbios git

Re: [Qemu-devel] [PATCH v2] ppc / sparc: Add a tester for checking whether OpenBIOS runs successfully

2016-06-17 Thread Mark Cave-Ayland
On 17/06/16 07:07, David Gibson wrote: > On Wed, Jun 15, 2016 at 01:10:18PM +1000, David Gibson wrote: >> On Tue, Jun 14, 2016 at 03:57:56PM +0200, Thomas Huth wrote: >>> Since the mac99 and g3beige PowerPC machines recently broke without >>> being noticed, it would be good to have a tester for

[Qemu-devel] [PULL] Update OpenBIOS images

2016-02-04 Thread Mark Cave-Ayland
11:17:44 +) Update OpenBIOS images Mark Cave-Ayland (1): Update OpenBIOS images pc-bios/openbios-ppc | Bin 746588 -> 750684 bytes pc-bios/openb

Re: [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups

2016-02-06 Thread Mark Cave-Ayland
On 23/01/16 20:39, Hervé Poussineau wrote: > Hi, > > This patchset cleans up a little bit the Apple CUDA emulation: > - correctly reject unknown commands > - correctly reject commands with wrong parameters > - support changing the frequency of auto-polling > - support changing device list probed

Re: [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups

2016-02-07 Thread Mark Cave-Ayland
On 06/02/16 14:30, Hervé Poussineau wrote: > Hi Mark, > > Le 06/02/2016 14:32, Mark Cave-Ayland a écrit : >> On 23/01/16 20:39, Hervé Poussineau wrote: >> >>> Hi, >>> >>> This patchset cleans up a little bit the Apple CUDA emulation: >>> -

[Qemu-devel] Serial telnet console drops incoming characters on git master

2016-02-07 Thread Mark Cave-Ayland
Hi Daniel, I found another issue where the latest set of console updates appear to break -serial telnet on qemu-system-sparc. What I see is that while incoming serial telnet appears fine, outgoing serial telnet traffic caused by typing over the serial connection results in every other character

Re: [Qemu-devel] [PATCH 04/13] cuda: port SET_AUTO_RATE command to new framework

2016-02-07 Thread Mark Cave-Ayland
On 23/01/16 20:40, Hervé Poussineau wrote: > Take requested autopoll rate into account > > Signed-off-by: Hervé Poussineau > --- > hw/misc/macio/cuda.c | 31 +++ > hw/ppc/mac.h | 1 + > 2 files changed, 28 insertions(+), 4 deletions(-)

Re: [Qemu-devel] [PATCH 05/13] cuda: port SET_DEVICE_LIST command to new framework

2016-02-07 Thread Mark Cave-Ayland
On 23/01/16 20:40, Hervé Poussineau wrote: > Take device list mask into account when polling ADB devices. > > Signed-off-by: Hervé Poussineau > --- > hw/input/adb.c | 18 ++ > hw/misc/macio/cuda.c | 17 +++-- > hw/ppc/mac.h |

Re: [Qemu-devel] Strange monitor/stdout issue on qemu-system-sparc/qemu-system-ppc

2016-02-08 Thread Mark Cave-Ayland
On 08/02/16 09:08, Paolo Bonzini wrote: > On 31/01/2016 16:54, Mark Cave-Ayland wrote: >> Aha! A quick test here shows that the patch fixes the serial port >> appearing on stdout and entering the monitor, but I still see the >> multiple echo problem in the GTK GUI.

Re: [Qemu-devel] [PATCH] char: fix repeated registration of tcp chardev I/O handlers

2016-02-08 Thread Mark Cave-Ayland
On 08/02/16 13:55, Daniel P. Berrange wrote: > In previous commit: > > commit f2001a7e0555b66d6db25a3ff1801540814045bb > Author: Daniel P. Berrange > Date: Tue Jan 19 11:14:30 2016 + > > char: don't assume telnet initialization will not block > > The code

Re: [Qemu-devel] Migrating decrementer

2016-02-02 Thread Mark Cave-Ayland
On 01/02/16 00:52, David Gibson wrote: >> Thanks for more pointers - I think I'm slowly getting there. My current >> thoughts are that the basic migration algorithm is doing the right thing >> in that it works out the number of host ticks different between source >> and destination. > > Sorry,

[Qemu-devel] [PATCH 1/3] ppc: fix timebase adjustment during migration

2016-01-31 Thread Mark Cave-Ayland
no timebase adjustment ever takes place. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/ppc/ppc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ppc/ppc.c b/hw/ppc/ppc.c index ce90b09..19f4570 100644 --- a/hw/ppc/ppc.c +++ b/hw/ppc/ppc.c @@

[Qemu-devel] [PATCH 3/3] ppc: include timebase in migration stream for g3beige/mac99 machines

2016-01-31 Thread Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/ppc/mac_newworld.c |4 hw/ppc/mac_oldworld.c |4 2 files changed, 8 insertions(+) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c index f95086b..3283f1d 100644 --- a/hw/ppc/mac_newworld.c ++

[Qemu-devel] [PATCH 0/3] ppc: add timebase migration support to Mac machines

2016-01-31 Thread Mark Cave-Ayland
to test this on KVM-enabled hardware, however it should preserve the existing behaviour, barring the bugfix in patch 1. Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> Mark Cave-Ayland (3): ppc: fix timebase adjustment during migration ppc: add support for timebase migration

[Qemu-devel] [PATCH 2/3] ppc: add support for timebase migration on non-PPC hosts

2016-01-31 Thread Mark Cave-Ayland
This patch provides support for migration of the PPC guest timebase on non-PPC host architectures (i.e those using QEMU's virtual emulated timebase). Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> --- hw/ppc/ppc.c | 33 +++-- 1 file chang

Re: [Qemu-devel] [PATCH 3/3] ppc: include timebase in migration stream for g3beige/mac99 machines

2016-01-31 Thread Mark Cave-Ayland
On 31/01/16 19:58, Peter Maydell wrote: > On 31 January 2016 at 19:19, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk> >> --- >> hw/ppc/mac_newworld.c |4 >> hw/

Re: [Qemu-devel] Strange monitor/stdout issue on qemu-system-sparc/qemu-system-ppc

2016-02-01 Thread Mark Cave-Ayland
On 31/01/16 21:57, Paolo Bonzini wrote: > On 31/01/2016 18:54, Peter Maydell wrote: >> On 31 January 2016 at 17:19, Paolo Bonzini <pbonz...@redhat.com> wrote: >>> On 31/01/2016 16:54, Mark Cave-Ayland wrote: >>>> I also notice that with the above c

Re: [Qemu-devel] Strange monitor/stdout issue on qemu-system-sparc/qemu-system-ppc

2016-01-31 Thread Mark Cave-Ayland
On 31/01/16 15:34, Peter Maydell wrote: > On 31 January 2016 at 15:19, Mark Cave-Ayland > <mark.cave-ayl...@ilande.co.uk> wrote: >> Hi Daniel, >> >> Commit d0d7708ba29cbcc343364a46bff981e0ff88366f "qemu-char: add logfile >> facility to all chardev

[Qemu-devel] Strange monitor/stdout issue on qemu-system-sparc/qemu-system-ppc

2016-01-31 Thread Mark Cave-Ayland
Hi Daniel, Commit d0d7708ba29cbcc343364a46bff981e0ff88366f "qemu-char: add logfile facility to all chardev backends" appears to be causing problems with the monitor and stdin/stdout on both qemu-system-sparc/qemu-system-ppc here. On current git master I see the following behaviour changes when

Re: [Qemu-devel] Migrating decrementer

2016-02-23 Thread Mark Cave-Ayland
On 03/02/16 04:59, David Gibson wrote: >> Going back to your earlier email you suggested that the host timebase is >> always continuously running, even when the guest is paused. But then >> resuming the guest then the timebase must jump in the guest regardless? >> >> If this is the case then this

<    2   3   4   5   6   7   8   9   10   11   >