Re: [PATCH v2 1/1][UPDATE]: [CCID2]: Remove redundant BUG_ON

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/25/07, Gerrit Renker <gerrit@xxxxxxxxxxxxxx> wrote:
> This is a resubmission for the test tree, the change to version 2 is that
> ssthresh has, after cwnd, also been converted to u32; which suggested itself
> since there are many operations and comparisons between these two variables.
>
>
> -----------------------------> Patch v2 <-----------------------------------
> [CCID2]: Remove redundant BUG_ON
>
> This removes a test for `val < 1' which would only have been triggered
> when val < 0, due to a preceding test for 0.
> Fixed by using an unsigned type for cwnd (as in TCP) instead.
> Since ssthresh derives from cwnd and appears in assignments and comparisons,
> the type of ssthresh was changed to matched that of cwnd.
>
> Signed-off-by: Gerrit Renker <gerrit@xxxxxxxxxxxxxx>

Signed-off-by: Ian McDonald <ian.mcdonald@xxxxxxxxxxx>
-- 
Web1: http://wand.net.nz/~iam4/
Web2: http://www.jandi.co.nz
Blog: http://iansblog.jandi.co.nz
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux