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 X-Spam-Level: X-Spam-Status: No, score=-6.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 589E7C432BE for ; Wed, 1 Sep 2021 16:51:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3E83460041 for ; Wed, 1 Sep 2021 16:51:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344181AbhIAQwM (ORCPT ); Wed, 1 Sep 2021 12:52:12 -0400 Received: from mail.kernel.org ([198.145.29.99]:38824 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245504AbhIAQwK (ORCPT ); Wed, 1 Sep 2021 12:52:10 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 84A0660525; Wed, 1 Sep 2021 16:51:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1630515072; bh=A9wkNg9aBJrglsUZ5b/buH7xJaIQnLA+gD3vgVvDd7s=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QPeXShP9yKHzHlMdGL7qlLOd2lcOPNGRl8kL0qYwIlxJPbLzdlLogxSB0S1nBPiOc Pm7JdoOZozq1ou1DXfeRSbjYD9TWSXqmORrEypMmDIJhPT07uUNT9aINUF9qNv51ar +IKAIzlaYNjJIU9AZQJC+5QNVoX4ufAex0OeNuGw= Date: Wed, 1 Sep 2021 18:51:09 +0200 From: Greg KH To: Krish Jain Cc: Bryan Brattlof , "Fabio M. De Francesco" , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Declare the file_operations struct as const Message-ID: References: <20210829144531.c2syu4vv4s22dlg7@h510> <20210831133533.6aayzjghdakrhy56@h510> <13366651.n50aozgL3V@localhost.localdomain> <20210831230014.cp46vho2hnndacjp@h510> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 01, 2021 at 05:34:36PM +0200, Krish Jain wrote: > Oh ok, thanks Greg. I only attempted to use "make CCFLAGS=-Werror W=1 > M=drivers/staging/android/" as that's the command Bryan used earlier > and it worked. > > "As for your patch, I built the driver using: > > $ make CCFLAGS=-Werror W=1 M=drivers/staging/android" > > > Can you tell me why this is the case? Again, it depends on your kernel configuration file as to what will, or will not, be built. If you have some things set as modules, they can be built as a module, but the ashmem code can not be built as a module, so you would never build it if you did the above line. Here, look at this sequence, starting with a tree that does nothing if I do a simple 'make' in it, as the whole kernel is already built, and ashmem is enabled in the kernel configuration $ grep ASHMEM .config CONFIG_ASHMEM=y $ make $ So, let's change the time stamp on the ashmem.c file and see what gets built if you use the M= option: $ touch drivers/staging/android/ashmem.c $ make M=drivers/staging/android MODPOST drivers/staging/android/Module.symvers $ Nothing gets built as ashmem is NOT a module, and M= only builds any modules in the directory you specified. But, if you tell make to just build the whole subdirectory, no matter what the setting is, it will be built: $ make drivers/staging/android/ CALL scripts/checksyscalls.sh CALL scripts/atomic/check-atomics.sh DESCEND objtool CC drivers/staging/android/ashmem.o AR drivers/staging/android/built-in.a $ So that's the difference, "M=" builds modules in that directory, but if you tell it to build the subdir, everything in there that needs to be built, will be built. Be careful about your kernel configuration, that is the key for what will, and will not, be built. Perhaps you should look at the book, "Linux Kernel in a Nutshell" that is free online. It talks all about building and configuring a kernel. Parts of it are out of date, but the general ideas are good. hope this helps, greg k-h