[Linux-hls-devel] [Fwd: [Linux-hls-cvs] Re: /proc/HLS/tasks listing schedulers correctly?]

[Luca A. <luc...@em...>]

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