[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJD7tkYwAQQB_90S8_6YiAq-j3Ew0KKK+a4gnBn2XD+8ocnG4A@mail.gmail.com>
Date: Thu, 2 May 2024 19:09:36 -0700
From: Yosry Ahmed <yosryahmed@...gle.com>
To: Usama Arif <usamaarif642@...il.com>
Cc: akpm@...ux-foundation.org, hannes@...xchg.org, nphamcs@...il.com,
chengming.zhou@...ux.dev, linux-mm@...ck.org, linux-kernel@...r.kernel.org,
kernel-team@...a.com
Subject: Re: [PATCH v2 0/1] cgroup: remove redundant addition of memory controller
On Thu, May 2, 2024 at 1:05 PM Usama Arif <usamaarif642@...il.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.com/.
>
> 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
>
Powered by blists - more mailing lists