Messages in this thread | | | From | Tabris <> | Subject | Re: Jiffies_64 for 2.4.22-ac | Date | Fri, 12 Sep 2003 11:07:03 -0400 |
| |
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On Friday 12 September 2003 10:00 am, Tim Schmielau wrote: > On Fri, 12 Sep 2003, Tabris wrote: > > I took Tim Schmielau's jiffies_64 patch, and ported it to -ac > > > > currently running on my machine here. > > comments? did i screw up horribly? > > see my comments below: > > +#define get_uidle_64() > > get_64bits(&(init_tasks[0]->times.tms_utime),\ + > > &uidle_msb_flips) > > +#define get_sidle_64() > > get_64bits(&(init_tasks[0]->times.tms_stime),\ + > > &sidle_msb_flips) > > for -ac this needs to be > > +#define get_uidle_64() get_64bits(&(init_task.times.tms_utime),\ > + &uidle_msb_flips) > +#define get_sidle_64() get_64bits(&(init_task.times.tms_stime),\ > + &sidle_msb_flips) > > > + check_one(init_tasks[0]->times.tms_utime, &uidle_msb_flips); > > + check_one(init_tasks[0]->times.tms_stime, &sidle_msb_flips); > > ditto > > > +#define get_uidle_64() (init_tasks[0]->times.tms_utime) > +#define get_sidle_64() (init_tasks[0]->times.tms_stime) > > ditto > > > - unsigned long uptime; > > - unsigned long idle; > > + u64 uptime, idle; > > int len; > > the declaration > unsigned long uptime_remainder, idle_remainder; > is missing > > > #if HZ!=100 > > len = sprintf(page,"%lu.%02lu %lu.%02lu\n", > > - uptime / HZ, > > - (((uptime % HZ) * 100) / HZ) % 100, > > - idle / HZ, > > - (((idle % HZ) * 100) / HZ) % 100); > > + (unsigned long) uptime, > > + uptime_remainder, > > + (unsigned long) idle / HZ, > > + idle_remainder); > > since we're in the HZ!=100 branch, this needs to be > > + (uptime_remainder * 100) / HZ, > + (unsigned long) idle, > + (idle_remainder * 100) / HZ); > > > > I wonder it actually compiled, but otherwise it looks good. > > Tim > oops. yeah, i merged it originally against a different tree... that patch didn't apply against 2.4.22-ac and i forgot to compile test it since i was too tired.
updated/fixed patch later today.
- -- tabris - - "...A strange enigma is man!" "Someone calls him a soul concealed in an animal," I suggested. "Winwood Reade is good upon the subject," said Holmes. "He remarked that, while the individual man is an insoluble puzzle, in the aggregate he becomes a mathematical certainty. You can, for example, never foretell what any one man will do, but you can say with precision what an average number will be up to. Individuals vary, but percentages remain constant. So says the statistician." -- Sherlock Holmes, "The Sign of Four" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (GNU/Linux)
iD8DBQE/YeEYtTgrITXtL+8RAm6zAJ93ljgW0p70fgrrjvhu52Dod+fOawCcCLjc Ns6HWVXlCbIHWf8t2FB9BzI= =ya6P -----END PGP SIGNATURE-----
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |