linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Muhammad Usama Anjum <usama.anjum@collabora.com>
To: Andrew Morton <akpm@linux-foundation.org>,
	Shuah Khan <shuah@kernel.org>,
	Ryan Roberts <ryan.roberts@arm.com>,
	David Hildenbrand <david@redhat.com>,
	linux-mm@kvack.org, linux-kselftest@vger.kernel.org,
	linux-kernel@vger.kernel.org
Cc: Muhammad Usama Anjum <usama.anjum@collabora.com>, kernel@collabora.com
Subject: [PATCH 0/4] selftest/mm: Remove warnings found by adding compiler flags
Date: Mon,  9 Dec 2024 23:56:20 +0500	[thread overview]
Message-ID: <20241209185624.2245158-1-usama.anjum@collabora.com> (raw)

Recently, I reviewed a patch on the mm/kselftest mailing list about a
test which had obvious type mismatch fix in it. It was strange why that
wasn't caught during development and when patch was accepted. This led
me to discover that those extra compiler options to catch these warnings
aren't being used. When I added them, I found tens of warnings in just
mm suite.

In this series, I'm fixing those warnings in a few files. More fixes
would be sent later.

Muhammad Usama Anjum (4):
  selftests/mm: thp_settings: remove const from return type
  selftests/mm: pagemap_ioctl: Fix types mismatches shown by compiler
    options
  selftests/mm: mseal_test: remove unused variables
  selftests/mm: mremap_test: Remove unused variable and type mismatches

 tools/testing/selftests/mm/mremap_test.c   |  15 +--
 tools/testing/selftests/mm/mseal_test.c    |   8 +-
 tools/testing/selftests/mm/pagemap_ioctl.c | 108 +++++++++++----------
 tools/testing/selftests/mm/thp_settings.c  |   4 +-
 tools/testing/selftests/mm/thp_settings.h  |   4 +-
 tools/testing/selftests/mm/vm_util.c       |   2 +-
 6 files changed, 75 insertions(+), 66 deletions(-)

-- 
2.39.5



             reply	other threads:[~2024-12-09 18:56 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-09 18:56 Muhammad Usama Anjum [this message]
2024-12-09 18:56 ` [PATCH 1/4] selftests/mm: thp_settings: remove const from return type Muhammad Usama Anjum
2024-12-09 18:56 ` [PATCH 2/4] selftests/mm: pagemap_ioctl: Fix types mismatches shown by compiler options Muhammad Usama Anjum
2024-12-09 18:56 ` [PATCH 3/4] selftests/mm: mseal_test: remove unused variables Muhammad Usama Anjum
2024-12-09 18:56 ` [PATCH 4/4] selftests/mm: mremap_test: Remove unused variable and type mismatches Muhammad Usama Anjum

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20241209185624.2245158-1-usama.anjum@collabora.com \
    --to=usama.anjum@collabora.com \
    --cc=akpm@linux-foundation.org \
    --cc=david@redhat.com \
    --cc=kernel@collabora.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ryan.roberts@arm.com \
    --cc=shuah@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox