[Linux-hls-devel] [Fwd: [Linux-hls-cvs] Re: /proc/HLS/tasks listing schedulers correctly?]
Status: Pre-Alpha
Brought to you by:
lucabe
|
From: Luca A. <luc...@em...> - 2003-08-27 08:51:26
|
I sent this message to the wrong list...
-----Forwarded Message-----
> From: Luca Abeni <luc...@em...>
> To: Paul Koufalas <pko...@au...>
> Cc: lin...@li...
> Subject: [Linux-hls-cvs] Re: /proc/HLS/tasks listing schedulers correctly?
> Date: 27 Aug 2003 10:35:15 +0100
>
> Hi Paul,
>
> > Luca, we ran schedtest and examined /proc/HLS/tasks while it was
> > running: it said that schedtest was using rr2, not res1 as the
> > scheduler.
> I checked, and it does not seem to be a typo. I think there is something
> wrong in hls_setsched()... Thanks for finding and reporting the bug.
>
> I will need to reproduce the problem, and to debug it; I'll do it this
> evening. I think I know where the bug is, but I'll need some testing;
> maybe something like this patch can fix it:
> Index: linux/hls_ctl.c
> ===================================================================
> RCS file: /cvsroot/linux-hls/hls/linux/hls_ctl.c,v
> retrieving revision 1.6
> diff -u -r1.6 hls_ctl.c
> --- linux/hls_ctl.c 25 Feb 2003 09:18:55 -0000 1.6
> +++ linux/hls_ctl.c 27 Aug 2003 08:24:48 -0000
> @@ -97,10 +97,12 @@
>
> HLS_ASSERT (th1->vp->State == VP_Waiting);
> th1->vp->TopSched = new_sched;
> -
> +
> status = th1->vp->TopSched->CB->B_RegisterVP(new_sched, th_inst,
> th1->vp);
> HLS_ASSERT(status == HLS_SUCCESS);
> -
> +
> + th->Parent = new_sched;
> +
> HLSDbgPrint(3, ("(infr) registered\n"));
>
> /* This cannot be done until the sched param are set...
>
>
> (WARNING: This is a completely untested patch)
> If I am right, the bug should not affect the scheduling (only /proc
> information are broken).
>
> It would be useful to know if schedtest is behaving correctly for you:
> can you send to the list the complete output of schedtest, together with
> the contents of /proc/HLS/* ?
>
> Thanks,
> Luca
> --
> _____________________________________________________________________________
> Copy this in your signature, if you think it is important:
> N O W A R ! ! !
--
_____________________________________________________________________________
Copy this in your signature, if you think it is important:
N O W A R ! ! !
--
Email.it, the professional e-mail, gratis per te: http://www.email.it/f
Sponsor:
Migliaia di prodotti al prezzo che decidi tu. Fai subito la tua offerta!
Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=1650&d=27-8
|