From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030217AbbE2PSb (ORCPT ); Fri, 29 May 2015 11:18:31 -0400 Received: from mail.windriver.com ([147.11.1.11]:48967 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750998AbbE2PSV (ORCPT ); Fri, 29 May 2015 11:18:21 -0400 Message-ID: <5568829A.5080109@windriver.com> Date: Fri, 29 May 2015 11:15:38 -0400 From: Paul Gortmaker User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Matt Turner CC: LKML , Richard Henderson , Ivan Kokshaysky , linux-alpha Subject: Re: [PATCH 06/15] alpha: don't use module_init for non-modular core code References: <1432860493-23831-1-git-send-email-paul.gortmaker@windriver.com> <1432860493-23831-7-git-send-email-paul.gortmaker@windriver.com> In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2015-05-29 01:51 AM, Matt Turner wrote: > On Thu, May 28, 2015 at 5:48 PM, Paul Gortmaker > wrote: >> The srm console is always built in. It will never be modular, >> so using module_init as an alias for __initcall is rather >> misleading. >> >> Fix this up now, so that we can relocate module_init from >> init.h into module.h in the future. If we don't do this, we'd >> have to add module.h to obviously non-modular code, and that >> would be a worse thing. >> >> Direct use of __initcall is discouraged, vs prioritized ones. >> Use of device_initcall is consistent with what __initcall >> maps onto, and hence does not change the init order, making the >> impact of this change zero. Should someone with real hardware >> for boot testing want to change it later to arch_initcall or >> console_initcall, they can do that at a later date. >> >> Cc: Richard Henderson >> Reviewed-by: Richard Henderson >> Cc: Ivan Kokshaysky >> Cc: Matt Turner >> Cc: linux-alpha@vger.kernel.org >> Signed-off-by: Paul Gortmaker >> --- > > I included this in my pull request to Linus that went upstream > yesterday. Thanks! OK, thanks - I'll drop it from here if/when I rebase onto the 4.1-rc6 pending on Sunday. If I drop it now I'll risk introducing a bisect failure for alpha, since my current baseline of 4.1-rc5 doesn't have this now-upstream change from the old original series. Paul. -- >