Message ID | 20230105090155.357604-1-irogers@google.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:4e01:0:0:0:0:0 with SMTP id p1csp200706wrt; Thu, 5 Jan 2023 01:04:54 -0800 (PST) X-Google-Smtp-Source: AMrXdXuvmlfitzHgo6zcc62tFxkOYgsq8tFDJyCLCgj4X1ncTBUTVp55qCOU05vylXV6rErysXau X-Received: by 2002:a17:906:7187:b0:7bd:f540:9bf9 with SMTP id h7-20020a170906718700b007bdf5409bf9mr46362243ejk.32.1672909494354; Thu, 05 Jan 2023 01:04:54 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1672909494; cv=none; d=google.com; s=arc-20160816; b=un4hU04ST48mzGnb20BcLwaVa5ea1WeMR3/87TzZHWueec6wIsKSUbCH57RhkBBZ42 JKbZ1/4vE0T6s5CdKjjFUSVM/CbkNEor5GSAzH1vlajZuCF92LzFAigwZjt6BrqNJ2Kz Y5L2SoyH3oQXC2umcUC6oqoOuFHM+GKvdhpKfLFSQUkFoo5iCQq8VCNW/l0XVmpri2ZX NqmraBVHQYnXe1AbUz5GW6TVcfwMmQ8GAlA42s2jixQcLyMpHfWrM8h8D0kzoc2flvcB Hzne3t8ar8fUYlGHkrWeJwjec8plQXvGMwlgD8cE+zZdSA/9483t3gYTZV775LT7B/Vj Tn3A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:from:subject:mime-version:message-id:date :dkim-signature; bh=gMeOMpb58LKwXdrW/getzyijMXrw75Kv1TqJV/9UOLk=; b=z2oEBURk0VIk7KdivcfYzF/CO7FiAgDAFX4tx6TXFhW8dz2CEAwcC6jXyK0usROeZ9 A+dY8bmPxxRsa712h62ZOMOMjqPgnflNhtL+wy+jRwm9IJM0fBK6aUFK5KReaqh1E3Io PUpzYFDhCb5gPoOBsbfHpPAq0havQ7gyQnvgAUsfvejggCcU88cOc6vDnO/4ndj2FuVu 3/OiUL1cw2qAcXNCZrxVLXcIMuZLsbEHDaIPxz+HEcM0OBEAVihL+7VVrk59HUrU5Xe+ REFSeQM/d6wuUhLeoFmKcXkE9Hp3JRzMPRMef57TIp67OISVkba/rTbE0PVblKVCbZIW FI4w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=MK3MwIy9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id dr5-20020a170907720500b007804b5a2c48si32336512ejc.521.2023.01.05.01.04.31; Thu, 05 Jan 2023 01:04:54 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=MK3MwIy9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231802AbjAEJCX (ORCPT <rfc822;tmhikaru@gmail.com> + 99 others); Thu, 5 Jan 2023 04:02:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52452 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231776AbjAEJCJ (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Thu, 5 Jan 2023 04:02:09 -0500 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5543D50073 for <linux-kernel@vger.kernel.org>; Thu, 5 Jan 2023 01:02:08 -0800 (PST) Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-45a51c37009so375241547b3.17 for <linux-kernel@vger.kernel.org>; Thu, 05 Jan 2023 01:02:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:mime-version:message-id:date:from:to:cc:subject :date:message-id:reply-to; bh=gMeOMpb58LKwXdrW/getzyijMXrw75Kv1TqJV/9UOLk=; b=MK3MwIy9qtEF2d12ORUyteoXB3r3fdkipI3neUqhPW9L4iFXR8Y9eHgFhPnQBPKEyw CcbA1i9gWaUKDLzVFLTm/qz1vlBeEaew0jj9T73xH0niNlDmQ1mVExHOxxAB3a6TMg3W XIwqUkTM0qZUBUZ24QeLdi0L+o7Cv1E5tgWd2Lr/5mNa5F6QdUfiLjhwlLQzxV1YyJah ljfXzChmqwm6Xj9Ti4BAXm0mIpkZm0Fe95ARzPSzpjwGJ9gNjqWIneqBSuqY3e7gHFQN uuO4+FhlSZVCaXWR45Qn4IVzqp2QfPwzlTdEIhD7BJJe7+Fru4bJ4XUo13EVq/6FxUu6 4kEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:mime-version:message-id:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=gMeOMpb58LKwXdrW/getzyijMXrw75Kv1TqJV/9UOLk=; b=EZUxcVkHzXLXIC1gWfNNxoQvux8EBa+eokCcvcEwvMlrd+sSvj4eci5wA/sDWCrrER PLvEj/5HUKsLaN5xDNIhvdW2fW+fXS1cCcSGwAVCziMPkHsSAGgoS8y5XPh/swUCR31d 0F7j7U8cBfKrH08RcP838qRVXAZz6Na+0eIfLHrgSbfNfDmm4dIEer/jIjBMLlxTyASD +kZx8QI2kCFUmTsIKAQg9cM0akGhaOCnQwbIwSbKVPVsrcOTx4OFOc8WMULloye7QFcb REm9tuPKYVDVSdLWgsMYAY6gTKftoJJSdeGdTlXtIooyFC4njKlygSW9diO89Ds4vhhI DdZw== X-Gm-Message-State: AFqh2ko2sy9+XIlmzUvpUbeOnnXUPs9PGzW8yM02/5BNL0AoD39iXaFk BSicpPbqJl+sR/dulmgnoWZ5zsLsb7Op X-Received: from irogers.svl.corp.google.com ([2620:15c:2d4:203:8775:c864:37e:2f9b]) (user=irogers job=sendgmr) by 2002:a05:690c:886:b0:367:27b7:af89 with SMTP id cd6-20020a05690c088600b0036727b7af89mr83353ywb.292.1672909327417; Thu, 05 Jan 2023 01:02:07 -0800 (PST) Date: Thu, 5 Jan 2023 01:01:52 -0800 Message-Id: <20230105090155.357604-1-irogers@google.com> Mime-Version: 1.0 X-Mailer: git-send-email 2.39.0.314.g84b9a713c41-goog Subject: [PATCH v3 0/3] objtool build improvements From: Ian Rogers <irogers@google.com> To: Josh Poimboeuf <jpoimboe@kernel.org>, Peter Zijlstra <peterz@infradead.org>, Nathan Chancellor <nathan@kernel.org>, Nick Desaulniers <ndesaulniers@google.com>, Tom Rix <trix@redhat.com>, Masahiro Yamada <masahiroy@kernel.org>, Nicolas Schier <nicolas@fjasle.eu>, linux-kernel@vger.kernel.org, llvm@lists.linux.dev Cc: Stephane Eranian <eranian@google.com>, Andrii Nakryiko <andrii@kernel.org>, Jiri Olsa <jolsa@kernel.org>, Arnaldo Carvalho de Melo <acme@kernel.org>, Namhyung Kim <namhyung@kernel.org>, Ian Rogers <irogers@google.com> Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1754172746108016983?= X-GMAIL-MSGID: =?utf-8?q?1754172746108016983?= |
Series |
objtool build improvements
|
|
Message
Ian Rogers
Jan. 5, 2023, 9:01 a.m. UTC
Install libsubcmd and then get headers from there, this avoids inadvertent dependencies on things in tools/lib. Fix V=1 support. Clean up how HOSTCC is used to override CC to avoid CFLAGS being set for say gcc, and then CC being overridden to clang. Support HOSTCFLAGS as a make option. v3. Is a rebase that removes the merged "tools lib subcmd: Add install target" patch. In: https://lore.kernel.org/lkml/CAKwvOd=kgXmpfbVa1wiEvwL0tX3gu+dDTGi-HEiRXSojwCLRrg@mail.gmail.com/ Nick rightly points out that: WARNINGS := $(EXTRA_WARNINGS) -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs became: WARNINGS := -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs losing the EXTRA_WARNINGS which v3 now adds back in. Previous testing had added the warnings to the end rather than the beginning, thereby causing unexpected build issues that aren't present in v3. v2. Include required "tools lib subcmd: Add install target" that is already in Arnaldo's tree: https://lore.kernel.org/lkml/20221109184914.1357295-3-irogers@google.com/ https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=perf/core&id=630ae80ea1dd253609cb50cff87f3248f901aca3 When building libsubcmd.a from objtool's Makefile, clear the subdir to avoid it being appended onto OUTPUT and breaking the build. Ian Rogers (3): objtool: Install libsubcmd in build objtool: Properly support make V=1 objtool: Alter how HOSTCC is forced tools/objtool/Build | 2 -- tools/objtool/Makefile | 66 ++++++++++++++++++++++++++++++------------ 2 files changed, 47 insertions(+), 21 deletions(-)
Comments
On Thu, Jan 5, 2023 at 1:02 AM Ian Rogers <irogers@google.com> wrote: > > Install libsubcmd and then get headers from there, this avoids > inadvertent dependencies on things in tools/lib. Fix V=1 > support. Clean up how HOSTCC is used to override CC to avoid CFLAGS > being set for say gcc, and then CC being overridden to clang. Support > HOSTCFLAGS as a make option. > > v3. Is a rebase that removes the merged "tools lib subcmd: Add install > target" patch. In: > https://lore.kernel.org/lkml/CAKwvOd=kgXmpfbVa1wiEvwL0tX3gu+dDTGi-HEiRXSojwCLRrg@mail.gmail.com/ > Nick rightly points out that: > WARNINGS := $(EXTRA_WARNINGS) -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs > became: > WARNINGS := -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs > losing the EXTRA_WARNINGS which v3 now adds back in. Previous > testing had added the warnings to the end rather than the > beginning, thereby causing unexpected build issues that aren't present in v3. > v2. Include required "tools lib subcmd: Add install target" that is > already in Arnaldo's tree: > https://lore.kernel.org/lkml/20221109184914.1357295-3-irogers@google.com/ > https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=perf/core&id=630ae80ea1dd253609cb50cff87f3248f901aca3 > When building libsubcmd.a from objtool's Makefile, clear the > subdir to avoid it being appended onto OUTPUT and breaking the > build. > > Ian Rogers (3): > objtool: Install libsubcmd in build > objtool: Properly support make V=1 > objtool: Alter how HOSTCC is forced > > tools/objtool/Build | 2 -- > tools/objtool/Makefile | 66 ++++++++++++++++++++++++++++++------------ > 2 files changed, 47 insertions(+), 21 deletions(-) Ping. Relatively small set of patches, with Reviewed-by and Tested-by, would be nice to land. Thanks! Ian > -- > 2.39.0.314.g84b9a713c41-goog >
On Thu, Jan 12, 2023 at 9:41 AM Ian Rogers <irogers@google.com> wrote: > > On Thu, Jan 5, 2023 at 1:02 AM Ian Rogers <irogers@google.com> wrote: > > > > Install libsubcmd and then get headers from there, this avoids > > inadvertent dependencies on things in tools/lib. Fix V=1 > > support. Clean up how HOSTCC is used to override CC to avoid CFLAGS > > being set for say gcc, and then CC being overridden to clang. Support > > HOSTCFLAGS as a make option. > > > > v3. Is a rebase that removes the merged "tools lib subcmd: Add install > > target" patch. In: > > https://lore.kernel.org/lkml/CAKwvOd=kgXmpfbVa1wiEvwL0tX3gu+dDTGi-HEiRXSojwCLRrg@mail.gmail.com/ > > Nick rightly points out that: > > WARNINGS := $(EXTRA_WARNINGS) -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs > > became: > > WARNINGS := -Wno-switch-default -Wno-switch-enum -Wno-packed -Wno-nested-externs > > losing the EXTRA_WARNINGS which v3 now adds back in. Previous > > testing had added the warnings to the end rather than the > > beginning, thereby causing unexpected build issues that aren't present in v3. > > v2. Include required "tools lib subcmd: Add install target" that is > > already in Arnaldo's tree: > > https://lore.kernel.org/lkml/20221109184914.1357295-3-irogers@google.com/ > > https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/commit/?h=perf/core&id=630ae80ea1dd253609cb50cff87f3248f901aca3 > > When building libsubcmd.a from objtool's Makefile, clear the > > subdir to avoid it being appended onto OUTPUT and breaking the > > build. > > > > Ian Rogers (3): > > objtool: Install libsubcmd in build > > objtool: Properly support make V=1 > > objtool: Alter how HOSTCC is forced > > > > tools/objtool/Build | 2 -- > > tools/objtool/Makefile | 66 ++++++++++++++++++++++++++++++------------ > > 2 files changed, 47 insertions(+), 21 deletions(-) > > Ping. Relatively small set of patches, with Reviewed-by and Tested-by, > would be nice to land. Thanks! > > Ian Ping2. Small set of patches, 2x Reviewed-by and 1x Tested-by. Thanks, Ian > > > -- > > 2.39.0.314.g84b9a713c41-goog > >