Re: Patch [PKT_SCHED]: PSCHED_TADD() and PSCHED_TADD2() can result,tv_usec = 1000000 seems wrong

2006-07-24 Thread David Miller
From: Guillaume Chazarain [EMAIL PROTECTED] Date: Wed, 19 Jul 2006 14:47:34 +0200 Shuya MAEDA wrote : while (__delta USEC_PER_SEC){ ... }, but I think it should be while (__delta = USEC_PER_SEC){ ... }. Is it right? I agree, good catch :-) Applied, thanks a lot. - To unsubscribe from

Re: Patch [PKT_SCHED]: PSCHED_TADD() and PSCHED_TADD2() can result,tv_usec = 1000000 seems wrong

2006-07-19 Thread Shuya MAEDA
Hello. In PSCHED_TADD2, if delta is less than tv.tv_usec (so, less than USEC_PER_SEC too) then tv_res will be before tv. The affectation (tv_res).tv_usec = __delta; is wrong. The same applies to PSCHED_TADD. You are right. It is my mistake. I think the correct fix is simply to restore the

Re: Patch [PKT_SCHED]: PSCHED_TADD() and PSCHED_TADD2() can result,tv_usec = 1000000 seems wrong

2006-07-19 Thread Guillaume Chazarain
Shuya MAEDA wrote : while (__delta USEC_PER_SEC){ ... }, but I think it should be while (__delta = USEC_PER_SEC){ ... }. Is it right? I agree, good catch :-) Thanks. -- Guillaume In PSCHED_TADD and PSCHED_TADD2, if delta is less than tv.tv_usec (so, less than USEC_PER_SEC too) then tv_res

Patch [PKT_SCHED]: PSCHED_TADD() and PSCHED_TADD2() can result,tv_usec = 1000000 seems wrong

2006-07-14 Thread Guillaume Chazarain
Hello, This patch: http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=4ee303dfeac6451b402e3d8512723d3a0f861857 looks wrong. In PSCHED_TADD2, if delta is less than tv.tv_usec (so, less than USEC_PER_SEC too) then tv_res will be before tv. The affectation