Received-SPF: pass (google.com: domain of linux-kernel+bounces-167171-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240502200529.4193651-1-usamaarif642@gmail.com>
In-Reply-To: <20240502200529.4193651-1-usamaarif642@gmail.com>
From: Yosry Ahmed <yosryahmed@google.com>
Date: Thu, 2 May 2024 19:09:36 -0700
Message-ID: <CAJD7tkYwAQQB_90S8_6YiAq-j3Ew0KKK+a4gnBn2XD+8ocnG4A@mail.gmail.com>
Subject: Re: [PATCH v2 0/1] cgroup: remove redundant addition of memory controller
To: Usama Arif <usamaarif642@gmail.com>
Cc: akpm@linux-foundation.org, hannes@cmpxchg.org, nphamcs@gmail.com, 
	chengming.zhou@linux.dev, linux-mm@kvack.org, linux-kernel@vger.kernel.org, 
	kernel-team@meta.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, May 2, 2024 at 1:05=E2=80=AFPM Usama Arif <usamaarif642@gmail.com> =
wrote:
>
> Memory controller is already added in main which invokes
> the test, hence this does not need to be done in
> test_no_kmem_bypass.

A cover letter is not required. In cases like this where providing
additional context to the commit log is not needed, it is better to
omit it. This especially applies when there is a single patch in the
series.

If you want to add version diff info or any additional context, you
can add it after "---" in the patch itself. Example:
https://lore.kernel.org/linux-mm/20240430133138.732088-1-ryan.roberts@arm.c=
om/.

>
> v1 -> v2:
> - Add more description to commit message.
>
> Usama Arif (1):
>   selftests: cgroup: remove redundant addition of memory controller
>
>  tools/testing/selftests/cgroup/test_zswap.c | 2 --
>  1 file changed, 2 deletions(-)
>
> --
> 2.43.0
>