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=-9.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 75AAAC47082 for ; Mon, 7 Jun 2021 15:12:07 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 3ECDD610FB for ; Mon, 7 Jun 2021 15:12:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3ECDD610FB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vWBE8OwraM2fp/AbqpDsiE8LkgRMUIfj6npfdEKd82Q=; b=jBEfSna9crWZMG yCSCSz1CmEQogTHBjmKeyyl9UnzmWlhXSHi4XEvs7SNJ/BDjydqKZf9VGhBfB9Bx7ggym79MAQR1Y kOb8tWmZZitbWcd8KsO3aIycEh0q/spmRCZVIQ7EIDco4VZDNIH1GIItjCy9MqwhWQXSRFK3QX0n+ S1N8Ywo0Ax4Naq5v8bGe7DUAsldJTIcAXbruOV/nbTkdtgrBc9tfCm9DtKvqyy0bW4VaCRVt056mT MlQnuqFg9fqbcyg43sZLe83Ztjf/wZDoHSgbwBJ3a3PnMzuR1gDw/WQQIUFyRX/a8lD0Ai+o8NH/E 0H7YhnCdlRKVoupPla8w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqGtT-004ADT-Gr; Mon, 07 Jun 2021 15:10:23 +0000 Received: from mail-pg1-f177.google.com ([209.85.215.177]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqGtN-004ABM-0R for linux-arm-kernel@lists.infradead.org; Mon, 07 Jun 2021 15:10:21 +0000 Received: by mail-pg1-f177.google.com with SMTP id e22so14002787pgv.10 for ; Mon, 07 Jun 2021 08:10:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=CyhSA0HZXSwzbxjylXSnlqEooN36dMYeyMrQufwloxc=; b=Qt1UPicDa+UU2VmOY0pJBQ259OpaqgZjE30H32TDyaaWKun4/1AbDZWdhroZZRbKtz vu1OWUPEuVphQY7wDVdqQ2FMKBq0HWmYsISn4BlbZYvtvpb7ZtqqvRew0THvyugcxuOD 0ZvzOzbIRSx3asMgBJlrQlmOy7Q93+cqRhqN8apFl/d39bo/rvyQWiVRzJR5YGV22zSN E7H+b1dYWEpUwhX+2S/6c2RGKCwJ6T7gsuuqL1W9QRc7XtHco9hXcO8XJ2h639FDP9vu Fnbsy2MKdEGfHq9tGsMIMT+SfjxS940KnLev9WoELaKQwsyKvIhPtOav3wmfwmoGJ936 Y7ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=CyhSA0HZXSwzbxjylXSnlqEooN36dMYeyMrQufwloxc=; b=Yz4PZqTiceQeG1KsN0bRWclgkN4EYKWH8q/KwpZztBMrVE2VhvOmkXubjxdEpHfLAL /FXnvWI7mKJbz5i1u37YweazLzB2BbcrjSg0Aon38OjcN96HjLIgeoJ/xSEeiSafqWNA n+ssPpu+eyWYbty3wlmleMvfAaqJvzKT0rr5vCsMcvhYwO1dyd+ej7vIfwW5Do3oafdp d/NQd7bUo2RECTD01Y7+2vouLJA5vUMi4k2s6iJZyCVrfUar7uEFLrf2LpOrV+cBOm99 wzR6ou0a4gGM3gz52LUVr62b6uuzDGmhjGypUE2tees8a6xM8FIA3rGRQAjq0qNXDH3/ iz2g== X-Gm-Message-State: AOAM5304aMhIi2kNOrA9fPLqpKL2A4NUUgI/t+GAokpquS90wjv23oNC sLg0qv5YdK1ZWN1ZhatKrH6OTg== X-Google-Smtp-Source: ABdhPJxOJVs7oeh8RVkMB2oNzC96Z1htkLVbcTpKTHChVCIqHYTE0GAU1CZ61m6jdGE8/kcmtk/YiA== X-Received: by 2002:aa7:8d82:0:b029:2ec:a73b:aff9 with SMTP id i2-20020aa78d820000b02902eca73baff9mr11722166pfr.51.1623078554412; Mon, 07 Jun 2021 08:09:14 -0700 (PDT) Received: from leoy-ThinkPad-X240s (ec2-18-167-84-74.ap-east-1.compute.amazonaws.com. [18.167.84.74]) by smtp.gmail.com with ESMTPSA id n6sm9071583pgm.79.2021.06.07.08.09.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 07 Jun 2021 08:09:13 -0700 (PDT) Date: Mon, 7 Jun 2021 23:09:03 +0800 From: Leo Yan To: Adrian Hunter Cc: Arnaldo Carvalho de Melo , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Thomas Gleixner , x86@kernel.org, "H. Peter Anvin" , Mathieu Poirier , Suzuki K Poulose , Mike Leach , Andi Kleen , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org, coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 8/8] perf record: Directly bail out for compat case Message-ID: <20210607150903.GC1071897@leoy-ThinkPad-X240s> References: <20210602103007.184993-1-leo.yan@linaro.org> <20210602103007.184993-9-leo.yan@linaro.org> <20210602123847.GE10272@leoy-ThinkPad-X240s> <96e5fac6-17a2-ea03-9b15-338b84321ecf@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <96e5fac6-17a2-ea03-9b15-338b84321ecf@intel.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210607_081017_076042_23219651 X-CRM114-Status: GOOD ( 36.66 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jun 07, 2021 at 01:23:43PM +0300, Adrian Hunter wrote: > On 2/06/21 3:38 pm, Leo Yan wrote: > > Hi Adrain, > > > > On Wed, Jun 02, 2021 at 02:18:47PM +0300, Adrian Hunter wrote: > >> On 2/06/21 1:30 pm, Leo Yan wrote: > >>> Since the 64-bit atomicity is not promised in 32-bit perf, directly > >>> report the error and bail out for this case. > >>> > >>> Now only applies on x86_64 and Arm64 platforms. > >>> > >>> Suggested-by: Adrian Hunter > >> > >> Maybe we can do better for the compat case. > >> > >> We can assume the upper 32-bits change very seldom, > >> and always increase. So for the 'read' case: > >> > >> u64 first, second, last; > >> u64 mask = (u64)((u32)-1) << 32; > >> > >> do { > >> first = READ_ONCE(pc->aux_head); > >> rmb(); > >> second = READ_ONCE(pc->aux_head); > >> rmb(); > >> last = READ_ONCE(pc->aux_head); > >> } while ((first & mask) != (last & mask)); > >> return second; > >> > >> For the write case, we can cause a fatal error only if the new > >> tail has non-zero upper 32-bits. That gives up to 4GiB of data > >> before aborting: > >> > >> if (tail & mask) > >> return -1; > >> smp_mb(); > >> WRITE_ONCE(pc->aux_tail, tail); > > > > Seems to me, it's pointless to only support aux_head for 64-bit and > > support aux_tail for 32-bit. I understand this can be helpful for the > > snapshot mode which only uses aux_head, but it still fails to support > > the normal case for AUX ring buffer using 64-bit head/tail. > > I am not sure why you say it is pointless. 'perf record' would still be > able to capture up to 4GiB of data. Do you mean you usually capture more > than 4GiB of data? Okay, understand. We can support 32-bit perf for compat mode when the trace data is less than 4GiB. > I was thinking we would separate out the compat case: > > #if BITS_PER_LONG == 32 > if (kernel_is_64_bit) > return compat_auxtrace_mmap__[read_head/write_tail]() > #endif > > So the non-compat cases would not be affected. Because I don't want to introduce the complexity for read/write head and tail, and we also need to handle the same issue for the perf ring buffer. So how about below change? The main idea for below change is it allows the perf to run normally on the compat mode and exitly if detects the buffer head is close to the low 32-bit's overflow: when detect the low 32-bit value is bigger than 0xf0000000 (so we have 256MiB margin to the overflow), it reports error and exit. diff --git a/tools/perf/util/auxtrace.c b/tools/perf/util/auxtrace.c index 1b4091a3b508..2a9965bfeab4 100644 --- a/tools/perf/util/auxtrace.c +++ b/tools/perf/util/auxtrace.c @@ -1693,6 +1693,14 @@ static int __auxtrace_mmap__read(struct mmap *map, pr_debug3("auxtrace idx %d old %#"PRIx64" head %#"PRIx64" diff %#"PRIx64"\n", mm->idx, old, head, head - old); +#ifdef BITS_PER_LONG == 32 + if (kernel_is_64bit() && head >= 0xf0000000) { + pr_err("32-bit perf cannot read 64-bit value atomically;\n"); + pr_err("exit to avoid the 4GB (32-bit) AUX buffer overflow on compat mode.\n"); + return -ENOMEM; + } +#endif + if (mm->mask) { head_off = head & mm->mask; old_off = old & mm->mask; diff --git a/tools/perf/util/env.c b/tools/perf/util/env.c index 9130f6fad8d5..823b69895b85 100644 --- a/tools/perf/util/env.c +++ b/tools/perf/util/env.c @@ -405,3 +405,20 @@ int perf_env__numa_node(struct perf_env *env, int cpu) return cpu >= 0 && cpu < env->nr_numa_map ? env->numa_map[cpu] : -1; } + +int perf_kernel_is_64bit(void) +{ + struct utsname uts; + int ret; + + ret = uname(&uts); + if (ret < 0) + return 0; + + if (!strncmp(uts.machine, "x86_64", 6) || + !strncmp(uts.machine, "aarch64", 7) || + !strncmp(uts.machine, "arm64", 5)) + return 1; + + return 0; +} diff --git a/tools/perf/util/env.h b/tools/perf/util/env.h index ca249bf5e984..c6c034fc08f6 100644 --- a/tools/perf/util/env.h +++ b/tools/perf/util/env.h @@ -147,4 +147,6 @@ void perf_env__insert_btf(struct perf_env *env, struct btf_node *btf_node); struct btf_node *perf_env__find_btf(struct perf_env *env, __u32 btf_id); int perf_env__numa_node(struct perf_env *env, int cpu); + +int perf_kernel_is_64bit(void); #endif /* __PERF_ENV_H */ diff --git a/tools/perf/util/mmap.c b/tools/perf/util/mmap.c index ab7108d22428..f1d3725d599a 100644 --- a/tools/perf/util/mmap.c +++ b/tools/perf/util/mmap.c @@ -323,6 +323,14 @@ int perf_mmap__push(struct mmap *md, void *to, if (rc < 0) return (rc == -EAGAIN) ? 1 : -1; +#ifdef BITS_PER_LONG == 32 + if (kernel_is_64bit() && head >= 0xf0000000) { + pr_err("32-bit perf cannot read 64-bit value atomically;\n"); + pr_err("exit to avoid the 4GB (32-bit) buffer overflow on compat mode.\n"); + return -ENOMEM; + } +#endif + size = md->core.end - md->core.start; if ((md->core.start & md->core.mask) + size != (md->core.end & md->core.mask)) { Thanks, Leo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel