From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0BEB4C43217 for ; Fri, 18 Mar 2022 22:46:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232698AbiCRWrU (ORCPT ); Fri, 18 Mar 2022 18:47:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37072 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238402AbiCRWrT (ORCPT ); Fri, 18 Mar 2022 18:47:19 -0400 Received: from mail-oo1-xc32.google.com (mail-oo1-xc32.google.com [IPv6:2607:f8b0:4864:20::c32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 79D0A22A288 for ; Fri, 18 Mar 2022 15:45:58 -0700 (PDT) Received: by mail-oo1-xc32.google.com with SMTP id x26-20020a4a9b9a000000b003211029e80fso11913284ooj.5 for ; Fri, 18 Mar 2022 15:45:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=ekiD2sX45IwGk6wtsqdwxLttTAGZcGccRjZl1i6g1JA=; b=dMDapo5gWo0mYQwoT0A/aOZqMdPyZSCOVsL5zPGffJvLJZp0AfN2E9xdlck/nSLv/N hEsSe5KNbjsmefIhgIIv1yDVHMP+6YIgd44fW9U2d/c1Akfns382n70WbM/10yIkuHjZ SUNIZp82WOT2MEcG49QwDBCrffF4++Gcu4CkY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=ekiD2sX45IwGk6wtsqdwxLttTAGZcGccRjZl1i6g1JA=; b=3aYE7dnk/gwO9tZ8xsAZnP3SzzdiDISfmVru4+t8TKz8E/DyZctshnTQck4eYb948V cIgkPSe6+nEGQoOFdk5RGf4iT6rMtvjCji7+O3c+UUaMGgKIkaTzEcMlDYBJaQ0y4tw+ KLl+O8EHRhDds8Zjho54Apu3HCW4xav8KUdqI8Ij4jvg6i+SsjpSscziaAJh+15H6nLl vlLTrZKf0YdWnSXdZWrU2aH/99TR9J2kj6ysi5mZmndLxcmwfimdysJ8GiHU4f+1xS/w QapSG/7K+mkqgP1ctun3IJWEFg4jo3cJDGz6Azsy1QnFxFxfgXrGfY9Lc5Hu6NUZf9GI AJYA== X-Gm-Message-State: AOAM532TAF51kTKDJ6pjp4DJtATpNM7d9C0aLQuzZHTKb+KIzg8cdLpk 3xTYGJAgIFsw6fmnIvlnLjDpSg== X-Google-Smtp-Source: ABdhPJziihDBziDyTkE7iKcLTBW6BUUFUpCgkLWdsk/9MySxxcozoxXb4xoR4U6FbxLxPWes+cEMkA== X-Received: by 2002:a05:6820:814:b0:322:b1b2:2456 with SMTP id bg20-20020a056820081400b00322b1b22456mr3591951oob.0.1647643557718; Fri, 18 Mar 2022 15:45:57 -0700 (PDT) Received: from [192.168.1.128] ([71.205.29.0]) by smtp.gmail.com with ESMTPSA id x12-20020a056830244c00b005ad233e0ba3sm4330223otr.48.2022.03.18.15.45.55 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 18 Mar 2022 15:45:57 -0700 (PDT) Subject: Re: [PATCH 7/9] usb: usbip: eliminate anonymous module_init & module_exit To: Randy Dunlap , linux-kernel@vger.kernel.org Cc: "Michael S. Tsirkin" , Jason Wang , Paolo Bonzini , Stefan Hajnoczi , Jens Axboe , Amit Shah , Arnd Bergmann , Greg Kroah-Hartman , Eli Cohen , Saeed Mahameed , Leon Romanovsky , Pablo Neira Ayuso , Jozsef Kadlecsik , Florian Westphal , "David S. Miller" , Jakub Kicinski , "James E.J. Bottomley" , "Martin K. Petersen" , Felipe Balbi , =?UTF-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= , Sebastian Andrzej Siewior , Krzysztof Opasiak , Igor Kotrasinski , Valentina Manea , Shuah Khan , Jussi Kivilinna , Joachim Fritschi , Herbert Xu , Thomas Gleixner , Steven Rostedt , Ingo Molnar , Karol Herbst , Pekka Paalanen , Dave Hansen , Andy Lutomirski , Peter Zijlstra , Borislav Petkov , "H. Peter Anvin" , netfilter-devel@vger.kernel.org, coreteam@netfilter.org, netdev@vger.kernel.org, linux-block@vger.kernel.org, linux-crypto@vger.kernel.org, linux-rdma@vger.kernel.org, linux-scsi@vger.kernel.org, linux-usb@vger.kernel.org, nouveau@lists.freedesktop.org, virtualization@lists.linux-foundation.org, x86@kernel.org, Shuah Khan References: <20220316192010.19001-1-rdunlap@infradead.org> <20220316192010.19001-8-rdunlap@infradead.org> From: Shuah Khan Message-ID: <282f4857-7b4f-810e-af0e-e9ca8129c7fc@linuxfoundation.org> Date: Fri, 18 Mar 2022 16:45:54 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: <20220316192010.19001-8-rdunlap@infradead.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: netfilter-devel@vger.kernel.org On 3/16/22 1:20 PM, Randy Dunlap wrote: > Eliminate anonymous module_init() and module_exit(), which can lead to > confusion or ambiguity when reading System.map, crashes/oops/bugs, > or an initcall_debug log. > > Give each of these init and exit functions unique driver-specific > names to eliminate the anonymous names. > > Example 1: (System.map) > ffffffff832fc78c t init > ffffffff832fc79e t init > ffffffff832fc8f8 t init > > Example 2: (initcall_debug log) > calling init+0x0/0x12 @ 1 > initcall init+0x0/0x12 returned 0 after 15 usecs > calling init+0x0/0x60 @ 1 > initcall init+0x0/0x60 returned 0 after 2 usecs > calling init+0x0/0x9a @ 1 > initcall init+0x0/0x9a returned 0 after 74 usecs > > Fixes: 80fd9cd52de6 ("usbip: vudc: Add VUDC main file") > Signed-off-by: Randy Dunlap > Cc: Krzysztof Opasiak > Cc: Igor Kotrasinski > Cc: Greg Kroah-Hartman > Cc: Valentina Manea > Cc: Shuah Khan > Cc: Shuah Khan > Cc: linux-usb@vger.kernel.org > --- > drivers/usb/usbip/vudc_main.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > --- lnx-517-rc8.orig/drivers/usb/usbip/vudc_main.c > +++ lnx-517-rc8/drivers/usb/usbip/vudc_main.c > @@ -28,7 +28,7 @@ static struct platform_driver vudc_drive > > static struct list_head vudc_devices = LIST_HEAD_INIT(vudc_devices); > > -static int __init init(void) > +static int __init vudc_init(void) > { > int retval = -ENOMEM; > int i; > @@ -86,9 +86,9 @@ cleanup: > out: > return retval; > } > -module_init(init); > +module_init(vudc_init); > > -static void __exit cleanup(void) > +static void __exit vudc_cleanup(void) > { > struct vudc_device *udc_dev = NULL, *udc_dev2 = NULL; > > @@ -103,7 +103,7 @@ static void __exit cleanup(void) > } > platform_driver_unregister(&vudc_driver); > } > -module_exit(cleanup); > +module_exit(vudc_cleanup); > > MODULE_DESCRIPTION("USB over IP Device Controller"); > MODULE_AUTHOR("Krzysztof Opasiak, Karol Kosik, Igor Kotrasinski"); > Thanks for fixing this. Acked-by: Shuah Khan thanks, -- Shuah