答复: [External Mail]Re: [PATCH v2 1/1] mm/cma: add 'available count' and 'total count' to trace_cma_alloc_start

[高翔 <gaoxiang17@xiaomi.com>]

[-- Attachment #1: Type: text/plain, Size: 2517 bytes --]

Is "request count" also written in the title? The previous "count" was changed to "request_count"


________________________________
发件人: David Hildenbrand <david@redhat.com>
发送时间: 2025年8月20日 17:14:10
收件人: Xiang Gao; akpm@linux-foundation.org
抄送: lorenzo.stoakes@oracle.com; Liam.Howlett@oracle.com; vbabka@suse.cz; rppt@kernel.org; surenb@google.com; mhocko@suse.com; linux-mm@kvack.org; linux-kernel@vger.kernel.org; 高翔
主题: [External Mail]Re: [PATCH v2 1/1] mm/cma: add 'available count' and 'total count' to trace_cma_alloc_start

[外部邮件] 此邮件来源于小米公司外部，请谨慎处理。若对邮件安全性存疑，请将邮件转发给misec@xiaomi.com进行反馈

On 20.08.25 03:53, Xiang Gao wrote:
> From: gaoxiang17 <gaoxiang17@xiaomi.com>
>
> This makes cma info more intuitive during debugging.
>
> Signed-off-by: gaoxiang17 <gaoxiang17@xiaomi.com>
> ---
>   include/trace/events/cma.h | 19 +++++++++++++------
>   mm/cma.c                   |  2 +-
>   2 files changed, 14 insertions(+), 7 deletions(-)
>
> diff --git a/include/trace/events/cma.h b/include/trace/events/cma.h
> index 383c09f583ac..5655ac521fb5 100644
> --- a/include/trace/events/cma.h
> +++ b/include/trace/events/cma.h
> @@ -38,25 +38,32 @@ TRACE_EVENT(cma_release,
>
>   TRACE_EVENT(cma_alloc_start,
>
> -     TP_PROTO(const char *name, unsigned long count, unsigned int align),
> +     TP_PROTO(const char *name, unsigned long request_count, unsigned long available_count,
> +             unsigned long total_count, unsigned int align),
>
> -     TP_ARGS(name, count, align),
> +     TP_ARGS(name, request_count, available_count, total_count, align),
>
>       TP_STRUCT__entry(
>               __string(name, name)
> -             __field(unsigned long, count)
> +             __field(unsigned long, request_count)
> +             __field(unsigned long, available_count)
> +             __field(unsigned long, total_count)
>               __field(unsigned int, align)
>       ),
>
>       TP_fast_assign(
>               __assign_str(name);
> -             __entry->count = count;
> +             __entry->count = request_count;
> +             __entry->available_count = available_count;
> +             __entry->total_count = total_count;
>               __entry->align = align;
>       ),
>
> -     TP_printk("name=%s count=%lu align=%u",
> +     TP_printk("name=%s count=%lu request_count=%lu total_count=%lu align=%u",

"requested_count" ?

--
Cheers

David / dhildenb


[-- Attachment #2: Type: text/html, Size: 5278 bytes --]