From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <libc-alpha-return-54225-listarch-libc-alpha=sources.redhat.com@sourceware.org>
Received: (qmail 12247 invoked by alias); 11 Nov 2014 13:05:12 -0000
Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <libc-alpha.sourceware.org>
List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/libc-alpha/>
List-Post: <mailto:libc-alpha@sourceware.org>
List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: libc-alpha-owner@sourceware.org
Received: (qmail 12129 invoked by uid 89); 11 Nov 2014 13:05:11 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2
X-HELO: mail-qc0-f181.google.com
MIME-Version: 1.0
X-Received: by 10.224.67.8 with SMTP id p8mr6378830qai.97.1415711108493; Tue,
 11 Nov 2014 05:05:08 -0800 (PST)
In-Reply-To: <002801cffaa5$eb2852f0$c178f8d0$@com>
References: <002701cffaa0$77623570$6626a050$@com>
	<002801cffaa5$eb2852f0$c178f8d0$@com>
Date: Tue, 11 Nov 2014 13:05:00 -0000
Message-ID: <CAFqB+Pw4oEhmORJGSjBNtaTn9ZOgWS6-25p=4AYFwGuv72jddg@mail.gmail.com>
Subject: Re: [RFC PATCH] aarch64: improve memset
From: Marcus Shawcroft <marcus.shawcroft@gmail.com>
To: Wilco Dijkstra <wdijkstr@arm.com>
Cc: Richard Henderson <rth@twiddle.net>, will.newton@linaro.org, 
	GNU C Library <libc-alpha@sourceware.org>
Content-Type: text/plain; charset=UTF-8
X-SW-Source: 2014-11/txt/msg00199.txt.bz2

On 7 November 2014 16:14, Wilco Dijkstra <wdijkstr@arm.com> wrote:
>> Richard Henderson wrote:

> I've got a few comments on this patch:
>
> * Do we really need variants for cache line sizes that are never going to be used?
>   I'd say just support 64 and 128, and default higher sizes to no_zva.

We shouldn't be removing support for the other sizes already supported
by the existing implementation.  If the other sizes were deprecated
from the architecture then fair game, but that is not the case.  From
offline conversation with Wilco I gather part of the motivation to
remove is that the none 64  cases cannot be readily tested on HW.
That particular issue was solved in the original implementation using
a hacked qemu.

Cheers
/Marcus

> * Why special case line size=64 only? Unrolling might not help for 128 but should not
>   harm either, and the alignment overhead only increases with larger line sizes, so you
>   want to bypass the zva code in all cases if N < 3-4x line size.
>
> * Is the no-ifunc variant still required/used? We're now having at least 4 different
>   variants which all need to be tested and maintained...
>
> * Finally, which version is used when linking statically? I presume there is some
>   makefile magic that causes the no-zva version to be used, however that might not be
>   optimal for all targets.
>
> Wilco
>
>