[lkml]   [2003]   [Jan]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: posix timers patch comments.
"Randy.Dunlap" wrote:
> Hi Jim,
> I'm reviewing both George's and your POSIX/high-res-timers patches.
> I might not send you comments every time that I come across something,
> and I might not cc lkml on every comment (unless you want me to).
> Anyway, here's a beginning.
> In functions get_eip(), check_expiry(), and run_posix_timers(),
> the <regs> parameter is a void *. Linus generally doesn't like
> void * parameters, so they should be avoided as much as possible,
> and I don't see any reason that <regs> in all of these can't be
> declared as <struct pt_regs *> instead of <void *>. Is there a
> good reason?
> BTW, when do expect to have any updated patches?
> Is your email address changing to

Hi Randy,

It makes sense to include lkml in the discussion. It's an
interesting balancing act. If you openly criticize code that you
hope will be included in the kernel, you risk having your arguments
used as a reason for its rejection. If there is no visible discussion,
the patch will be perceived as un-reviewed.

The code that uses get_eip() is just debug. It may go away soon.
I was feeling lazy and didn't want to chase down the header files needed
to for "struct pt_regs". I just made the change and it was easy.
It turns out that pt_regs was already defined. I also found the
instruction_pointer() macro so I can get rid of the get_eip() function I
added. The timer code has to deal with timer overruns so it always
calculates the amount of time that a time interrupt is delayed. I'm logging
the EIP values which correspond to these long interrupt lockouts.

I'm not sure when I will do the next patch. I only have a few small
changes in the queue. I'm tempted to put it off a few days.

I'm hoping that Comcast won't make me change email addresses.

Jim Houston
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:32    [W:0.025 / U:6.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site