David S. Miller wrote:

> +ENTRY(memset)
> +	/* %o0=buf, %o1=pat, %o2=len */
> +	and		%o1, 0xff, %o3
> +	mov		%o2, %o1
> +	sllx		%o3, 8, %g1
> +	or		%g1, %o3, %o2
> +	sllx		%o2, 16, %g1
> +	or		%g1, %o2, %o2
> +	sllx		%o2, 32, %g1
> +	ba,pt		%XCC, 1f
> +	 or		%g1, %o2, %o2
> +
> +ENTRY(__bzero)

This is confusing data.  No two functions should overlap.  I suggest to
make bzero a little function calling memset similar to what you did with
bcopy.

Second, add cfi.

-- 
➧ Ulrich Drepper ➧ Red Hat, Inc. ➧ 444 Castro St ➧ Mountain View, CA ❖