Re: [PATCH][Bug 5132] fix sys_poll() large timeout handling

On 31.08.2005 [13:01:09 -0700], Nishanth Aravamudan wrote:
> Sorry everybody, forgot the most important Cc: :)
> 
> -Nish
> 
> Hi Andrew,
> 
> In looking at Bug 5132 and sys_poll(), I think there is a flaw in the
> current code.
> 
> The @timeout parameter to sys_poll() is in milliseconds but we compare
> it to (MAX_SCHEDULE_TIMEOUT / HZ), which is jiffies/jiffies-per-sec or
> seconds. That seems blatantly broken. Also, I think we are better served
> by converting to jiffies first then comparing, as opposed to converting
> our maximum to milliseconds (or seconds, incorrectly) and comparing.
> 
> Comments, suggestions for improvement?

I haven't got any responses (here or on the bug)... A silent NACK?
Anything I should change to make people happier?

Thanks,
Nish
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Follow-Ups:
- [UPDATE PATCH][Bug 5132] fix sys_poll() large timeout handling
  - From: Nishanth Aravamudan <[email protected]>

References:
- [PATCH][Bug 5132] fix sys_poll() large timeout handling
  - From: Nishanth Aravamudan <[email protected]>

Prev by Date: Re: kbuild & C++
Next by Date: Re: wakeup on lan enable without compiling as module
Previous by thread: [PATCH][Bug 5132] fix sys_poll() large timeout handling
Next by thread: [UPDATE PATCH][Bug 5132] fix sys_poll() large timeout handling
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Netfilter] [Bugtraq] [Photo] [Gimp] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Video 4 Linux] [Linux for the blind]