Re: [Qemu-devel] [PATCH] MIPS: Correct FCR0 initialization

[Blue Swirl]

On Tue, Aug 7, 2012 at 12:10 PM, Peter Maydell <address@hidden> wrote:
> On 6 August 2012 19:21, Phil Staub <address@hidden> wrote:
>> On Tue, Jun 12, 2012 at 10:28:14AM -0400, address@hidden wrote:
>>> From: Richard Henderson <address@hidden>
>>> On 2012-06-07 18:04, Maciej W. Rozycki wrote:
>>> >  I have verified this change with system emulation running the GDB test
>>> > suite for the mips-sde-elf target (o32, big endian, 24Kf CPU emulated),
>>> > there were 55 progressions and no regressions.
>>> >
>>> > Signed-off-by: Maciej W. Rozycki <address@hidden>
>>> > ---
>>> >
>>> >  Sent on behalf of Nathan, who's since left the company.  Please apply.
>>> >
>>> >   Maciej
>>> >
>>> > qemu-mips-fcr0.diff
>>> > Index: qemu-git-trunk/target-mips/translate.c
>>> > ===================================================================
>>> > --- qemu-git-trunk.orig/target-mips/translate.c     2012-06-04 
>>> > 05:35:53.245610241 +0100
>>> > +++ qemu-git-trunk/target-mips/translate.c  2012-06-04 05:39:26.245563823 
>>> > +0100
>>> > @@ -12776,6 +12776,7 @@ void cpu_state_reset(CPUMIPSState *env)
>>> >      env->CP0_SRSConf3 = env->cpu_model->CP0_SRSConf3;
>>> >      env->CP0_SRSConf4_rw_bitmask = 
>>> > env->cpu_model->CP0_SRSConf4_rw_bitmask;
>>> >      env->CP0_SRSConf4 = env->cpu_model->CP0_SRSConf4;
>>> > +    env->active_fpu.fcr0 = env->cpu_model->CP1_fcr0;
>>>
>>> Reviewed-by: Richard Henderson <address@hidden>
>
>> What are the plans for this patch? It doesn't appear to have been
>> applied in any of the repository branches.
>
> Basically MIPS is currently without an active maintainer, so
> people submitting patches need to keep pinging them until
> one of the core maintainers (usually Blue Swirl) applies them.
> For this purpose the usual approach is to follow up to the patch
> mail saying "Ping" and giving a url to the patch in patchwork,
> like this one:
> http://patchwork.ozlabs.org/patch/163705/
>
> Eventually somebody will take pity on it and apply it, but
> it does require a bit more persistence than for more actively
> maintained areas of the codebase.
>
> -- PMM

Thanks, applied.