This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
[ping][PATCH 2/2] Add probe markers for sin, cos, asin and acos
- From: Siddhesh Poyarekar <siddhesh at redhat dot com>
- To: libc-alpha at sourceware dot org
- Cc: Allan McRae <allan at archlinux dot org>
- Date: Tue, 19 Nov 2013 19:04:34 +0530
- Subject: [ping][PATCH 2/2] Add probe markers for sin, cos, asin and acos
- Authentication-results: sourceware.org; auth=none
- References: <20131025123854 dot GM23099 at spoyarek dot pnq dot redhat dot com> <526A680D dot 20405 at archlinux dot org> <CAAHN_R25QQYcNtakKNQd-svpitNvCKbddQoTDB8F1Ak8M_NKPg at mail dot gmail dot com> <20131028064947 dot GD1633 at spoyarek dot pnq dot redhat dot com>
Ping!
On Mon, Oct 28, 2013 at 12:19:48PM +0530, Siddhesh Poyarekar wrote:
> On Fri, Oct 25, 2013 at 06:48:21PM +0530, Siddhesh Poyarekar wrote:
> > On 25 October 2013 18:16, Allan McRae <allan@archlinux.org> wrote:
> > > You are doing more than add the probe here. I would have thought that
> > > clean-up would need to be a separate patch, or at least have a ChangeLog
> > > entry.
> >
> > OK, I'll split it up in my next iteration.
>
>
> ... and this is part two, which adds the probes.
>
> Siddhesh
>
> * manual/probes.texi (Mathematical Function Probes): Add
> documentation for sin, cos, asin and acos probes.
> * sysdeps/ieee754/dbl-64/sincos32.c: Include stap-probe.h.
> (__sin32): Add slowasin probe.
> (__cos32): Add slowacos probe.
> (__mpsin): Add slowsin probe.
> (__mpcos): Add slowcos probe.
>
> diff --git a/manual/probes.texi b/manual/probes.texi
> index 5492bb7..108f460 100644
> --- a/manual/probes.texi
> +++ b/manual/probes.texi
> @@ -353,3 +353,45 @@ results in multiple precision computation with precision 32. Argument
> @var{$arg1} is the input to the function and @var{$arg2} is the computed
> result.
> @end deftp
> +
> +@deftp Probe slowasin (double @var{$arg1}, double @var{$arg2})
> +This probe is hit when the @code{asin} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function and @var{$arg2} is the computed
> +result.
> +@end deftp
> +
> +@deftp Probe slowacos (double @var{$arg1}, double @var{$arg2})
> +This probe is hit when the @code{acos} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function and @var{$arg2} is the computed
> +result.
> +@end deftp
> +
> +@deftp Probe slowsin (double @var{$arg1}, double @var{$arg2})
> +This probe is hit when the @code{sin} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function and @var{$arg2} is the computed
> +result.
> +@end deftp
> +
> +@deftp Probe slowcos (double @var{$arg1}, double @var{$arg2})
> +This probe is hit when the @code{cos} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function and @var{$arg2} is the computed
> +result.
> +@end deftp
> +
> +@deftp Probe slowsin_dx (double @var{$arg1}, double @var{$arg2}, double @var{$arg3})
> +This probe is hit when the @code{sin} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function, @var{$arg2} is the error bound of
> +@var{$arg1} and @var{$arg3} is the computed result.
> +@end deftp
> +
> +@deftp Probe slowcos_dx (double @var{$arg1}, double @var{$arg2}, double @var{$arg3})
> +This probe is hit when the @code{cos} function is called with an input that
> +results in multiple precision computation with precision 32. Argument
> +@var{$arg1} is the input to the function, @var{$arg2} is the error bound of
> +@var{$arg1} and @var{$arg3} is the computed result.
> +@end deftp
> diff --git a/sysdeps/ieee754/dbl-64/sincos32.c b/sysdeps/ieee754/dbl-64/sincos32.c
> index 11654d6..b6e93a4 100644
> --- a/sysdeps/ieee754/dbl-64/sincos32.c
> +++ b/sysdeps/ieee754/dbl-64/sincos32.c
> @@ -43,6 +43,7 @@
> #include "mpa.h"
> #include "sincos32.h"
> #include <math_private.h>
> +#include <stap-probe.h>
>
> #ifndef SECTION
> # define SECTION
> @@ -149,6 +150,7 @@ __sin32 (double x, double res, double res1)
> /* if a > 0 return min (res, res1), otherwise return max (res, res1). */
> if ((a.d[0] > 0 && res > res1) || (a.d[0] <= 0 && res < res1))
> res = res1;
> + LIBC_PROBE (slowasin, 2, &res, &x);
> return res;
> }
>
> @@ -182,6 +184,7 @@ __cos32 (double x, double res, double res1)
> /* if a > 0 return max (res, res1), otherwise return min (res, res1). */
> if ((a.d[0] > 0 && res < res1) || (a.d[0] <= 0 && res > res1))
> res = res1;
> + LIBC_PROBE (slowacos, 2, &res, &x);
> return res;
> }
>
> @@ -240,6 +243,7 @@ __mpsin (double x, double dx, bool reduce_range)
> default:
> __mp_dbl (&s, &y, p);
> }
> + LIBC_PROBE (slowsin, 3, &x, &dx, &y);
> return y;
> }
>
> @@ -298,6 +302,7 @@ __mpcos (double x, double dx, bool reduce_range)
> default:
> __mp_dbl (&c, &y, p);
> }
> + LIBC_PROBE (slowcos, 3, &x, &dx, &y);
> return y;
> }
>