Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers

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

To: Colin Ian King <colin.i.king@xxxxxxxxx>
Subject: Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
Date: Tue, 25 Jan 2022 22:32:31 +0100
Cc: Ingo Molnar <mingo@xxxxxxxxxx>, Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>, Mark Rutland <mark.rutland@xxxxxxx>, Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx>, Jiri Olsa <jolsa@xxxxxxxxxx>, Namhyung Kim <namhyung@xxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Borislav Petkov <bp@xxxxxxxxx>, Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>, x86@xxxxxxxxxx, "H . Peter Anvin" <hpa@xxxxxxxxx>, linux-perf-users@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20220125201403.670011-1-colin.i.king@gmail.com>
References: <20220125201403.670011-1-colin.i.king@gmail.com>

On Tue, Jan 25, 2022 at 08:14:03PM +0000, Colin Ian King wrote:
> Pointers should be initialized with NULL rather than zero. Fix these.

Why ? This isn't C++, heck, this isn't even C, this is the kernel, we
hard rely on NULL being 0.

Follow-Ups:
- Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
  - From: Dan Carpenter

References:
- [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
  - From: Colin Ian King

Prev by Date: Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
Next by Date: Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
Previous by thread: Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
Next by thread: Re: [PATCH][next] perf/x86/rapl: replace 0 with NULL to initialize pointers
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]