From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm2-f12.google.com (mail-wm2-f12.google.com [74.125.225.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D162E484232 for ; Fri, 25 Sep 2026 09:28:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.140 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790328493; cv=none; b=MLKvefni/tAck7ejJPigzXvF8VuU5Tq66ybA5RqloZvjfUKU3xUs/khf2///LKC/oe9c8270/BKcnNQ7NjuCYar0X7IyjTZm77ebXQ+rlL7iCppXEEiUXmwh0pAnrPo2Pnr789B2eoBJr72CqjmmATxuE7IHOK7GH1i5/LGneXA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790328493; c=relaxed/simple; bh=RyhPOU+vvqd3rl1UlRjckMuKAfbjLaCIW52sS+yfk0s=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=JDwudrzU+dU36/2S2SHoZyYN0qkCNzaoQHXsyD7CGbIU+cVgOiBoAViQZ05JEe87ppYtzD9fk9nAwCy9Xfr/S5oKEWEQPOmCC1OlBdKNGQztvOdHRTR6wDSAeJjBpxQ2fW0u7XBrofnhfXMKGFCPg3HgddXnRs6qY+zJEQr0b/U= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=YCaa5B8/; arc=none smtp.client-ip=74.125.225.140 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="YCaa5B8/" Received: by mail-wm2-f12.google.com with SMTP id 5b1f17b1804b1-49e620fa473so4207205e9.1 for ; Fri, 25 Sep 2026 02:28:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1790328488; x=1790933288; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=GuAasn6M17RraKJz85D0VGepUYYLmKKd6LjzFHjVWiA=; b=YCaa5B8/bbraaj9MXbfcKga+R1L3Oy53Dlm+QLJ9PluOw2HJuSZ4RD0OUgpEKjEDys 1iIjGIzlaLXJl+fJzhD4jsJKpX5ftjb/LWFjKjNSvN0uwzojBFSmYZYw1vDOyCInGbUe d5kHIKVqyg3wLnQ6C0VSNz7g/+iaP/c5K3E5bSPY0TEFeHNVg4IhHe4Q2p8E6MzsC8dT n8na7+M39OVdS6Nak1AG1EpnfI6rPQBVhwmwML6M8v349eGyR3HHS2N8OshqSNp6V+WE IUEAemD4YTNyORvF5Su0LA+6HBShR5Rw8n06UjjwBYuJSJyZ6FP2VmqYggRcG9WSYKNN nCKg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790328488; x=1790933288; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=GuAasn6M17RraKJz85D0VGepUYYLmKKd6LjzFHjVWiA=; b=tA2rmOtrEfuzPbt2eHnK/D4BAkC3NkMHdUiBUyOjPNY7RQ6YiiLuF5A+XqbSnoMsPg B1e3gUzbVHPhTmOcX2W3ZM7s7RleAJWCUbWxkBqyUF2eqGecOcI4NAMpKbDXMMJrGaG1 2FqhTh/FPJ5E5j0B4JOixu1DkTAdW8oKhQiC2izCt8v5vZCaIVcvyvYiDUUwXsaQrcB0 XOhj9ENIh8YQUuofE0FWkFy9YKSidaNNCAKWQp6noET73NpsFaRZ0FAvNp26WRiA74eU Q0jjZ3eu2amnb1zsHr1lr5i1aJcOM6ImxAidlFjFNQz9K2hLTjrG/Dv092tkTpdeNGmD 3DQg== X-Forwarded-Encrypted: i=1; AKwUvBx7JIceFyfDdw/c42j67OTlBwz4wBA5ADOiGpLj1QRcqw4c9jMovkKo3hwdryu6Ax4Re4YSMIkczxgrWIsF@vger.kernel.org X-Gm-Message-State: AFuF++nLJPXm8etn11SUFfkeXi2aKfxteiwpdtqiI7uaRSkXYUf88G2N MM8Kt+9CsvIS0PRcy2BwvfQ83W4a99qRjz4b9jHgp/k97dopTrUqiIesUzQ78DpHYeI= X-Gm-Gg: AYBFou3SZf0qpb9Oy0qiWZXNJW/2DVcrpSW7sJGIH828GEpwg02Et+nhs33TcM0RgTn F1rHUV2ftYVWq5fXkVBfni2IebAoyi2CviBhKE/x58i52DVZ+Iko0DvMoGvwStthXJesSGG7eYv 1PAaRtVitZ/V3pDYMaJv4if0p/zQcN6ExenzOYy4apVpxl0Sqd0uOddZwlf+dAmDC5tLhHmpwIc TeL+antyB2YUvkZXRxQ5tfF4D6j5IhTHebPMAU7MdjCpU2/DbqBy6hCuxDzU0d3zH2Lh0ylrM5b dnhiC8vMI+Jx+d1EagPvsA1xmB13g8SkfOGf9cs00laR+8qvum6DJM9PknqC/Yl1oDL+laQEj99 wrJnbhHWWKntKqeJs3fDkWxyHQm5YBvVQ5is0zzxoC6IK4kbwk9DuPi60ZyoDRrAnmxrofhBkFW uQB1GWtuFQbCxsmIwIaB/qWThfrhnhS1++gHQiZQ+yZSh4m/dGu98C80hnu8htxgbByJMQbYtJI Z/bjJU/dpO3wuL+ZK4J1srA7Im/uWQ1afL4LJum8aGfjA== X-Received: by 2002:a05:600c:19cd:b0:49d:2536:402e with SMTP id 5b1f17b1804b1-49ff06f7a45mr28683465e9.30.1790328488006; Fri, 25 Sep 2026 02:28:08 -0700 (PDT) Received: from ?IPV6:2a07:de40:8100:0:89a9:fd0e:583d:4a53? ([2001:af0:8000:1409:193:86:92:181]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49fee6ced69sm37140085e9.0.2026.09.25.02.28.06 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 25 Sep 2026 02:28:07 -0700 (PDT) Message-ID: Date: Fri, 25 Sep 2026 11:28:06 +0200 Precedence: bulk X-Mailing-List: linux-modules@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v11 20/38] dyndbg,module: make proper substructs in _ddebug_info To: Jim Cromie Cc: Jason Baron , Shuah Khan , Andrew Morton , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Arnd Bergmann , Luis Chamberlain , Daniel Gomez , Sami Tolvanen , Aaron Tomlin , Jonathan Corbet , Greg Kroah-Hartman , Nathan Chancellor , Nicolas Schier , Shuah Khan , Randy Dunlap , "Rafael J. Wysocki" , Pavel Machek , Len Brown , Jonathan Corbet , Petr Mladek , Steven Rostedt , John Ogness , Sergey Senozhatsky , linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arch@vger.kernel.org, linux-modules@vger.kernel.org, linux-doc@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-pm@vger.kernel.org, Louis Chauvet References: <20260923-dd-cmap-part2-clean-v11-0-9b6c217fdf2f@gmail.com> <20260923-dd-cmap-part2-clean-v11-20-9b6c217fdf2f@gmail.com> Content-Language: en-US From: Petr Pavlu In-Reply-To: <20260923-dd-cmap-part2-clean-v11-20-9b6c217fdf2f@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 9/24/26 12:34 AM, Jim Cromie wrote: > recompose struct _ddebug_info, inserting proper sub-structs. > > The struct _ddebug_info has 2 pairs of _vec, num_##_vec fields, for > descs and classes respectively. for_subvec() makes walking these > vectors less cumbersome, now lets move those field pairs into their > own "vec" structs: _ddebug_descs & _ddebug_class_maps, and re-compose > struct _ddebug_info to contain them cleanly. This also lets us get > rid of for_subvec()'s num_##_vec paste-up. > > Also recompose struct ddebug_table to contain a _ddebug_info. This > reinforces _ddebug_info's use as a cursor into relevant data for a > builtin module, and access to the full _ddebug state for modules. > > NOTES: > > rename section:__dyndbg_classes to _class_maps, to better align with > struct _ddebug_class_maps. > > names together, for more obvious name pairing. > > Invariant: These vectors ref a contiguous subrange of __section memory > in builtin/DATA or in loadable modules via mod->dyndbg_info; with > guaranteed life-time for us. > > struct module contains a _ddebug_info field and module/main.c sets it > up, so that gets adjusted rather obviously. > > Signed-off-by: Jim Cromie > Reviewed-by: Louis Chauvet > --- > v8: set maps.len = nc unconditionally > v3: squash in section name changes. > > v2: > > Move RvB after SoB > In structs _ddebug_descs & _ddebug_class_maps, change int length to unsigned int > No use of <0 vals is contemplated. > > dyndbg: improve section names > > change __dyndbg to __dyndbg_descs > change __dyndbg_classes to __dyndbg_class_maps > > this sets up for adding __dyndbg_class_users > > fixup-drmprint > --- > include/asm-generic/dyndbg.lds.h | 14 +++--- > include/linux/dynamic_debug.h | 34 ++++++++----- > kernel/module/main.c | 12 ++--- > lib/dynamic_debug.c | 101 +++++++++++++++++++-------------------- > 4 files changed, 86 insertions(+), 75 deletions(-) Acked-by: Petr Pavlu # modules -- Petr