| > 7. Miscellaneous DCCP updates | > ----------------------------- <snip> | > [DCCP]: Promote CCID2 as default CCID | | Applied to simplify things for now, but I still think that for some | setups it may be interesting to allow CCID2 not to be built. | Yes, there is room to revise this for something better later on. I think this is much similar to the "use Reno or Cubic as default" in TCP, and maybe one could use a similar Kconfig menu as it is currently done for the TCP modules. | > | > 8. The Feature-Negotiation patch set | > ------------------------------------ | > The set splits into several smaller (self-contained) blocks, all documented in this section. | > The first block is distinct from feature negotiation, it only concerns request sockets: | > | > [DCCP]: Allow to parse options on Request Sockets | | This one can be folded with the next, I guess, since its only use of | the newly introduce dreq on dccp_parse_options is to test against NULL, | but that could be achieved by checking the packet type. | | > [DCCP]: Add (missing) option parsing to request_sock processing Yes this is a tiny one. The first patch introduces the general code, the second is separate just because it adds the newly updated dccp_parse_options() to dccp_check_req().
This is a question I am often puzzled about -- picking the right granularity of patches. Often it is quite fine-granular, so please do feel free to combine; breaking them into single ones is mostly for reviewing/readability. - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html