Received: by 2002:a05:6a10:9afc:0:0:0:0 with SMTP id t28csp273330pxm; Wed, 2 Mar 2022 15:08:23 -0800 (PST) X-Google-Smtp-Source: ABdhPJyQf4yxFlNSTFRmjhvk6k6ketGylEDyE3ngLV5Q2L+05YHCuQ/V6RHnahiW5cC5Olxp96gi X-Received: by 2002:a17:902:f606:b0:14b:4c2d:e1fa with SMTP id n6-20020a170902f60600b0014b4c2de1famr32717015plg.24.1646262502896; Wed, 02 Mar 2022 15:08:22 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1646262502; cv=none; d=google.com; s=arc-20160816; b=wIfAf/BxonOx+042sUWJQ4qnbVXnAhGM7Eg6Skvs62FmoCGpU4UTcsLKWKkpEtKQ8w ZWr5cVfK1UdYwG5Ftx4mNdGcFCPOVkBxuDX+uBYOecS1q2MoeTu+Q82IlYQP1mz9LmEg McFWQKA3vzhk59IgMfSSt0nRT1ZJHJ143n3Vvqx4+Sn3wu8Bi47MCpqPPz5zkKtfZoHE uJRVx3BEozf19WcMmWuVTAZSbYp2YBoAWa94NECBCj0TTqWC6aij3nbD3FQToM3rN3vO JfSRth8rWkBKL4D85tanlJAQ5Our8QTRjRHsRfJ7wUVYFURLxuY09pTes+Li8d2Xfha5 Zmpw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=yDbfU47jWNOjxRm+g55MeVjOG1X552M4JoV2V3U9LQI=; b=Um3BDDK49fClgI3tOvtWU4K48zNw+9trXhcS0HbWIHjqdhLHOo+ZleZehFYmp9AeMo DDJVwWXhUyPK30MIs7oPvNXLnax7G3ypCS44r1zgO/D6vo/fo65Gr1LdlZIHJajqMyaC OT5K2ryahdgsfPBm0Kn7gCINqpVFJxnhGb7FSaXxpsdBck6GMJrB38wFcjG2demHXfWV tmxpxF+ZMzWOWmFDTO59nuMBpWsIlglQJNOr1UYeKNYuUGyGDmD1Ba1INprhsx/+v+RY bEKRO9/XcRgrR1DA5IekDJ2Dua0V8vg9siDTV1PQipkFooXUiim4i/XF6iV2hIYLZU1F bFSg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=tjW7FbHV; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 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 lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [23.128.96.19]) by mx.google.com with ESMTPS id e8-20020a170902784800b0014fe7092ecbsi331060pln.82.2022.03.02.15.08.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Mar 2022 15:08:22 -0800 (PST) Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=tjW7FbHV; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C7C834D9CA; Wed, 2 Mar 2022 14:49:23 -0800 (PST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238737AbiCBAMm (ORCPT + 99 others); Tue, 1 Mar 2022 19:12:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43320 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232067AbiCBAMl (ORCPT ); Tue, 1 Mar 2022 19:12:41 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BEE0B5004B for ; Tue, 1 Mar 2022 16:11:56 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4C5F961510 for ; Wed, 2 Mar 2022 00:11:56 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5E5A4C340EE; Wed, 2 Mar 2022 00:11:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1646179915; bh=I2vZtwuFZPbmioJOa3mkvX78X/wdYqIxDTqM6MMnvi8=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=tjW7FbHVKn+IK4cq3vBDTCRzC1B5K+dKjory6f3iLNmAmVFU9wIQYGIJ8yeP4R31/ TXAiTGcGNee8GBxKVcJMpcv7QE5cORi3Je6A2+NrlDUwXwdo8dg8yzEHeYz6rDciYq bWuSfDyDy7kx6gez26Lud1J1utPj0mrvR7oYQdGWWvBEPhZxYH7upsoZk4oilKD4wY eQi6srWcwcclj259Hbjfo93dxKXU7k7G18ENlEHNkznQbtCap9VO9nk1cTXC2uMKcF 1WHDp8xkJH9okbP1arFmZ01BGREwu7WynozTi/4VLFiAF8e4bHrpMCMuas3tyQbExL ZHZPl9KnzEiRg== Date: Wed, 2 Mar 2022 09:11:50 +0900 From: Masami Hiramatsu To: Peter Zijlstra Cc: x86@kernel.org, joao@overdrivepizza.com, hjl.tools@gmail.com, jpoimboe@redhat.com, andrew.cooper3@citrix.com, linux-kernel@vger.kernel.org, ndesaulniers@google.com, keescook@chromium.org, samitolvanen@google.com, mark.rutland@arm.com, alyssa.milburn@intel.com, mbenes@suse.cz, rostedt@goodmis.org, alexei.starovoitov@gmail.com, naveen.n.rao@linux.vnet.ibm.com Subject: Re: [PATCH v2 15/39] x86/ibt,kprobes: Fix more +0 assumptions Message-Id: <20220302091150.21daa1b3f153a98206ee8d9a@kernel.org> In-Reply-To: References: <20220224145138.952963315@infradead.org> <20220224151322.892372059@infradead.org> <20220228150705.aab2d654b973109bab070ffe@kernel.org> <20220228232513.GH11184@worktop.programming.kicks-ass.net> <20220301114905.e11146ad69d6e01998101c3b@kernel.org> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.32; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,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 lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 1 Mar 2022 09:28:49 +0100 Peter Zijlstra wrote: > On Tue, Mar 01, 2022 at 11:49:05AM +0900, Masami Hiramatsu wrote: > > > +static kprobe_opcode_t * > > > +_kprobe_addr(kprobe_opcode_t *addr, const char *symbol_name, > > > + unsigned long offset, bool *on_func_entry) > > > { > > > if ((symbol_name && addr) || (!symbol_name && !addr)) > > > goto invalid; > > > > > > if (symbol_name) { > > > + /* > > > + * Input: @sym + @offset > > > + * Output: @addr + @offset > > > + * > > > + * NOTE: kprobe_lookup_name() does *NOT* fold the offset > > > + * argument into it's output! > > > + */ > > > addr = kprobe_lookup_name(symbol_name, offset); > > > > Hmm, there are 2 issues. > > > > - the 'addr' includes the 'offset' here. > > AFAICT it doesn't (I ever wrote that in the comment on top). There's two > implementations of kprobe_lookup_name(), the weak version doesn't even > use the offset argument, and the PowerPC implementation only checks for > !offset and doesn't fold it. Oops, OK. > > > - the 'offset' is NOT limited under the symbol size. > > (e.g. symbol_name = "_text" and @offset points the offset of target symbol from _text) > > > > This means we need to call kallsyms_lookup_size_offset() in this case too. > > I'm feeling we should error out in that case. Using sym+offset beyond > the limits of sym is just daft. No, this is required for pointing some local scope functions, which has same name. (And perf-probe does that) > > But if you really want/need to retain that, then yes, we need that > else branch unconditionally :/ Thank you, -- Masami Hiramatsu