Re: [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Jaewon Kim <jaewon31.kim@xxxxxxxxxxx>
Subject: Re: [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area
From: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Date: Sun, 29 Mar 2020 09:08:58 -0700
Cc: walken@xxxxxxxxxx, bp@xxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, srostedt@xxxxxxxxxx, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, jaewon31.kim@xxxxxxxxx
In-reply-to: <20200320055823.27089-3-jaewon31.kim@samsung.com>
References: <20200320055823.27089-1-jaewon31.kim@samsung.com> <CGME20200320055839epcas1p189100549687530619d8a19919e8b5de0@epcas1p1.samsung.com> <20200320055823.27089-3-jaewon31.kim@samsung.com>

On Fri, Mar 20, 2020 at 02:58:23PM +0900, Jaewon Kim wrote:
> +	TP_printk("addr=%lx err=%ld total_vm=0x%lx flags=0x%lx len=0x%lx lo=0x%lx hi=0x%lx mask=0x%lx ofs=0x%lx\n",

Shouldn't addr be printed as 0x%lx?  I think it's arguable whether to print
len as %ld or 0x%lx.

Follow-Ups:
- Re: [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area
  - From: Kirill A. Shutemov

References:
- [PATCH v3 0/2] mm: mmap: add mmap trace point
  - From: Jaewon Kim
- [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area
  - From: Jaewon Kim

Prev by Date: Re: [RFC v2 2/2] kernel/sysctl: support handling command line aliases
Next by Date: Re: mmotm 2020-03-28-22-17 uploaded (staging/octeon/)
Previous by thread: [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area
Next by thread: Re: [PATCH v3 2/2] mm: mmap: add trace point of vm_unmapped_area
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]

Powered by Linux