svn commit: r298856 - head/sys/powerpc/powerpc

[Justin Hibbits]

On Apr 30, 2016 13:56, "Pedro F. Giffuni" <pfg at freebsd.org> wrote:
>
> Author: pfg
> Date: Sat Apr 30 18:56:35 2016
> New Revision: 298856
> URL: https://svnweb.freebsd.org/changeset/base/298856
>
> Log:
>   powerpc: Make use of our rounddown() macro when sys/param.h is
available.
>
>   No functional change.
>
> Modified:
>   head/sys/powerpc/powerpc/exec_machdep.c
>
> Modified: head/sys/powerpc/powerpc/exec_machdep.c
>
==============================================================================
> --- head/sys/powerpc/powerpc/exec_machdep.c     Sat Apr 30 18:07:13 2016
      (r298855)
> +++ head/sys/powerpc/powerpc/exec_machdep.c     Sat Apr 30 18:56:35 2016
      (r298856)
> @@ -162,7 +162,7 @@ sendsig(sig_t catcher, ksiginfo_t *ksi,
>                 code = siginfo32.si_code;
>                 sfp = (caddr_t)&sf32;
>                 sfpsize = sizeof(sf32);
> -               rndfsize = ((sizeof(sf32) + 15) / 16) * 16;
> +               rndfsize = rounddown(sizeof(sf32) + 15, 16);
>
>                 /*
>                  * Save user context
> @@ -189,9 +189,9 @@ sendsig(sig_t catcher, ksiginfo_t *ksi,
>                  * 64-bit PPC defines a 288 byte scratch region
>                  * below the stack.
>                  */
> -               rndfsize = 288 + ((sizeof(sf) + 47) / 48) * 48;
> +               rndfsize = 288 + rounddown(sizeof(sf) + 47, 48);
>                 #else
> -               rndfsize = ((sizeof(sf) + 15) / 16) * 16;
> +               rndfsize = rounddown(sizeof(sf) + 15, 16);
>                 #endif
>
>                 /*
>

I think these should all be roundup(sizeof(sf), (16 or 48)).

- Justin