From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yb1-f201.google.com (mail-yb1-f201.google.com [209.85.219.201]) (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 705E469DF3 for ; Tue, 30 Jan 2024 18:55:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706640948; cv=none; b=ZbZ2g1QMxALtxQabS6QktPIARoYxFCd4+xxbtTdHfgtK0I6U4G12m1SRtGGn6ypZpoH1QeROjadlc562K+uLsLVDp/roDOW8haRT5YpznbKQiQ6z8Vj1P+1Qb0rDhwawdoTvU1iCG18XJ5qvrwIONLTPONSu/6JhJ983vb5Msbo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706640948; c=relaxed/simple; bh=Fv/bTQ1xMvf2SnP8rg7hnK7lCBHHXhYmdjHeTk9P6G8=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=P/2zYj2LBGWLtqGebYP/kEwllMoO2xa8UCosdKJ+kfWOG3TZ+hZoal50wxKA+X4Z3IlszN9A8Y8BjJkp5efKLBJhF0M/JiS4MIs4TRS7tk1pTpHYVUbQ3ek8n31k7aOD3wCrAZTPp4VJR+AZVb4M7NTc7ESV9nRxYzRazA7oynU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--yosryahmed.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=F8yIP0wd; arc=none smtp.client-ip=209.85.219.201 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--yosryahmed.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="F8yIP0wd" Received: by mail-yb1-f201.google.com with SMTP id 3f1490d57ef6-dc6b26d3505so357582276.3 for ; Tue, 30 Jan 2024 10:55:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1706640945; x=1707245745; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:from:to:cc:subject:date:message-id :reply-to; bh=Fv/bTQ1xMvf2SnP8rg7hnK7lCBHHXhYmdjHeTk9P6G8=; b=F8yIP0wdLqMQSIeRBUEeIKuMOFHz1mm/OScxL0vyxE5sZq4r1RIyxh2vUQ07+XwGzK aa0MKae9HfTMjR5RWxvl7LDtnxmOfppJIoMBsRpODsvwsEtrmqg2rXCR18Ouhc/TZDEe AVNPo8ocP8wITgzabtV9JChleFPotYWMxab43OpIVobw144/4KM5OphvUkBnOYODKJeF rKjpl2OUeDNtHA0QxJ44BPBlj/WCN+/iB/umCplAVjbCj30USI22yA6pGHlwaDOQsvy/ +h+5HDQGQxqnNU46BKs5HSNSBEnASzj5Z0lIWQKe9OWqstj9negPcP2ah0a/p+QFOnoR fXtQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706640945; x=1707245745; h=content-transfer-encoding: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=Fv/bTQ1xMvf2SnP8rg7hnK7lCBHHXhYmdjHeTk9P6G8=; b=F8aIpfJllW3f6JNT9Diw2wazzwC4ofUUx+0KOo8v2oUGMLCOanlxExoAac0/K4ffrg THHXYt7g2sNN9zZFIVh0R7zgSfnkO/JYgIiFbRzeD9nmLtfqVhOPhN119fLNsxMo5wA1 8EQUqyQXA0JZGIvlrl30CbXHhdqYho/lYgrsiysRbP6k0XYi5eggDykMPt1gMt4sruya 5LlyJ74ouXan5sGn/mahHEfYLuAf3YqX9dEdB2SANWpGqbLs2OdwkthiPQq3YHpT0WC3 nF+wxEu0GCZ+3dVvZUhy79dL9/CivIqNMG/EK7YB747zFADlLQnl/X1iFTuavbqOqpni /6pQ== X-Gm-Message-State: AOJu0YwfO0huDVNw6YEiWMgCZ4CZ/uyHmiF5kx8SDqRWYeeaP3K4+4g5 RZGp9yU3xC8MavuDNpY/exJrOZGRVSUHGyUZQeMOxAk07KZK7rQd2urZOZsRcRzE4Xab5VPt46p PR/0v1yy9kHSB1hWd1g== X-Google-Smtp-Source: AGHT+IGkYCoroPgQ0vC/qjOZHRglhy7FVefWVZSNBL/apsveQNFhYpaPcmuSmn1r9c0pb5X44+CYburGBcOpRHsF X-Received: from yosry.c.googlers.com ([fda3:e722:ac3:cc00:20:ed76:c0a8:29b4]) (user=yosryahmed job=sendgmr) by 2002:a05:6902:2406:b0:dc2:260b:64d9 with SMTP id dr6-20020a056902240600b00dc2260b64d9mr576357ybb.10.1706640945579; Tue, 30 Jan 2024 10:55:45 -0800 (PST) Date: Tue, 30 Jan 2024 18:55:43 +0000 In-Reply-To: Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240129224542.162599-1-nphamcs@gmail.com> <20240129224542.162599-2-nphamcs@gmail.com> Message-ID: Subject: Re: [PATCH 1/3] selftests: zswap: add zswap selftest file to zswap maintainer entry From: Yosry Ahmed To: Nhat Pham Cc: akpm@linux-foundation.org, shuah@kernel.org, hannes@cmpxchg.org, tj@kernel.org, lizefan.x@bytedance.com, linux-mm@kvack.org, kernel-team@meta.com, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, linux-kselftest@vger.kernel.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On Tue, Jan 30, 2024 at 10:37:15AM -0800, Nhat Pham wrote: > On Mon, Jan 29, 2024 at 5:02=E2=80=AFPM Yosry Ahmed wrote: > > > > On Mon, Jan 29, 2024 at 02:45:40PM -0800, Nhat Pham wrote: > > > Make it easier for contributors to find the zswap maintainers when th= ey > > > update the zswap tests. > > > > > > Signed-off-by: Nhat Pham > > > > I guess I had to check the zswap tests at some point :) >=20 > We sorely need more zswap tests :) >=20 > I'm one of the offenders of adding new features without including > tests, so no judging anyone of course, and admittedly zswap is quite > intertwined with other parts of MM, so it's kinda hard to write > unit-ish tests for zswap only. I often had to resort to scripting > stress tests to iron out bugs. >=20 > But there are still tests that we can write to verify public API > (cgroup's zswap options come to mind), simple tests that cover crucial > code paths, etc. that we should probably add in. At the very least > this can be a quick/sanity check for developing and backporting > patches into the production system. Agreed. I am yet to take a close look at the existing tests tbh, but I took a quick look before and there is room for improvement (and little bandwidth to act on it).