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