[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20081208.011843.06192839.davem@davemloft.net>
Date: Mon, 08 Dec 2008 01:18:43 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: gerrit@....abdn.ac.uk
Cc: dccp@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH 6/7] dccp: Remove manual influence on NDP Count feature
From: Gerrit Renker <gerrit@....abdn.ac.uk>
Date: Sat, 6 Dec 2008 17:40:56 +0100
> Updating the NDP count feature is handled automatically now:
> * for CCID-2 it is disabled, since the code does not use NDP counts;
> * for CCID-3 it is enabled, as NDP counts are used to determine loss lengths.
>
> Allowing the user to change NDP values leads to unpredictable and failing
> behaviour, since it is then possible to disable NDP counts even when they
> are needed (e.g. in CCID-3).
>
> This means that only those user settings are sensible that agree with the
> values for Send NDP Count implied by the choice of CCID. But those settings
> are already activated by the feature negotiation (CCID dependency tracking),
> hence this form of support is redundant.
>
> At startup the initialisation of the NDP count feature uses the default
> value of 0, which is done implicitly by the zeroing-out of the socket when
> it is allocated. If the choice of CCID or feature negotiation enables NDP
> count, this will then be updated via the NDP activation handler.
>
> Signed-off-by: Gerrit Renker <gerrit@....abdn.ac.uk>
> Acked-by: Ian McDonald <ian.mcdonald@...di.co.nz>
Applied.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists