Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757662Ab0KVWKu (ORCPT ); Mon, 22 Nov 2010 17:10:50 -0500 Received: from mail.perches.com ([173.55.12.10]:1782 "EHLO mail.perches.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756860Ab0KVWKs (ORCPT ); Mon, 22 Nov 2010 17:10:48 -0500 Subject: Re: [PATCH 00/62] drivers/net: Use static const From: Joe Perches To: David Miller Cc: netdev@vger.kernel.org, e1000-devel@lists.sourceforge.net, linux-usb@vger.kernel.org, linux-wireless@vger.kernel.org, ath5k-devel@venema.h4ckr.net, ath9k-devel@venema.h4ckr.net, libertas-dev@lists.infradead.org, users@rt2x00.serialmonkey.com, linux-kernel@vger.kernel.org, socketcan-core@lists.berlios.de In-Reply-To: <20101120.205059.193725921.davem@davemloft.net> References: <20101120.205059.193725921.davem@davemloft.net> Content-Type: text/plain; charset="UTF-8" Date: Mon, 22 Nov 2010 14:10:46 -0800 Message-ID: <1290463846.27683.36.camel@Joe-Laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4595 Lines: 113 On Sat, 2010-11-20 at 20:50 -0800, David Miller wrote: > From: Joe Perches > Date: Sat, 20 Nov 2010 18:38:01 -0800 > > > Using static const generally increases object text and decreases data size. > > It also generally decreases overall object size. > > Joe, I'm going to be frank and say that I'm not going to review and > apply such a large chunk of networking patches. Hi David. No worries. > Separation is cool, splitting up patches is cool to make review > perhaps easier and more distributed. I know you have ownership of netdev. Thanks for doing all that work. You may not get enough feedback on the very good job that you do at it. I posted these patches not so much to get you to immediately pick them up, but to get review from and/or notify the maintainers of each of these subsystems. I would have submitted them to Jiri Kosina/trivial, but he asked me to post and track these style patches separately and wait for a few weeks before submitting any remaining patches not picked up by maintainers to him. > It severely negatively effects my mood, Can't have that... > You also make this more difficult for me by not using GIT. I do of course use git locally. I have a public tree as well, but I hardly update it. I updated it recently for this. These patches are available in: git://repo.or.cz/linux-2.6/trivial-mods.git 20101121_net_next_static_const Perhaps I should ask Stephen Rothwell to include some trivial tree branch like this in next. > Even with patchwork helping me significantly, it's still a lot of work > to apply large sets of patches. I think that cc'ing netdev is always appropriate for netdev patches, but maybe trivial@kernel.org could become linux-trivial@vger.kernel.org or something like that. Perhaps there could be some way to automatically mark these exceptionally trivial patches as something like "not applicable" so they don't appear on your personal queue. > If I had to take in John Linville's > wireless stuff without GIT I'd be banging my head on a wall. > > And these incessant huge patch bombs also take time away from me for > the things I'd like to at least occiasionally work on that involve > more intellect than monkeying around with such mindless patches. Establishing trust is always a long term thing. Breaking trust is easy too. Anyway, I think these patches are obvious and correct and can be directly applied without significant risk. I think the the issues are: o should the nominal maintainers of the subsystems pick them up o should these subsystem maintainers should be bypassed o should the subsystem mailing lists be cc'd o should these be pulled as a single changeset or multiples I think that getting the nominal subsystem maintainers involved is good, but perhaps not too necessary for these sorts of patches. btw: it doesn't seem that, other than John Linville for wireless, you currently pull from many (any?) other people. Do let me know if you'd consider pulling these sorts of changes from me. $ git log --merges --since=2-years-ago drivers/net | grep "^Author: " | sort | uniq -c | sort -rn 259 Author: David S. Miller 201 Author: Linus Torvalds 45 Author: John W. Linville 15 Author: Russell King 15 Author: Ingo Molnar 14 Author: Benjamin Herrenschmidt 7 Author: Russell King 6 Author: Jiri Kosina 4 Author: Reinette Chatre 3 Author: Roland Dreier 3 Author: James Morris 2 Author: Tony Lindgren 2 Author: Grant Likely 1 Author: Stefano Stabellini 1 Author: root 1 Author: Paul Mackerras 1 Author: Kevin Hilman 1 Author: James Bottomley > Thanks for your understanding. Feel free to submit this stuff in > smaller chunks, say ~10 patches at a time. Let me know if you want to pull or want them in smaller chunks in say a month from now. cheers, Joe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/