| Message ID | 1359293537-8251-8-git-send-email-afaerber@suse.de |
|---|---|
| State | New |
| Headers | show |
Blue, Am 27.01.2013 14:32, schrieb Andreas Färber: > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > target-sparc/ldst_helper.c | 2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c > index cf1bddf..7decd66 100644 > --- a/target-sparc/ldst_helper.c > +++ b/target-sparc/ldst_helper.c > @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, > DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", > oldreg, env->lsu); > #ifdef DEBUG_MMU > - dump_mmu(stdout, fprintf, env1); > + dump_mmu(stdout, fprintf, env); > #endif > tlb_flush(env, 1); > } This patch was not applied to master. Was that an oversight or is something wrong with it? (Preparing a v2 series for 1.5.) Andreas
On Thu, Feb 14, 2013 at 3:46 PM, Andreas Färber <afaerber@suse.de> wrote: > Blue, > > Am 27.01.2013 14:32, schrieb Andreas Färber: >> Signed-off-by: Andreas Färber <afaerber@suse.de> >> --- >> target-sparc/ldst_helper.c | 2 +- >> 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) >> >> diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c >> index cf1bddf..7decd66 100644 >> --- a/target-sparc/ldst_helper.c >> +++ b/target-sparc/ldst_helper.c >> @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, >> DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", >> oldreg, env->lsu); >> #ifdef DEBUG_MMU >> - dump_mmu(stdout, fprintf, env1); >> + dump_mmu(stdout, fprintf, env); >> #endif >> tlb_flush(env, 1); >> } > > This patch was not applied to master. Was that an oversight or is > something wrong with it? (Preparing a v2 series for 1.5.) Wasn't this a part of RFC series? What happened with the rest of the 19? > > Andreas > > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Am 14.02.2013 22:42, schrieb Blue Swirl: > On Thu, Feb 14, 2013 at 3:46 PM, Andreas Färber <afaerber@suse.de> wrote: >> Am 27.01.2013 14:32, schrieb Andreas Färber: >>> Signed-off-by: Andreas Färber <afaerber@suse.de> >>> --- >>> target-sparc/ldst_helper.c | 2 +- >>> 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) >>> >>> diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c >>> index cf1bddf..7decd66 100644 >>> --- a/target-sparc/ldst_helper.c >>> +++ b/target-sparc/ldst_helper.c >>> @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, >>> DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", >>> oldreg, env->lsu); >>> #ifdef DEBUG_MMU >>> - dump_mmu(stdout, fprintf, env1); >>> + dump_mmu(stdout, fprintf, env); >>> #endif >>> tlb_flush(env, 1); >>> } >> >> This patch was not applied to master. Was that an oversight or is >> something wrong with it? (Preparing a v2 series for 1.5.) > > Wasn't this a part of RFC series? What happened with the rest of the 19? This is a standalone bugfix, originally intended for 1.4 as indicated by the subject. cris, ppc and s390x bugfixes were applied through the maintainers during Soft Freeze. The cris Coding Style cleanup is still pending, no feedback from Edgar on whether it is okay or not yet (already 3x on the list). The actual RFC patches based on those are being redone similar to the tmp105 v3 patch I ping'ed, based on suggestions from Anthony and Alex. I inserted a patch to add qemu_log_mask_vprintf() as prerequisite. Andreas
Cleanup patch is ok with me. Acked-by: Edgar E. Iglesias <edgar.iglesias@gmail.com> --- Sent from my phone On Feb 15, 2013 12:24 AM, "Andreas Färber" <afaerber@suse.de> wrote: > > Am 14.02.2013 22:42, schrieb Blue Swirl: > > On Thu, Feb 14, 2013 at 3:46 PM, Andreas Färber <afaerber@suse.de> wrote: > >> Am 27.01.2013 14:32, schrieb Andreas Färber: > >>> Signed-off-by: Andreas Färber <afaerber@suse.de> > >>> --- > >>> target-sparc/ldst_helper.c | 2 +- > >>> 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > >>> > >>> diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c > >>> index cf1bddf..7decd66 100644 > >>> --- a/target-sparc/ldst_helper.c > >>> +++ b/target-sparc/ldst_helper.c > >>> @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, > >>> DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", > >>> oldreg, env->lsu); > >>> #ifdef DEBUG_MMU > >>> - dump_mmu(stdout, fprintf, env1); > >>> + dump_mmu(stdout, fprintf, env); > >>> #endif > >>> tlb_flush(env, 1); > >>> } > >> > >> This patch was not applied to master. Was that an oversight or is > >> something wrong with it? (Preparing a v2 series for 1.5.) > > > > Wasn't this a part of RFC series? What happened with the rest of the 19? > > This is a standalone bugfix, originally intended for 1.4 as indicated by > the subject. > > cris, ppc and s390x bugfixes were applied through the maintainers during > Soft Freeze. > > The cris Coding Style cleanup is still pending, no feedback from Edgar > on whether it is okay or not yet (already 3x on the list). > > The actual RFC patches based on those are being redone similar to the > tmp105 v3 patch I ping'ed, based on suggestions from Anthony and Alex. > I inserted a patch to add qemu_log_mask_vprintf() as prerequisite. > > Andreas > > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Thanks, applied. On Sun, Jan 27, 2013 at 1:32 PM, Andreas Färber <afaerber@suse.de> wrote: > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > target-sparc/ldst_helper.c | 2 +- > 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) > > diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c > index cf1bddf..7decd66 100644 > --- a/target-sparc/ldst_helper.c > +++ b/target-sparc/ldst_helper.c > @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, > DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", > oldreg, env->lsu); > #ifdef DEBUG_MMU > - dump_mmu(stdout, fprintf, env1); > + dump_mmu(stdout, fprintf, env); > #endif > tlb_flush(env, 1); > } > -- > 1.7.10.4 >
diff --git a/target-sparc/ldst_helper.c b/target-sparc/ldst_helper.c index cf1bddf..7decd66 100644 --- a/target-sparc/ldst_helper.c +++ b/target-sparc/ldst_helper.c @@ -1850,7 +1850,7 @@ void helper_st_asi(CPUSPARCState *env, target_ulong addr, target_ulong val, DPRINTF_MMU("LSU change: 0x%" PRIx64 " -> 0x%" PRIx64 "\n", oldreg, env->lsu); #ifdef DEBUG_MMU - dump_mmu(stdout, fprintf, env1); + dump_mmu(stdout, fprintf, env); #endif tlb_flush(env, 1); }
Signed-off-by: Andreas Färber <afaerber@suse.de> --- target-sparc/ldst_helper.c | 2 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-)