Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp5005429rdb; Tue, 12 Dec 2023 16:27:34 -0800 (PST) X-Google-Smtp-Source: AGHT+IFXO0/tZl/CWq5yPYX/B8Y8cKJFruj4HsGe+8FV3eZZVVeK4CZAq8jbHiB/Pcf/x4585rG5 X-Received: by 2002:a05:6808:6488:b0:3ba:d81:841e with SMTP id fh8-20020a056808648800b003ba0d81841emr4995823oib.55.1702427254575; Tue, 12 Dec 2023 16:27:34 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702427254; cv=none; d=google.com; s=arc-20160816; b=rdvkstwtMtQUesLauMCcD+HKziFlzfo+Z0iUWi4tFbtYyOIQQHujznBH+1ashc2SFN VR34OV2BZQAltR7HXc4d3SjluhVAZKN3DjsHGdqycMyxfRTCisGi6cL2vg9Qjhd+spnb XQ2wC+fWSkGHfPV+s4Ak3c7WGd602bXxr/XpMXan2kaErDurW+Xq68Qc+dRAFArqbXHe trlk8D6dMBwFyU3yaQ811gT0X6GKoHSEEu/BubhI8kKn5KcsXSNk2gbw5ujU4NS1H5bl asRlb7IL3qXvDQaNv5PzFOAsN4KD4CLBbaxLNdj6iOu4ME3MHdMv8uW13uGbvmH5Kk4J WEgQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=uWz5Hy8oRJp3Mg2l9ap4etd1Eqjufz/r5J3YDmQ/a3Q=; fh=xpzMXz8SRrLG+6yY1/aJR4zltJdFY9WdeG3HfqKg7RQ=; b=lDOaDID62ISOviXXLq6sWEr9qMPw9J0CCNlVKMB7YhAQGi18KwVRqqcprGRows3YcP aQY/bkk/26ObzfK9V3rr8t5LAk531ioNE+p+a1WkobjM2JgbAIsDlpl7L5yCEoONbBgz WY/nzzWGfXdE9wpjfPjd2Y3BbsMHdlvAat+f4abbDR0qxudPP3Yb9PiPRu5rLBvShcob GklV+Da7qpsmObrv2sHnKqsoDt6U/2SZAnBWwNY51hPGnqYLHWBguOGkWLVuE4oYW2G5 edtDDezEHx0y7l3quTzKBHeZuoHgRLZn/scIcjC9wi3C/YwfiVvinNL5ijDvzLDo+zPP WPcw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=KcSt3eln; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from groat.vger.email (groat.vger.email. [2620:137:e000::3:5]) by mx.google.com with ESMTPS id 17-20020a630011000000b005c2783b71cfsi8494340pga.810.2023.12.12.16.27.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Dec 2023 16:27:34 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=KcSt3eln; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id 149418049D2C; Tue, 12 Dec 2023 16:27:29 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232606AbjLMA1M (ORCPT + 99 others); Tue, 12 Dec 2023 19:27:12 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33284 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232505AbjLMA1L (ORCPT ); Tue, 12 Dec 2023 19:27:11 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A769192 for ; Tue, 12 Dec 2023 16:27:17 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1AF93C433C7; Wed, 13 Dec 2023 00:27:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702427237; bh=/H2khaMF8vfAJM4ApUDC7NOWw3ck3r3U1TwZFrVK9ms=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=KcSt3eln+BKAmUyTk71p1Gk6qbrqaamO+PJwTOuVkOnIffWJDeE62pLLyzF5/FhUi fsutcvPuqwu5Hqg24mMh/3PIZYYx/IcaaIR860xHfY7BFU7mFYc7gybnZtfp2YbZ8M liIcHj/63BIVsyCESyZK0XofpkNuWXua+5mpfinuRr958yMdB312N89rNSABjwiTe7 aHk3t5Rj8NmqYNVC102m/ereHCjTixuse6KsbFC4NXlCRTVBCqpAmKLuKhjmW29+qD 6Uzdo4WaPrsBYUaLLgtUA49Rbm4bxhZMyB4G2NTjwu/0Dv6NfrW0w90wfYXCcb/NFP NdkHfVNSrK+yg== Date: Tue, 12 Dec 2023 16:27:15 -0800 From: Josh Poimboeuf To: Dimitri John Ledkov Cc: Peter Zijlstra , linux-kernel@vger.kernel.org Subject: Re: [PATCH] objtool: Make objtool check actually fatal upon fatal errors Message-ID: <20231213002715.w5cqo5ataaevivop@treble> References: <20231212185339.1562967-1-dimitri.ledkov@canonical.com> <20231212203031.uudk57xhegi5yziz@treble> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-1.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Tue, 12 Dec 2023 16:27:29 -0800 (PST) On Tue, Dec 12, 2023 at 08:39:30PM +0000, Dimitri John Ledkov wrote: > Well, I'm hoping to add --error option next, and yeah, make warnings > fatal. Or for example at least make the retpoline, sls, rethumb ones > to be fatal - because i found myself in a very surprising situation > where retpoline enabled build of kernel, had non-retpoline paths > remaining and not otherwise silenced as safe. > > Maybe I should finish that second patch and make it available as a > config option. > > At least right now, with Ubuntu Noble (development series) and > v6.7-rc4 everything is squeaky clean to enable CONFIG_OBJTOOL_WERROR=y > and make all warnings, fatal. > > Also i think having it as a config option will result in various > automation tools able to flip it on, and submit bug reports when > something somewhere regresses. Right, the warnings should never be ignored. I agree we need CONFIG_OBJTOOL_WERROR. Problem is, upstream supports a lot more than just Ubuntu configs, and there are several outstanding warnings, reported by both bots and humans. Fixing them is on my TODO list, it's just that other things are taking priority. If we introduce CONFIG_OBJTOOL_WERROR now, build bots will start reporting build failures and people will start complaining more loudly. Without more help I'm lacking the bandwidth to stay on top of that. -- Josh