mbox series

[v2,0/1] cgroup: remove redundant addition of memory controller

Message ID 20240502200529.4193651-1-usamaarif642@gmail.com (mailing list archive)
Headers show
Series cgroup: remove redundant addition of memory controller | expand

Message

Usama Arif May 2, 2024, 8:04 p.m. UTC 
Memory controller is already added in main which invokes
the test, hence this does not need to be done in
test_no_kmem_bypass.

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(-)

Comments

Yosry Ahmed May 3, 2024, 2:09 a.m. UTC | #1 
On Thu, May 2, 2024 at 1:05 PM 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.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
>