Re: [Qemu-devel] [PATCH v3 00/15] target-sparc improvements

[Mark Cave-Ayland]

On 27/10/16 18:07, Richard Henderson wrote:

> The two main goals in this patch set are:
>   * Make use of the new MO_ALIGN_* flags, to allow less use of check_align,
>     and support partially misaligned fp memory ops.
>   * More cleanups for ASIs, in the end using the new atomic ops.
> 
> Changes since v2:
>   * Rebased on master with the atomic patches merged; one minor conflict 
> fixed.
>   * In patch 11, move and simplify address_mask.  It's no longer used
>     at all by sparc32 and clang warned about the unused inline function.
> 
> Changes since v1:
>   * The "Remove asi helper code handled inline" patch retains the code within
>     ldda to handle asis that must be handled out of line.  This fixes the
>     FreeBSD 10.3 boot problem.  While the UA2007 spec (and thus sun4v?) 
> doesn't
>     allow for such, it would seem that US2 hardware does.
>   * Dropped the "Optimize writeback of cpu_cond" patch.  This fixes the
>     debian 7.8 install.  Not sure exactly what's wrong with it yet, but
>     let's just drop it for now.
> 
> Mark, you gave your Tested-by to v2.  Do you want to take the
> patch set yourself and send the pull?  Do you want to give me
> a Reviewed-by and have me send the pull?

Hi Richard,

I'm happy for you to send the pull request for this. For various reasons
I haven't had much time this dev cycle to do much other than testing,
let alone in depth review but since both yourself and Artyom have
reviewed the patches then that's fine with me.

Acked-by: Mark Cave-Ayland <address@hidden>


ATB,

Mark.