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

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

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 ! ! !



--
Email.it, the professional e-mail, gratis per te: http://www.email.it/f

Sponsor:
Cosa ne pensi della patente a punti? Se vuoi dare la tua opinione clicca qui
Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=1723&d=27-8