Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp3598308pxv; Mon, 5 Jul 2021 00:52:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJykst5Q7/GKyclUHmYlQjJPIz2MtHpBxU+eFFxcEoM3LouzxnfSv6r7RZmsTOLuNh53CcT3 X-Received: by 2002:aa7:cb84:: with SMTP id r4mr14419155edt.205.1625471572319; Mon, 05 Jul 2021 00:52:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1625471572; cv=none; d=google.com; s=arc-20160816; b=AXdRMh1UaJGLVulvIGvzzY+/508Nd1CDXElDjD2QvPf/vvpRmSM6lfljS5GTq1EJSO H4AF4qif2B+vyFkUM45krDnK/znnbOWXdS9q64DXrJOY8AHtbWee9GdButNnMjm72z4q bJyvoaKattaNcQ9xgRvQovV3CqHV0ighE9YmSxvjFkgYF5adfFSrrA2Ky/84l7YrsUge +3ZfOTgn0FOoNMep0Ux3Cv8aMlXfQBodxZ03FEwASKBaNq5eVuz2zWpb0x/RsUXlREID M0ybB/IhLNa6mUgZABfxqtMhrX3VCdmSTAWs9Fpi2gRFi/gOxu2Gbxeby7apMvFFPFDR Kcdw== 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:sender:dkim-signature; bh=d0dQZRJ3T9P2oADq6Txm5lSu38y0NAynZvCdSK4/Cao=; b=ThJA6oVCtj4OMfzouJxsssxXWMy67leYkAd2qBryaHX5yZ0llbPrnbGt8FR5PM66og jK1twS0inOET5vN5dJtHVHiygbex2kTDYn+YrwH0lZjLTy36wcGlHZKkbuHOoTuCei0v Pvn6pRuPzPew4p1iLdT77u9LaYoIb+SYhXsOTycUDzoK4tpKGUMvFOkIsdEDVh86j9Q+ bT0hUl/R8jjgeX9ML8bP3ZTLXmsPXQpR9sfxFAKjxx3amad0KfkQ9nk4nSg55CRseuQx Ot4PXS1sFFlAwti6A8JJLVgcKz7s0fHv2WUQsqKATrazwac18TFNU2WTtNVAsiA0nyH+ PkPg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=HLcDlujD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 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 vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 3si10167974ejr.392.2021.07.05.00.52.28; Mon, 05 Jul 2021 00:52:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=HLcDlujD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230026AbhGEHwP (ORCPT + 99 others); Mon, 5 Jul 2021 03:52:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47192 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229884AbhGEHwO (ORCPT ); Mon, 5 Jul 2021 03:52:14 -0400 Received: from mail-wr1-x42b.google.com (mail-wr1-x42b.google.com [IPv6:2a00:1450:4864:20::42b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1B712C061574; Mon, 5 Jul 2021 00:49:38 -0700 (PDT) Received: by mail-wr1-x42b.google.com with SMTP id a8so9185848wrp.5; Mon, 05 Jul 2021 00:49:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=d0dQZRJ3T9P2oADq6Txm5lSu38y0NAynZvCdSK4/Cao=; b=HLcDlujDpRoNX7m9HZff0LdF8NBRLnSnmM0Uhbag1lNq/wa9op2N3WcnUEziL6gRF8 ZVtqxftQOEBMunfMZy5jAJ2WWwgDZZU1w0ILeou5SeuN+ft032yhkiY1XpVwdhTkTG7i ZDtDvmQFiDdPeg/cW5wpx2aJ0YUV3UDrns/Ew8sIw5T2XoHwjzOFXIR22nekXkZ4qby0 2kun0p5dGvh6B+RiM48fmtXjlMsaY18nqpLV2IQUqG9hC0U98VZy9A2G6tv8GwXXx2CA VnczIipniTEtJtHv15OObIpDhVb4ax8boHKHMCjr0qbIKQIc14apJ6nyoUuZ/sY3U6kF CS+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to; bh=d0dQZRJ3T9P2oADq6Txm5lSu38y0NAynZvCdSK4/Cao=; b=B1ocmmUUe24zQuNfqpcAmcSzLv3BaJ4nral0DswV5E8KWIkERkUWbA5o4rzu60xLNr QjGhE6W5jlvh6ZFzC4G7YUvlpr4RcYwjtMZzoTjBjp34JlYSDLfU4NZOBuE1UJxmvC/m G0ep2Bz1Vvf1n2AvfMrSovq28e7t8eA3AowuINEMJ1SMOSd50j6xTcESPbVe3QWJ7/TN 1A2sekPmngKMSIBBHNXaUpAOSatCHDyNJxiX545J/XGnEgTTTAf3DAFZR5d7Yi5lSzpM 6Ijht+0lRPGhZxkLr4hEgAFH5f1pikZp1aB1Wkw1ckbS1+ehc2ocJCcQReAqCh6J7QIn 6Kbw== X-Gm-Message-State: AOAM5318TpqjR5Ku8dL0sN2ZuOBo+pPq//iy2inokrNgjHeUvMAy850z /IzNbHB42Fq8cY938aPhsL0= X-Received: by 2002:adf:a4d8:: with SMTP id h24mr14010038wrb.416.1625471376798; Mon, 05 Jul 2021 00:49:36 -0700 (PDT) Received: from gmail.com (178-164-188-14.pool.digikabel.hu. [178.164.188.14]) by smtp.gmail.com with ESMTPSA id y13sm12287630wrp.80.2021.07.05.00.49.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 05 Jul 2021 00:49:36 -0700 (PDT) Sender: Ingo Molnar Date: Mon, 5 Jul 2021 09:49:34 +0200 From: Ingo Molnar To: Masami Hiramatsu Cc: Steven Rostedt , Josh Poimboeuf , X86 ML , Daniel Xu , linux-kernel@vger.kernel.org, bpf@vger.kernel.org, kuba@kernel.org, mingo@redhat.com, ast@kernel.org, Thomas Gleixner , Borislav Petkov , Peter Zijlstra , kernel-team@fb.com, yhs@fb.com, linux-ia64@vger.kernel.org, Abhishek Sagar , Andrii Nakryiko Subject: Re: [PATCH -tip v8 03/13] kprobes: treewide: Remove trampoline_address from kretprobe_trampoline_handler() Message-ID: References: <162399992186.506599.8457763707951687195.stgit@devnote2> <162399994996.506599.17672270294950096639.stgit@devnote2> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <162399994996.506599.17672270294950096639.stgit@devnote2> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Masami Hiramatsu wrote: > Remove trampoline_address from kretprobe_trampoline_handler(). > Instead of passing the address, kretprobe_trampoline_handler() > can use new kretprobe_trampoline_addr(). > > Signed-off-by: Masami Hiramatsu > Tested-by: Andrii Nakryik A better changelog: The __kretprobe_trampoline_handler() callback, called from low level arch kprobes methods, has the 'trampoline_address' parameter, which is entirely superfluous as it basically just replicates: dereference_kernel_function_descriptor(kretprobe_trampoline) In fact we had bugs in arch code where it wasn't replicated correctly. So remove this superfluous parameter and use kretprobe_trampoline_addr() instead. Thanks, Ingo