All of lore.kernel.org
 help / color / mirror / Atom feed
* [merged mm-nonmm-stable] resource-kunit-stop-selecting-get_free_region.patch removed from -mm tree
@ 2026-10-11 22:20 Andrew Morton
  2026-10-13 19:17 ` Andy Shevchenko
  0 siblings, 1 reply; 2+ messages in thread
From: Andrew Morton @ 2026-10-11 22:20 UTC (permalink / raw)
  To: mm-commits, raemoar63, huang.ying.caritas, geert, david,
	brendan.higgins, brads, andriy.shevchenko, kmehltretter, akpm


The quilt patch titled
     Subject: resource, kunit: stop selecting GET_FREE_REGION
has been removed from the -mm tree.  Its filename was
     resource-kunit-stop-selecting-get_free_region.patch

This patch was dropped because it was merged into the mm-nonmm-stable branch
of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm

------------------------------------------------------
From: Karl Mehltretter <kmehltretter@gmail.com>
Subject: resource, kunit: stop selecting GET_FREE_REGION
Date: Sat, 26 Sep 2026 02:26:51 +0200

RESOURCE_KUNIT_TEST selects GET_FREE_REGION even when no other option
needs it. Remove the selection to follow the dependency rule in
Documentation/dev-tools/kunit/style.rst.

Skip resource_test_region_intersects() when GET_FREE_REGION is disabled.
GET_FREE_REGION has no prompt, so configurations without a production
consumer cannot enable it. Most configurations will therefore skip this
case; this is intentional. The union and intersection tests remain
available.

Since kunit_skip() does not return, the compiler drops the reference to
the unavailable alloc_free_mem_region(), as in the CONFIG_OF_ADDRESS
check in drivers/of/of_test.c.

Link: https://lore.kernel.org/20260926002651.87267-1-kmehltretter@gmail.com
Fixes: 99185c10d5d9 ("resource, kunit: add test case for region_intersects()")
Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Reviewed-by: Bradley Morgan <brads@mainlining.org>
Tested-by: Bradley Morgan <brads@mainlining.org> # Power10
Assisted-by: LLM
Cc: Ying Huang <huang.ying.caritas@gmail.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Brendan Higgins <brendan.higgins@linux.dev>
Cc: David Gow <david@davidgow.net>
Cc: Rae Moar <raemoar63@gmail.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

---

 kernel/resource_kunit.c |    3 +++
 lib/Kconfig.debug       |    1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

--- a/kernel/resource_kunit.c~resource-kunit-stop-selecting-get_free_region
+++ a/kernel/resource_kunit.c
@@ -225,6 +225,9 @@ static void resource_test_region_interse
 	struct resource *parent;
 	resource_size_t start;
 
+	if (!IS_ENABLED(CONFIG_GET_FREE_REGION))
+		kunit_skip(test, "CONFIG_GET_FREE_REGION is disabled");
+
 	/* Find an iomem_resource hole to hold test resources */
 	parent = alloc_free_mem_region(&iomem_resource, RES_TEST_TOTAL_SIZE, SZ_1M,
 				       "test resources");
--- a/lib/Kconfig.debug~resource-kunit-stop-selecting-get_free_region
+++ a/lib/Kconfig.debug
@@ -2776,7 +2776,6 @@ config RESOURCE_KUNIT_TEST
 	tristate "KUnit test for resource API" if !KUNIT_ALL_TESTS
 	depends on KUNIT
 	default KUNIT_ALL_TESTS
-	select GET_FREE_REGION
 	help
 	  This builds the resource API unit test.
 	  Tests the logic of API provided by resource.c and ioport.h.
_

Patches currently in -mm which might be from kmehltretter@gmail.com are

mm-secretmem-fix-the-enable-parameter-description.patch
mm-damon-core-preserve-the-quota-passed-to-damon_new_scheme.patch
mm-damon-tests-core-kunit-test-preservation-of-quota-state.patch
mm-damon-tests-core-kunit-test-psi-goal-values-with-explicit-samples.patch
squashfs-fix-fragment-index-table-sizing-overflow-on-32-bit.patch
squashfs-make-the-fragment-index-table-bounds-check-overflow-safe.patch


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [merged mm-nonmm-stable] resource-kunit-stop-selecting-get_free_region.patch removed from -mm tree
  2026-10-11 22:20 [merged mm-nonmm-stable] resource-kunit-stop-selecting-get_free_region.patch removed from -mm tree Andrew Morton
@ 2026-10-13 19:17 ` Andy Shevchenko
  0 siblings, 0 replies; 2+ messages in thread
From: Andy Shevchenko @ 2026-10-13 19:17 UTC (permalink / raw)
  To: Andrew Morton; +Cc: mm-commits

On Sun, Oct 11, 2026 at 03:20:05PM -0700, Andrew Morton wrote:

...

> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> Reviewed-by: Bradley Morgan <brads@mainlining.org>
> Tested-by: Bradley Morgan <brads@mainlining.org> # Power10
> Assisted-by: LLM

I'm wondering if it will be a big deal to you to add to your scripts something
that will emit a '---' line here (before Cc list). This will reduce a lot of
noise in the commit messages, especially when the Cc list is hypothetically 20
lines while commit message hypothetically only 3. Note, that emails will be still
send as usual (in case you use some kind of `git send-email` or `b4`.

> Cc: Ying Huang <huang.ying.caritas@gmail.com>
> Cc: Geert Uytterhoeven <geert@linux-m68k.org>
> Cc: Brendan Higgins <brendan.higgins@linux.dev>
> Cc: David Gow <david@davidgow.net>
> Cc: Rae Moar <raemoar63@gmail.com>
> Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> ---

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-10-13 19:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-11 22:20 [merged mm-nonmm-stable] resource-kunit-stop-selecting-get_free_region.patch removed from -mm tree Andrew Morton
2026-10-13 19:17 ` Andy Shevchenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.