Received-SPF: pass (google.com: domain of linux-kernel+bounces-27636-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: =?UTF-8?q?Micha=C5=82=20Winiarski?= <michal.winiarski@intel.com>
To: <dri-devel@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>
CC: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Maxime Ripard
	<mripard@kernel.org>, Thomas Zimmermann <tzimmermann@suse.de>, David Airlie
	<airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, "Javier Martinez
 Canillas" <javierm@redhat.com>, =?UTF-8?q?Ma=C3=ADra=20Canal?=
	<mcanal@igalia.com>, Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>,
	=?UTF-8?q?Micha=C5=82=20Winiarski?= <michal.winiarski@intel.com>
Subject: [PATCH] drm/tests: mm: Call drm_mm_print in drm_test_mm_debug
Date: Tue, 16 Jan 2024 18:46:02 +0100
Message-ID: <20240116174602.1019512-1-michal.winiarski@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RS9lUm9hdEx6RVovTGp4a20yNldOY0ZhTklNWHk4VXcyQ3h5S1Q2SlFoVytq?=
 =?utf-8?B?QjdrY1VKZWUrOUgxb1VWTThzQ0xXN2tRRFhraU8reWtHR2o1eDJDeU5ZemxQ?=
 =?utf-8?B?elNnMmFpKzhSMzNZYnQzK3lRRHdkY3U2NFdFNEdFTUFZRzA0UWI1N2M2SWN0?=
 =?utf-8?B?L3JYeDZvYUYzZngraENMRlRvOEMwMDNqL2hJL2g4QmswRVFpeGV3YWw4UTRn?=
 =?utf-8?B?MWlaL2JDb0puR1BPd3FmQ25PYmxFRDZEbU1QTDgrR1hGdXFiZUN2Y0xmdk5s?=
 =?utf-8?B?UFl3YWpxcURXRHdYS1JES1pKMG9CM0pPa3hFWUFRSFNiQ3JDUlNaTWx5Qk8r?=
 =?utf-8?B?N0lzZTN6UWdPaDNCR2dIaWhrTUdlVitnd0J3R2dvbUk1ZU5aT3F0SCtvZnZj?=
 =?utf-8?B?azJlMDNpYnNES0t2S1BVakVzL3hLT1B4aUREUVJzSVNqdmd0Mm5DNWlLVE5a?=
 =?utf-8?B?ZEpCM21TR2k4QnZOY3g5eU5CbGxDY1oyTjRxZEZmOWpKK2QyaDFyVFF3VnlR?=
 =?utf-8?B?czZyMDFBY3Y4MEhHa1MyMXh1QzdkR1Z3bk5MZDcwMGJrL1AwMmlIVjEvTFpN?=
 =?utf-8?B?VlBMTXdQcDNhdDdBQjdoZXhFL1R3bW5tY04xaGQ0MXVIU1hqejNyc0daamc4?=
 =?utf-8?B?S29yRkVQK1d5TDltcExvK3lxTko5SVByVW1qMzUxNkFGaVhWS28yMVRTRGtz?=
 =?utf-8?B?bGNiY1BGNGcwcWVDQngrS0E3cU5Vb29vVGo2eWFSVy9HMGpwVnk1RCtJekxp?=
 =?utf-8?B?YXNBMlgrZE5xMlo4cWxOQVdyalNPV0xQbTFPM1hhaVVBbExCNG1sQ24xcjZr?=
 =?utf-8?B?eXJQZFNYeGVmdzdGNy9VQkpyS09kQzdtU2d0UjBYTGV2RCtzZHNaNnRMUVIz?=
 =?utf-8?B?VC9LNmEvRTA3dFUxTlpKRUdvZ2VOS2VKVnRFYklRV3l2VmZ6TkpSKzluMkNm?=
 =?utf-8?B?NHhLLzAxSUkyYXA3RTg2cFMwRXJYOEJuTXJvVzBuMkZTamtCeVJ3c3JhWTZ6?=
 =?utf-8?B?a2NNNmdDTVBkUUVDL3AzQkQ4alJBTU0rcVlNV3JEaDdyZ0k1TlFnTVpjK3VR?=
 =?utf-8?B?SVZPMmpsMWlSSzVhL3JoeFZFRFl1TDJCeDlOckNrQm9iaUNVeEhEb2M4ZS84?=
 =?utf-8?B?S1RDN3dReDJpTlp2VUZJLzV3M0N0OHJOZmVjaU5Hc2ZjZEJTY1phT1RnK0N3?=
 =?utf-8?B?K0xBeElHQ2VtRnh3a2k4bWhXOVJhRXdQWnQ1UzFkckNOcVdwVEgwbjRBaktV?=
 =?utf-8?B?SzZPaFNwdmw1NmRnVG1lUml1QWN6Rmh1Zk1BSVVGMG9tYWExVmduT2hySW4z?=
 =?utf-8?B?U0xPdlVIaHJwUS94VkwwMU83dXQyWkthSktDNmNKY1VkZTRmcGRURVBNVDl1?=
 =?utf-8?B?bnBlVXRiSU5CZ1ErSWM4d2ZXOWNCZjM0cGtJY21ycjVVMEFtNWpKam1rZFlj?=
 =?utf-8?B?bFdFVTdGUngzM2sxYlFJbU94ZjRTWWR2VUw5K056cjZha2l5bnRHSGdjYVcw?=
 =?utf-8?B?UWRIQ0JRZ0hYRy9PRWZtaDF1UmNRczNiZks1cElQSDJ6aGdrZDJZd0poQkRt?=
 =?utf-8?B?bVF5NFY1ejgzYlZldjMrdkkwckgwQlRrTnVQQnNyT3F3L3FQODdwQkx1Tmd0?=
 =?utf-8?B?aW54bUNIVTBsNi9YS25CSWwrMmlJZ2JiNW5GUTY4VnZCeU5GOEdKM0RCZkk1?=
 =?utf-8?B?NkZ5TDkySGF5MjlvTHB0YVczcnF1bHc1Q2h3RXVXcTl5QVR5QU9zcDhObFYv?=
 =?utf-8?B?V010Um11N0h3Z25OMkdBcWVCSXN3bzVVL2QreUpWay80SjZTUm9wT0NudzU0?=
 =?utf-8?B?SGtqd0x5V3piaU5IL3dER2JiUzlLWkk5MXBtTTl6ZDI4dFdhaVlReFYzVE5o?=
 =?utf-8?B?R1hkZXVNWFNYT2s4dUtrZCtQYXBSeURRL1lTOUJmNTVkNWNIZEFobmxZSGxn?=
 =?utf-8?B?MTZ0VWdWYys3M3F6QWRlSGIvUGlVdW0wTXFmcnFqTlQ5aDhIcUgzaWhGeGln?=
 =?utf-8?B?cDdFa05yeGRXakpSOUN3R3JqM2lERFBpb2FoeG5uWFJtL3ZLREdRWWFTWXNC?=
 =?utf-8?B?VHpIS2s0L1BJZVhRMWFWVE56ZkprVGhYOC9UcUkzR3hHa1p0VmhCUDB5VVhX?=
 =?utf-8?B?VHFYTDNLR1FQYW9jd001Zzlwd1EwSnlibFBIeERwV3hMVjRmQko4OGVVS1Nw?=
 =?utf-8?B?VXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4b68c35d-75bd-4005-90e1-08dc16bb11fa
X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB5373.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jan 2024 17:46:29.8629
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 6t/UB8eJtH0iwV7TawGcz/z1UKSq5dOYmnZhal/Ge6UuJeJl55CR1sxPr+HqjqsdsHyY9dL6CmdXA8hgVzrmnQCp14nnQv50bOnc1piPDZc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB8665
X-OriginatorOrg: intel.com

The original intent behind the test was to sanity check whether calling
the debug iterator (drm_mm_print) doesn't cause any problems.
Unfortunately - this call got accidentally removed during KUnit
transition. Restore it.

Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
---
 drivers/gpu/drm/tests/drm_mm_test.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/tests/drm_mm_test.c b/drivers/gpu/drm/tests/drm_mm_test.c
index 4e9247cf9977f..1eb0c304f9607 100644
--- a/drivers/gpu/drm/tests/drm_mm_test.c
+++ b/drivers/gpu/drm/tests/drm_mm_test.c
@@ -188,13 +188,13 @@ static void drm_test_mm_init(struct kunit *test)
 
 static void drm_test_mm_debug(struct kunit *test)
 {
+	struct drm_printer p = drm_debug_printer(test->name);
 	struct drm_mm mm;
 	struct drm_mm_node nodes[2];
 
 	/* Create a small drm_mm with a couple of nodes and a few holes, and
 	 * check that the debug iterator doesn't explode over a trivial drm_mm.
 	 */
-
 	drm_mm_init(&mm, 0, 4096);
 
 	memset(nodes, 0, sizeof(nodes));
@@ -209,6 +209,9 @@ static void drm_test_mm_debug(struct kunit *test)
 	KUNIT_ASSERT_FALSE_MSG(test, drm_mm_reserve_node(&mm, &nodes[1]),
 			       "failed to reserve node[0] {start=%lld, size=%lld)\n",
 			       nodes[0].start, nodes[0].size);
+
+	drm_mm_print(&mm, &p);
+	KUNIT_SUCCEED(test);
 }
 
 static bool expect_insert(struct kunit *test, struct drm_mm *mm,
-- 
2.43.0