moving sctp to a separate directory ? (Re: svn commit: r201523
- head/sys/netinet)
Robert Watson
rwatson at FreeBSD.org
Thu Jan 7 11:29:36 UTC 2010
On Mon, 4 Jan 2010, Luigi Rizzo wrote:
> taking a random commit to this tree -- would you guys consider moving the
> sctp sources to its own directory, e.g. netinet/sctp/ or something that
> suits better ?
>
> It would help to browse through the directory:
>
> NAME FILES LINES
>
> netinet/ 156 153k
> sctp* 36 66k
> tcp* 24 16k
> ipfw/ 9 9k
>
>
> I understand that some of the headers should remain in netinet/ but surely
> the code is not bound to a specific place...
Keep in mind that every file rename invalidates outstanding patch sets against
those files maintained in our own branches, and at external vendors. I think
we should be cautious about gratuitous renaming, as it increases the work
factor both inside and outside the project significantly for relatively little
gain. Especially when the files already have a quite consistent naming scheme
that makes it easy to know which files belong to which component -- tcp_*,
sctp_*, udp_*, etc.
Robert N M Watson
Computer Laboratory
University of Cambridge
>
> cheers
> luigi
>
> On Mon, Jan 04, 2010 at 06:25:38PM +0000, Michael Tuexen wrote:
>> Author: tuexen
>> Date: Mon Jan 4 18:25:38 2010
>> New Revision: 201523
>> URL: http://svn.freebsd.org/changeset/base/201523
>>
>> Log:
>> Correct usage of parenthesis.
>>
>> PR: kern/142066
>> Approved by: rrs (mentor)
>> Obtained from: Henning Petersen, Bruce Cran.
>> MFC after: 2 weeks
>>
>> Modified:
>> head/sys/netinet/sctp_pcb.c
>>
>> Modified: head/sys/netinet/sctp_pcb.c
>> ==============================================================================
>> --- head/sys/netinet/sctp_pcb.c Mon Jan 4 18:21:27 2010 (r201522)
>> +++ head/sys/netinet/sctp_pcb.c Mon Jan 4 18:25:38 2010 (r201523)
>> @@ -5528,7 +5528,7 @@ sctp_pcb_init()
>>
>> /* Init the TIMEWAIT list */
>> for (i = 0; i < SCTP_STACK_VTAG_HASH_SIZE; i++) {
>> - LIST_INIT(&SCTP_BASE_INFO(vtag_timewait[i]));
>> + LIST_INIT(&SCTP_BASE_INFO(vtag_timewait)[i]);
>> }
>>
>> #if defined(SCTP_USE_THREAD_BASED_ITERATOR)
>> @@ -6385,7 +6385,7 @@ sctp_is_vtag_good(struct sctp_inpcb *inp
>> }
>> skip_vtag_check:
>>
>> - chain = &SCTP_BASE_INFO(vtag_timewait[(tag % SCTP_STACK_VTAG_HASH_SIZE))];
>> + chain = &SCTP_BASE_INFO(vtag_timewait)[(tag % SCTP_STACK_VTAG_HASH_SIZE)];
>> /* Now what about timed wait ? */
>> if (!LIST_EMPTY(chain)) {
>> /*
>
More information about the svn-src-head
mailing list