From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 A45F23D3D0E for ; Wed, 27 May 2026 17:00:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779901261; cv=none; b=ic1IFDYUXUlMdj9qU1WOegEfj0L1ioQdVGYdAdrxfxZN8CIrV6snFFsWlX3acsui/ELS1pbXmOiuFm2T0ngwnf8PnmCnuZa718ceNUo/EDlo5jnEKkSmrbmmbQAfJ/kLliQ+WzCHfTBYH1HQiEwobGHLacerb29F2vEF++nRuOY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779901261; c=relaxed/simple; bh=IC03rV44yx8SbaDtxThdFCvVl5fVz1TnoYNrddEg7m8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=rxuSmTNyDcSj+c7iSpkn92QOJRKdg1UdMQJJ4loakRdMcTfzXiOcTjpMieqZZGRtrq+E8Ec0Bcmlf0hnVnsARZVMTMx3Jcss18faU3fKtAtWCsbu+JGQHhMgfYxrhn/t7iPUMt2SJ++JV7OxbJhhowytfpCAoiVApN7cL0l//1I= 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=GDxCEQSd; arc=none smtp.client-ip=209.85.214.173 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="GDxCEQSd" Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-2ba3b9bcf69so95ad.0 for ; Wed, 27 May 2026 10:00:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1779901259; x=1780506059; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=7Og5FR/9qWtaybmoKawxFptjk13+mvY5AZnqWPtKcDM=; b=GDxCEQSdMC0hubLsATO5WComY3s4aOxIvo8zo5MBQcddD0s2rbrd8T/SRY42mlbEAX mGAM/d47Lk34RE1vmQRvMbeyoTNLrlRloFv1yvixSD1FgwZUdLk9SS9RfNm9HZW7M5rk xyAbZRM6JHq6pQvp5ZeOsNNIzTf4ieP0xceKeCD71HSF/ITKratMsvfj8eux72jnQcV7 yt0bf0D3eNGPijaVycsEfGqjkE1CqjMNgoKt2X52HQksuDw0b3PorZX4o7Lgv6zQEpFB TmeZk2HHoBpqd+zUsuKdmyto6LCiYETfp3mjhGoMnONEFi2FdEzpA/T79bjAUhYWUD2R DXZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779901259; x=1780506059; h=in-reply-to:content-disposition: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; bh=7Og5FR/9qWtaybmoKawxFptjk13+mvY5AZnqWPtKcDM=; b=SAMpmKQ5t/8cQF1rAUkz7l2pVf2RTeawu+H3C/hhOwsP/Ksjt9XMZvyogjdt/JyZ5Y 9nEVuG3unL6MSkjMWuYBpQLV8qQQN+cwNHnqEUsxrJmcpF3nmV4rW9UgLwUuQgKkm8Q7 /+Fp/wAO12eqGbvfdzsQNZFwaDnmoSQqhFM304rnYY5gmRA02BzWpUTpnsD/HLlkKGaF E/68UO+Sr6HMkgvW0GErUn2kLcYyV/pRRgf1BJOYah4/KD+ZdtRyRF5VAoJM4DeK6dRc RtLKakkfGkkjcVaP3pI4mgO+0I/JDondM/Tl20+KsmemKU6rkspMWvcaqkuBpiI1WAYN wo8A== X-Forwarded-Encrypted: i=1; AFNElJ+5Hlp42/Fg2Pg/ChI9BBbYo7DU5qiH3ovhqL6N2LMyRHg4zCo6iWOIPqDDaPdAHXGcZMl75eBXLCrBquBjO5k=@vger.kernel.org X-Gm-Message-State: AOJu0Yyqu5YYJwJjXD6cXvKou4kps9QawiIpXsvH+SdtgvSvMoimSLkb FEGEZyIFvZDV76SJVvIiz69P1rV3l6hhBpRnXsTUvN1IdA8FxB5uC5faCUWBcWFXIw== X-Gm-Gg: Acq92OE8v5XtFaiYlvv8sRsWImkU5JtnjhUP5X3zb9xiKWStt2yNZlcOKalNqRMnt/q pvvNuqukQPK54zbRdj8C5V1KD+c9jrXzu5Pgv1Ne683lMfbbTlRcM9bR0/BXdXLNyvOXLJRSJ+k CiL+uhJ3AIiz24MJgJFgUTAxvyM9cwnJaIsQ2Wd75743m/v1bO0c91cTN9JtAqt9cTdRaQJzdLK GA08LSOd+r0/P+lOIbghiGz+e2vu31MBJNfVtASI9CGTwV+OmpOUznaNJi3W768+EyUk2YJ96I7 nUXxcyi75rQRrG9DLms5jmWW+ZJ0xAT7zlNy5qI4u4Azyh712e2oKwtI1JhGIm5os3sow576NTp KwyGOzxUHFkv0WX4EWui6frd26DWpOmZi8FIWgJPDs5aL9IK08bOpCb8PZ6wtoywgnmTa5E7UhQ GwveyMQ6Tu4bAjHxWLu1r8eMWujbqe4pe9eRaJH8HPczHfG+oZ5ipnJTQ5rOMEP82xg/M= X-Received: by 2002:a17:902:e551:b0:2bd:7e8e:ad56 with SMTP id d9443c01a7336-2bebe9feb6emr8862045ad.6.1779901258238; Wed, 27 May 2026 10:00:58 -0700 (PDT) Received: from google.com (230.0.127.34.bc.googleusercontent.com. [34.127.0.230]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-36b7653c9ffsm334915a91.4.2026.05.27.10.00.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 May 2026 10:00:56 -0700 (PDT) Date: Wed, 27 May 2026 17:00:52 +0000 From: Lisa Wang To: Miaohe Lin Cc: 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, Naoya Horiguchi , Andrew Morton , Paolo Bonzini , Shuah Khan , Hugh Dickins , Baolin Wang , David Hildenbrand , Lorenzo Stoakes , linux-kselftest@vger.kernel.org, "Liam R. Howlett" , Vlastimil Babka , Mike Rapoport , Suren Baghdasaryan , Michal Hocko , linux-mm@kvack.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org Subject: Re: [PATCH RFC v3 5/7] mm: selftests: Add shmem into memory failure test Message-ID: References: <20260408-memory-failure-mf-delayed-fix-rfc-v3-v3-0-718f45eb7c75@google.com> <20260408-memory-failure-mf-delayed-fix-rfc-v3-v3-5-718f45eb7c75@google.com> <9effbeca-3453-f0fe-571f-85f8923e1385@huawei.com> Precedence: bulk X-Mailing-List: linux-kselftest@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: <9effbeca-3453-f0fe-571f-85f8923e1385@huawei.com> > On Mon, Apr 13, 2026 at 07:57:55PM +0800, Miaohe Lin wrote: [...snip...] > > - > > + case MADV_SOFT_DIRTY_SHMEM: > > /* The page content should remain unchanged. */ > > ASSERT_TRUE(check_memory(vaddr, self->page_size)); > > Why we skip check_memory() for case MADV_SOFT_CLEAN_SHMEM? Because we did not write 0xce to clean_shmem. The difference between clean and dirty pagecache is whether we synced the dirty pages to disk. However, the difference between clean and dirty shmem is whether we initialized it with 0xce. Thus, we need to skip check_memory(), which checking 0xce, for clean_shmem. > > + case MADV_HARD_CLEAN_SHMEM: > > + case MADV_SOFT_CLEAN_SHMEM: > > + /* It is not expected to receive a SIGBUS signal. */ > > + ASSERT_EQ(setjmp, 0); [...snip...] > > + if (variant->type == MADV_HARD) { > > + check(_metadata, self, addr, MADV_HARD_DIRTY_SHMEM, ret); > > + ret = sigsetjmp(signal_jmp_buf, 1); > > + if (ret == 0) > > + FORCE_READ(*addr); > > + check(_metadata, self, addr, READ_ERROR, ret); > > + } else { > > Should we always add a FORCE_READ() just after variant->inject to verify the accessibility > to the addr for soft-offline case ? > > Thanks. In dirty_shmem, check() already verifies accessibility to the address through memcmp(). Thus, we only need FORCE_READ() in clean_shmem. Will add this in the next version. Thanks ! Lisa > > + check(_metadata, self, addr, MADV_SOFT_DIRTY_SHMEM, ret);> + } [...snip...]