Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1277164
| From | Vlastimil Babka <vbabka@suse.cz> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags |
| Date | 2015-11-25 11:30 +0100 |
| Message-ID | <qyFuO-4Xh-19@gated-at.bofh.it> (permalink) |
| References | <qyl33-86O-5@gated-at.bofh.it> <qylcK-8af-25@gated-at.bofh.it> <qyDt0-3y9-11@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 11/25/2015 09:16 AM, Joonsoo Kim wrote:
> On Tue, Nov 24, 2015 at 01:36:18PM +0100, Vlastimil Babka wrote:
>> --- a/include/trace/events/gfpflags.h
>> +++ b/include/trace/events/gfpflags.h
>> @@ -8,8 +8,8 @@
>> *
>> * Thus most bits set go first.
>> */
>> -#define show_gfp_flags(flags) \
>> - (flags) ? __print_flags(flags, "|", \
>> +
>> +#define __def_gfpflag_names \
>> {(unsigned long)GFP_TRANSHUGE, "GFP_TRANSHUGE"}, \
>> {(unsigned long)GFP_HIGHUSER_MOVABLE, "GFP_HIGHUSER_MOVABLE"}, \
>> {(unsigned long)GFP_HIGHUSER, "GFP_HIGHUSER"}, \
>> @@ -19,9 +19,13 @@
>> {(unsigned long)GFP_NOFS, "GFP_NOFS"}, \
>> {(unsigned long)GFP_ATOMIC, "GFP_ATOMIC"}, \
>> {(unsigned long)GFP_NOIO, "GFP_NOIO"}, \
>> + {(unsigned long)GFP_NOWAIT, "GFP_NOWAIT"}, \
>> + {(unsigned long)__GFP_DMA, "GFP_DMA"}, \
>> + {(unsigned long)__GFP_DMA32, "GFP_DMA32"}, \
>> {(unsigned long)__GFP_HIGH, "GFP_HIGH"}, \
>> {(unsigned long)__GFP_ATOMIC, "GFP_ATOMIC"}, \
>> {(unsigned long)__GFP_IO, "GFP_IO"}, \
>> + {(unsigned long)__GFP_FS, "GFP_FS"}, \
>> {(unsigned long)__GFP_COLD, "GFP_COLD"}, \
>> {(unsigned long)__GFP_NOWARN, "GFP_NOWARN"}, \
>> {(unsigned long)__GFP_REPEAT, "GFP_REPEAT"}, \
>> @@ -36,8 +40,12 @@
>> {(unsigned long)__GFP_RECLAIMABLE, "GFP_RECLAIMABLE"}, \
>> {(unsigned long)__GFP_MOVABLE, "GFP_MOVABLE"}, \
>> {(unsigned long)__GFP_NOTRACK, "GFP_NOTRACK"}, \
>> + {(unsigned long)__GFP_WRITE, "GFP_WRITE"}, \
>> {(unsigned long)__GFP_DIRECT_RECLAIM, "GFP_DIRECT_RECLAIM"}, \
>> {(unsigned long)__GFP_KSWAPD_RECLAIM, "GFP_KSWAPD_RECLAIM"}, \
>> {(unsigned long)__GFP_OTHER_NODE, "GFP_OTHER_NODE"} \
>> - ) : "GFP_NOWAIT"
>>
>> +#define show_gfp_flags(flags) \
>> + (flags) ? __print_flags(flags, "|", \
>> + __def_gfpflag_names \
>> + ) : "none"
>
> How about moving this to gfp.h or something?
> Now, we use it in out of tracepoints so there is no need to keep it
> in include/trace/events/xxx.
Hm I didn't want to pollute such widely included header with such defines. And
show_gfp_flags shouldn't be there definitely as it depends on __print_flags.
What do others think?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 0/9] page_owner improvements for debugging Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:40 +0100
[PATCH v2 4/9] mm, page_owner: copy page owner info during migration Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:40 +0100
[PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:40 +0100
Re: [PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Michal Hocko <mhocko@kernel.org> - 2015-11-25 16:00 +0100
Re: [PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Vlastimil Babka <vbabka@suse.cz> - 2015-11-25 16:10 +0100
Re: [PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Peter Zijlstra <peterz@infradead.org> - 2015-11-25 16:30 +0100
Re: [PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Michal Hocko <mhocko@kernel.org> - 2015-11-25 16:50 +0100
[PATCH v2 2/9] mm, page_owner: print symbolic migratetype of both page and pageblock Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:40 +0100
Re: [PATCH v2 2/9] mm, page_owner: print symbolic migratetype of both page and pageblock Joonsoo Kim <iamjoonsoo.kim@lge.com> - 2015-11-25 09:20 +0100
[PATCH v2 8/9] mm, page_alloc: print symbolic gfp_flags on allocation failure Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:50 +0100
Re: [PATCH v2 8/9] mm, page_alloc: print symbolic gfp_flags on allocation failure Michal Hocko <mhocko@kernel.org> - 2015-11-25 15:40 +0100
[PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags Vlastimil Babka <vbabka@suse.cz> - 2015-11-24 13:50 +0100
Re: [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags Joonsoo Kim <iamjoonsoo.kim@lge.com> - 2015-11-25 09:20 +0100
Re: [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags Vlastimil Babka <vbabka@suse.cz> - 2015-11-25 11:30 +0100
Re: [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags yalin wang <yalin.wang2010@gmail.com> - 2015-11-27 04:50 +0100
Re: [PATCH v2 0/9] page_owner improvements for debugging Michal Hocko <mhocko@kernel.org> - 2015-11-25 15:40 +0100
csiph-web