Re: [Xenomai] question: XENO_OPT_TIMING_SCHEDLAT

[Christophe Blaess <mailist@logilin.fr>]

On 01/07/2012 21:33, Gilles Chanteperdrix wrote:
> It looks like a buffer overflow with a buffer on stack. Please try the
> following patch:
>
> diff --git a/ksrc/nucleus/vfile.c b/ksrc/nucleus/vfile.c
> index 5928aef..a6ad363 100644
> --- a/ksrc/nucleus/vfile.c
> +++ b/ksrc/nucleus/vfile.c
> @@ -811,7 +811,7 @@ ssize_t xnvfile_get_blob(struct xnvfile_input *input,
>   {
>          ssize_t nbytes = input->size;
>
> -       if (nbytes < size)
> +       if (nbytes > size)
>                  nbytes = size;
>
>          if (nbytes > 0 && copy_from_user(data, input->u_buf, nbytes))
> @@ -904,7 +904,7 @@ ssize_t xnvfile_get_integer(struct xnvfile_input
> *input, long *valp)
>          ssize_t nbytes;
>          long val;
>
> -       nbytes = xnvfile_get_blob(input, buf, sizeof(buf));
> +       nbytes = xnvfile_get_blob(input, buf, sizeof(buf) - 1);
>          if (nbytes < 0)
>                  return nbytes;
>

The patch is ok, I do not have any segfault.

But, there's still something weird (I run on a stock xenomai 2.6.0 with 
adeos-ipipe-2.6.38.8-arm-1.18-04.patch, I'll try on a 2.6 git kernel)

[Panda]#*echo 2388 > /proc/xenomai/latency*
[Panda]#*cat /proc/xenomai/latency*
2386
[Panda]#*echo 2386 > /proc/xenomai/latency*
[Panda]#*cat /proc/xenomai/latency*
2384
[Panda]#*echo 2384 > /proc/xenomai/latency*
[Panda]#*cat /proc/xenomai/latency*
2382
[Panda]#

I suspect something wrong in xnarch_tsc_to_ns()/xnarch_ns_to_tsc() maybe 
in xnarch_llimd(). I'll investigate more this afternoon.