From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f177.google.com (mail-pl1-f177.google.com [209.85.214.177]) (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 DDE244582CC for ; Wed, 26 Aug 2026 17:57:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787767067; cv=none; b=Un0uWCYtpfaWPPhbv+KOPJEa7OLAEF/WQzU6ypDC3jW1TZTtuOzVXwYagJLyMp8QJ7HB1duF24aitO60aDVTAY1JUakz3IcGVuJFPhvXvV7M1PXhNir8thc4oCx3/f+0kRjbYy9AcePu2XqXD08Vaiw++ZrPOIOSdmN6oxy6QsQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787767067; c=relaxed/simple; bh=WP3lzz3sPwNr/uWnF1dnEtr6qfIaUelhmsrLvivFud4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=B+cjCi/7Xin2nLZ7Y91HU8MoszkbgH5xBVVS0wSczaNZzJsXkrxfzb/YSO7sQRP96p1bPfBNdSiMFi2Bk37bXspRdI7DiAxRhguaOekhSi10sdSMNrgv5UTl+jRzq4BQMIXGYoSW54rMN4I+FhJ0qK0uEof3vlgWTugujs7ewPk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=hquZmByv; arc=none smtp.client-ip=209.85.214.177 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=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="hquZmByv" Received: by mail-pl1-f177.google.com with SMTP id d9443c01a7336-2d3b440b97aso10725ad.1 for ; Wed, 26 Aug 2026 10:57:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1787767053; x=1788371853; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=mOyt5LoY374Qkpz6cQhUC33i8CoGf2Yn2cDcltrm0Cc=; b=hquZmByvqvONjAlbXKuBbz8QPQjRA5q24l+0d+Wr/FDWY71EaBqGj8IlwKdWAZ/7rN qfhOOD0fECT1ixEw028sq04lhB5Z07hNlWPm6R3OajIctcl5IT/kz9pvApm4OixZ97oz VZxiM4k8JmKXtVKR8jk7Gp9I4YGU5A7Chzn6ygBLmMqOe6Oa3kypdshgeQT3A/tHJtny haMbvbBbdWrldqdotn11ykqbR6cZYMaVgFxEYpU5hbmi6FjSNGAfZNFD30VDJSmHUmlR W7RvcM+aVp9V7cuZCosewrtpuz1dmLCXobqo+V3HFTZhZC6AHrBOyzbhjJdfQJguASa7 7CYA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787767053; x=1788371853; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=mOyt5LoY374Qkpz6cQhUC33i8CoGf2Yn2cDcltrm0Cc=; b=a8XnJUxeo/ZykZZ3IYMd9AuaxUiTKe/dhT9PYF4XtNCnrsqv5hTpqc+4qUaW+Uvm82 j9nJDik+0COzJXDi87BwVqeAXRHCut/XMA9xi87nTaz403ze/cMHYnXuThgO97NHb7qE arl1u6Gql3Td0F2sFoXUPKPROp6ujRgBbXk4s3s/3QXQyTyLwQ5DwHlKC+0xGNt8qovx UcGBnPjc29M9iWGo2xzheljMcuDa064ZTBq0h6OhmW45spVwGfQV9c/DBojshHMuqRsp uj9DTYDOjT4pUCX4AFLju0zfFxTCXAF6Tz9DSi6zFzJIdjz9YKdXWVNZsztkg0mNnJ6/ aoSA== X-Forwarded-Encrypted: i=1; AHgh+RpC1oZbuHPoRLL6pNtfoyKFNQpgP3cAYL+h1R/gWEHbpYYYc0InXJODgAw0VXMZ8oqIMb4gFsmSJTyDgeM=@vger.kernel.org X-Gm-Message-State: AFuF++l0q9GDD13WrkTHbvY2d9WGcKWJUwiltmBGadsgCTzBoSq0p76/ gz5QuWrh+YgJCflhmJZM38vQb6603bbtYSWbrx84ZqffXzPWRjPtNJUUD3dbl+Nlvg== X-Gm-Gg: AR+sD11JaKbzfCxgcbLp8AdJhwreREKSttA5KDZSDmLivw7gVtMLfSq8BAUcZNW7qiS KvN297IaJGusONxyA/qn1Um8fo/9SjMQTawv5wVC9HWmz1paz0Mht/eKt553MDJmwPRup0b1iQH 1nkUOPVTpvkdaBPb+HCXuYiCZcYBVUC81QC01xOkbF0UADUgp4MqWARsP6lCHn3Liac4d797DuN rA5EwLhpYqSnjr1ienQ4Td8huuK4N+LLDyUKBO4AYHB6cFAwKLAyvhAuoPeEl/AWVZr1/rd/mw4 lQLCr5ouLtxEUZocdqMOWG78IKvfviK5VMzPfbZiU9XXK2c/kXy5eNHMt+0l+S6V/FZqkUc+7AO dnlMT3PMqZ9Q9cpSJFkk/xF8k74OEOmetifzvGUBBQsqyvWa1Oq9J9b/NYLNMLc0JSR9BUgT+sy Lltw6+TNIexbUMGh0s8zEvXsa5TW9H5rDjvV1DHbTb4yiSNGn7lI6zzW9GjL14VsTlFudIHrVir sfZxEFMDMiq3R/qGMrfvA== X-Received: by 2002:a17:902:d4c7:b0:2c9:b404:b55 with SMTP id d9443c01a7336-2d72a6912edmr556605ad.6.1787767052531; Wed, 26 Aug 2026 10:57:32 -0700 (PDT) Received: from google.com (195.5.127.34.bc.googleusercontent.com. [34.127.5.195]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-39645d31e8csm9002581a91.9.2026.08.26.10.57.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 26 Aug 2026 10:57:31 -0700 (PDT) Date: Wed, 26 Aug 2026 17:57:28 +0000 From: Lisa Wang To: "David Hildenbrand (Arm)" Cc: Miaohe Lin , Naoya Horiguchi , Andrew Morton , Paolo Bonzini , Shuah Khan , Hugh Dickins , Baolin Wang , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , linux-mm@kvack.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-kselftest@vger.kernel.org, Andi Kleen , Hidehiro Kawai , Rik van Riel , Yu Zhang , Lorenzo Stoakes , "Liam R. Howlett" , rientjes@google.com, seanjc@google.com, ackerleytng@google.com, vannapurve@google.com, michael.roth@amd.com, jiaqiyan@google.com, tabba@google.com, dave.hansen@linux.intel.com, Isaku Yamahata , Xiaoyao Li Subject: Re: [PATCH v5 5/5] mm: selftests: Add shmem into memory failure test Message-ID: References: <20260823-memory-failure-mf-delayed-fix-v5-0-d97a5703e45a@google.com> <20260823-memory-failure-mf-delayed-fix-v5-5-d97a5703e45a@google.com> <8a99f753-0713-4194-8eb5-90da0724e647@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8a99f753-0713-4194-8eb5-90da0724e647@kernel.org> On Mon, Aug 24, 2026 at 03:00:47PM +0200, David Hildenbrand (Arm) wrote: > > +++ b/tools/testing/selftests/mm/memory-failure.c > > @@ -20,6 +20,10 @@ > > > > #include "vm_util.h" > > > > +#ifndef fallthrough > > +# define fallthrough __attribute__((__fallthrough__)) > > +#endif > > This doesn't belong into this test. > > rseq/rseq.c selftest already uses it. > > I think tools/testing/include/ already provides it. > > Maybe > > #include > > does the trick? Hi David, Thank you for replying. Would it be better to change it to /* FALLTHROUGH */ like the rest of the mm selftests? If we prefer to include , we can certainly do that, but we would also need to update the Makefile and fix a redefinition in pkey-helpers.h. --- a/tools/testing/selftests/mm/Makefile +++ b/tools/testing/selftests/mm/Makefile @@ -37,7 +37,7 @@ endif # LDLIBS. MAKEFLAGS += --no-builtin-rules -CFLAGS = -Wall -O2 -I $(top_srcdir) $(EXTRA_CFLAGS) $(KHDR_INCLUDES) $(TOOLS_INCLUDES) +CFLAGS = -Wall -O2 -I $(top_srcdir) $(EXTRA_CFLAGS) $(KHDR_INCLUDES) $(TOOLS_INCLUDES) -I$(top_srcdir)/tools/include CFLAGS += -Wunreachable-code LDLIBS = -lrt -lpthread -lm --- a/tools/testing/selftests/mm/pkey-helpers.h +++ b/tools/testing/selftests/mm/pkey-helpers.h @@ -22,7 +22,7 @@ typedef __u8 u8; typedef __u16 u16; typedef __u32 u32; -typedef __u64 u64; +// typedef __u64 u64; Lisa