From patchwork Tue Sep 10 16:23:34 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vincent Donnefort X-Patchwork-Id: 13798867 Received: from mail-wm1-f73.google.com (mail-wm1-f73.google.com [209.85.128.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 36CDE1A01CA for ; Tue, 10 Sep 2024 16:23:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.73 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725985424; cv=none; b=lbouslKZPJ7L7eLSELVqv94yawfIGn4wlH7K2dRzMF9+Z6JFA6VlgejOFuRr4FRBW9vlLgw9LfhziCV4Go2D3+vkHypI/H+7YXmsM7upAM6hCFaFA9PSfSGi4VE0Lnfs/j9kU9zvjPLa7/+2avn0QfvtJxsu3CifWGa9Krtd4zc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725985424; c=relaxed/simple; bh=GlHp+sDP2l3iuMBGJMRzuH4oTR65BYOzxJ6K7GQv29A=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=Va8RXuVDbBix+udYbznpmJFgn4c7hTMuAFGCUW4svvuOcObG8kTOew8wZdGLJfHyco/JqQm5h98XvBSCiDAniuZFhmjUhgwxb/yejy5QTi3YB5UCqD38ALkb2/blamRarVMO47YiONoma9cCtTBfspfAlqYC3Aj/UcyAkq2yMe0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--vdonnefort.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=pmHDTV1L; arc=none smtp.client-ip=209.85.128.73 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--vdonnefort.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="pmHDTV1L" Received: by mail-wm1-f73.google.com with SMTP id 5b1f17b1804b1-42cbfc2aac6so6491945e9.1 for ; Tue, 10 Sep 2024 09:23:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1725985421; x=1726590221; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=ebNtXOw1zw+Y/P3jFG4DVeMhgHXmSDzSdOEI97BWEhQ=; b=pmHDTV1LBzxEn57OPtZ959cM9yCo6FtV14bdmSYOr0i8OOMznK96DIlafADdJD6h1w EvJ+sYMKEw+A2pXMXPwJ09hYcsNOv5YRYXEWFUHFuY/DLt1aw2465I52yWkklKzrCpq/ OVYh23XTnKW3gfyX2aIjagpvWG4GWghMjqp9oYPJ7xragDC1p5z8zmgnzjlKZ5+C+4f6 yq2bByUNbO9N/WbNpWrq1hoR752A8DS4WVPfJUqrijsrsq2VrHuesDze5GMRcX3GBPP4 Validza4xhvrA8IrwCJV1sUJZIgoGNH7XN2wG2F3s2V2ynDecMPZ0tmN4lCvxTM4lb6M 3u4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725985421; x=1726590221; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ebNtXOw1zw+Y/P3jFG4DVeMhgHXmSDzSdOEI97BWEhQ=; b=vqvvHqxkmoAMls3+cRjcq78wmvbuPT2HhyHTS3DZOrNAulfhnhITzYmCrwsMWXCi1p 7hznNTc2DPJ7GvwUE+D0pnzmVJBw2GFoH6p7afT5wPTJOaTmFxYWOhIA18T/HqM42KkZ NRPl4PZuXxoXwPv9ftFmDdjnYXkjuu7qEN4klEZcGPjH5t2wRJEGMkqWe1m7PPjER90u 7otkAGsAaUvcBZNe45i0uS1UGg16iFfJF3WQ4cq+v/9+ScCwFZBU2HthWNKsGuqtv/Ej gsd8JYo12TIaSDoEjcDvo6q1hXScw2vVNeh8iASY2VA44wo5sW/6z7ctukTI2Q6Hkt1p 0irQ== X-Forwarded-Encrypted: i=1; AJvYcCWVq3p1+dyDgz8aalaCglWRmP5yWwgC6YLiI5elEjzpyAVGzKIetEXZMcGHztnJFVB10aHplyhZ8HKWBS3unWU=@vger.kernel.org X-Gm-Message-State: AOJu0Yw0050oUxAUM2hTQ80WAm5C6FyDlXYNzfQbPKY7Nz3VhD0rXjv2 LaK6JF0rzLAHHCjbjPm1FpqF+wZ8fFAko6x96G750u/A9ZPRxSoEMEbVKkfjauw5Oj8pr/ntG3S S4fq/t116QARm48TZaw== X-Google-Smtp-Source: AGHT+IH1o/yd45zaUsI2O3gDVbqqWAfhLEcKZtzUdUtDk58M1Kz36FWtvb8nlXBaGxXyqHyWojvmUOgrKh/T56M4 X-Received: from vdonnefort.c.googlers.com ([fda3:e722:ac3:cc00:28:9cb1:c0a8:2eea]) (user=vdonnefort job=sendgmr) by 2002:a05:600c:880e:b0:42c:b0ef:c70d with SMTP id 5b1f17b1804b1-42cb0efc875mr383485e9.1.1725985421494; Tue, 10 Sep 2024 09:23:41 -0700 (PDT) Date: Tue, 10 Sep 2024 17:23:34 +0100 In-Reply-To: <20240910162335.2993310-1-vdonnefort@google.com> Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240910162335.2993310-1-vdonnefort@google.com> X-Mailer: git-send-email 2.46.0.598.g6f2099f65c-goog Message-ID: <20240910162335.2993310-2-vdonnefort@google.com> Subject: [PATCH RESEND 1/2] ring-buffer/selftest: Verify the entire meta-page padding From: Vincent Donnefort To: rostedt@goodmis.org Cc: mhiramat@kernel.org, linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, kernel-team@android.com, Vincent Donnefort , Shuah Khan , linux-kselftest@vger.kernel.org Improve the ring-buffer meta-page test coverage by checking for the entire padding region to be 0 instead of just looking at the first 4 bytes. Cc: Shuah Khan Cc: linux-kselftest@vger.kernel.org Signed-off-by: Vincent Donnefort Acked-by: Steven Rostedt (Google) Acked-by: Shuah Khan diff --git a/tools/testing/selftests/ring-buffer/map_test.c b/tools/testing/selftests/ring-buffer/map_test.c index 4bb0192e43f3..ba12fd31de87 100644 --- a/tools/testing/selftests/ring-buffer/map_test.c +++ b/tools/testing/selftests/ring-buffer/map_test.c @@ -231,15 +231,15 @@ TEST_F(map, data_mmap) /* Verify meta-page padding */ if (desc->meta->meta_page_size > getpagesize()) { - void *addr; - data_len = desc->meta->meta_page_size; data = mmap(NULL, data_len, PROT_READ, MAP_SHARED, desc->cpu_fd, 0); ASSERT_NE(data, MAP_FAILED); - addr = (void *)((unsigned long)data + getpagesize()); - ASSERT_EQ(*((int *)addr), 0); + for (int i = desc->meta->meta_struct_len; + i < desc->meta->meta_page_size; i += sizeof(int)) + ASSERT_EQ(*(int *)(data + i), 0); + munmap(data, data_len); } } From patchwork Tue Sep 10 16:23:35 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vincent Donnefort X-Patchwork-Id: 13798868 Received: from mail-yw1-f202.google.com (mail-yw1-f202.google.com [209.85.128.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 32F631A01CA for ; Tue, 10 Sep 2024 16:23:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725985426; cv=none; b=GNZzGg8LqEqYIpTJJxZhJnCCwW9xWx9j3etFbyFoADU/86YRwCFJJAio7EYw4yJtSLSYwqSQxG6VUhzVzOzKwx87u2jcItYvJqloTgTBsVFQZnTX9IB5Ec88buS1Az86NbYn1E68b2/UhLURicbLi+X+rBLCG1AFlJzkebbjTIU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725985426; c=relaxed/simple; bh=IDxzP8NIFFLJrT3njglStxjT1iS4xr2kKaPxUTaw2eI=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=GxY6YdK7JmlrXWRGPbDwzAzCRtQA5BXvVaAA5vVWLN6lrzdUEDPV3uKyIaPJOsklGEGDEFUS95yaMbQXTpLa00VX7CHfZst8pAN7O1nk1DAq9d56Uti28X264I4MrvCYCp02Q4fk48ZFINAia+7ajweSVdWxZVakJXOSGteWnuY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--vdonnefort.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=xF04tJ0U; arc=none smtp.client-ip=209.85.128.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--vdonnefort.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="xF04tJ0U" Received: by mail-yw1-f202.google.com with SMTP id 00721157ae682-6d54ab222fcso202634877b3.1 for ; Tue, 10 Sep 2024 09:23:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1725985424; x=1726590224; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=k3gEWXD/IZ+HnaT6C3cDeHppcfA/fBRmYbhkS6oYyo8=; b=xF04tJ0U4KKfInAsQ2RTTKV19JHTtz/h/Ad93cLe89MZ3K6qUDQWFN2V4xAKPgblGp V5mv+JXD0dvp5CINnv+l9SZwq18cbj3CUY+rO6ob+x2M592R4hnj4TLFf4KUltqkuSmk 1tlXmhF6JRgHV7VjrLqpHQ2SvA2EjGDSwFY2VQmT2jRVc8HbJcQs2vWa2+oLkivX7+Et zKRzSVW7q6QqNpXLooUOW+9NPpWwYAspk5NXN50gzF9FE7+UoNVEKq7wYn4ZW0ol1Ryr JnhM3Ya9MxSqsIHnGqPcr7qN3HlTjZsmX+5gk2+UAiozgjClNCqJyaPfviMyXO/VLhdZ Vypg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725985424; x=1726590224; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=k3gEWXD/IZ+HnaT6C3cDeHppcfA/fBRmYbhkS6oYyo8=; b=L6Ghh8a7Yg/bBNTNmGXRY5hqvkK7xuNU5aId4w4y92DG+yIadxXKe5Y1qLGtniKPMB 59xfiapZt7Cob6SW/GRjwPI1S38FlqNuM2UHf1Zaw6G4/j2iTiel99uuBhwZR9Xbtj6o yeU77IOjlW887/vyRJ16tH33awNYE30rcsH8/CDVKzRITOFWvRpMRA/s8SKepDRiYCMk RL/nYcEu6QaDd3p7uPG4ggslNUgzLX62//DQFEay8/BBojxne+cNuQyLwwxxnl81J7GP 92bRLLJwklSNrR8xppMBQ1gfdTwfEcBZ9aLAm/3mIxmcHb6XlDeQgM07UWOgprwASVre 09Mg== X-Forwarded-Encrypted: i=1; AJvYcCXvrEjrKFPr4MxyxHhaiIdmb+2lXavRYCyLbt1dtVdhy4bu8QvP3hKysT/lOBPAvYiKHoic2zL/xvlfsvhhixc=@vger.kernel.org X-Gm-Message-State: AOJu0Yw+j8ETH0i/Bm/sqbLJ+j6Gtnq5ikq776Tyav4mVpFPKl2NW5CI QfYtxZZf0i5NpF/IkzmnZ+yV55fbjgHx/wMiRyZC/iHGU2hi5g6B9/R/6X05TSCOOYdRiqJJsic jNiQw4V/481/c8eSQqA== X-Google-Smtp-Source: AGHT+IFIKA+XvPoOxw+v8DGyTZTQiSOBgMRqezK95cE9yrJOsqwf2B357BaQ0JA4EDgrcvgiuQT7VwrhPTbXI7AW X-Received: from vdonnefort.c.googlers.com ([fda3:e722:ac3:cc00:28:9cb1:c0a8:2eea]) (user=vdonnefort job=sendgmr) by 2002:a05:690c:448a:b0:6db:34db:bc80 with SMTP id 00721157ae682-6db45163fc5mr3107447b3.5.1725985424256; Tue, 10 Sep 2024 09:23:44 -0700 (PDT) Date: Tue, 10 Sep 2024 17:23:35 +0100 In-Reply-To: <20240910162335.2993310-1-vdonnefort@google.com> Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240910162335.2993310-1-vdonnefort@google.com> X-Mailer: git-send-email 2.46.0.598.g6f2099f65c-goog Message-ID: <20240910162335.2993310-3-vdonnefort@google.com> Subject: [PATCH RESEND 2/2] ring-buffer/selftest: Handle meta-page bigger than the system From: Vincent Donnefort To: rostedt@goodmis.org Cc: mhiramat@kernel.org, linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, kernel-team@android.com, Vincent Donnefort , Shuah Khan , linux-kselftest@vger.kernel.org Handle the case where the meta-page content is bigger than the system page-size. This prepares the ground for extending features covered by the meta-page. Cc: Shuah Khan Cc: linux-kselftest@vger.kernel.org Signed-off-by: Vincent Donnefort Acked-by: Steven Rostedt (Google) Acked-by: Shuah Khan diff --git a/tools/testing/selftests/ring-buffer/map_test.c b/tools/testing/selftests/ring-buffer/map_test.c index ba12fd31de87..d10a847130fb 100644 --- a/tools/testing/selftests/ring-buffer/map_test.c +++ b/tools/testing/selftests/ring-buffer/map_test.c @@ -92,12 +92,22 @@ int tracefs_cpu_map(struct tracefs_cpu_map_desc *desc, int cpu) if (desc->cpu_fd < 0) return -ENODEV; +again: map = mmap(NULL, page_size, PROT_READ, MAP_SHARED, desc->cpu_fd, 0); if (map == MAP_FAILED) return -errno; desc->meta = (struct trace_buffer_meta *)map; + /* the meta-page is bigger than the original mapping */ + if (page_size < desc->meta->meta_struct_len) { + int meta_page_size = desc->meta->meta_page_size; + + munmap(desc->meta, page_size); + page_size = meta_page_size; + goto again; + } + return 0; }