Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754397AbZDYBvR (ORCPT ); Fri, 24 Apr 2009 21:51:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752643AbZDYBvA (ORCPT ); Fri, 24 Apr 2009 21:51:00 -0400 Received: from mx2.redhat.com ([66.187.237.31]:47166 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752627AbZDYBu7 (ORCPT ); Fri, 24 Apr 2009 21:50:59 -0400 Message-ID: <49F26C38.9000301@redhat.com> Date: Fri, 24 Apr 2009 21:49:44 -0400 From: Masami Hiramatsu User-Agent: Thunderbird 2.0.0.21 (X11/20090320) MIME-Version: 1.0 To: Tim Abbott CC: Linus Torvalds , Linux kernel mailing list , Anders Kaseorg , Waseem Daher , Denys Vlasenko , Rusty Russell , Andi Kleen , "H. Peter Anvin" , Stephen Rothwell , Jeff Arnold , Andrew Morton , Jon Masters , "Theodore Ts'o" , Nikanth Karthikesan , Arjan van de Ven , Paul Mundt , =?ISO-8859-1?Q?Am=E9rico_Wang?= , Ralf Baechle , Kyle McMaartin , David Howells Subject: Re: [PATCH v3 0/3] Add support for compiling with -ffunction-sections -fdata-sections References: <1238973278-31735-1-git-send-email-tabbott@mit.edu> <49F1CA6E.5080407@redhat.com> In-Reply-To: X-Enigmail-Version: 0.95.7 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2643 Lines: 72 Tim Abbott wrote: > On Fri, 24 Apr 2009, Masami Hiramatsu wrote: > >> What would you think about posting these patches plus -ffunction-sections/ >> -fdata-sections patch to -mm tree, -tip tree, or -next tree as >> "playable" Ksplice patchset? > > The section rename patch often merge conflicts with other changes. I > think that having it sit out in one of those trees for another release > would result in a lot of unnecessary work rebasing patches between that > tree and Linus' tree. I think those are not unnecessary work, because those changes will also be merged to linus tree in the future. Don't you think new feature proposer should pay the cost for updating related works (or, obtaining agreement of each developer for updating their patches)? > Once these -ffunction-sections support patches are merged, I intend to > post the rest of the Ksplice patchset for one of those trees. Of course, but doesn't it need to be merged into linus tree? >> If there are actual problems on those arch, I think you'd better post >> these patches as bugfixes with bug reports. > > These problems are all discussed in the commit messages of the relevant > patches. > > One patch fixes modposting a kernel with more than 65536 ELF sections. > It is certainly possible to get this many with allyesconfig and > -ffunction-sections -fdata-sections. This will be reasonable bugfix for the arches which already use -ffunction-sections -fdata-sections. > Another fixes the issue that when you build with -ffunction-sections, > modpost will print a large number of spurious warnings when it sees > sections like .rodata.__func__.12345 which are generated by the __FUNC__ > macro. Is that possible to change the patches to work if the kernel is compiled with -ffunction-sections? If so, no one will complain about those changes. > The patch with many scattered changes fixes the problem that with > -ffunction-sections -fdata-sections, a function named head gets put in the > ".text.head" section, and your "static int percpu" ends up in the > ".data.percpu" section and probably ends up being made percpu. This is a > potentially nasty problem. For this patch, I agree with Sam Ravnborg. I think his suggestion is a good idea. Thank you, -- Masami Hiramatsu Software Engineer Hitachi Computer Products (America) Inc. Software Solutions Division e-mail: mhiramat@redhat.com -- 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/