linux-kernel - Re: [PATCH] irqdomain: remove the confused log

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.11.1508252131280.15006@nanos>
Date:	Tue, 25 Aug 2015 21:34:00 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Huang Shijie <shijie.huang@....com>
cc:	jiang.liu@...ux.intel.com, marc.zyngier@....com,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] irqdomain: remove the confused log

On Tue, 25 Aug 2015, Huang Shijie wrote:

> When the dynamic debug is enabled for irq domain, the kernel will print out:
>   ------------------------------------------------------
>     irqdomain:__irq_domain_add: irq: Added domain (null)
>   ------------------------------------------------------
> 
> It does not print out the irq domain's name, since the @domain->name is NULL.
> Keep the log here just makes people nervous.

Well, it might be NULL in your case, but there are enough cases where
its not NULL and the output valuable. 


Aside of that I don't understand what's making one nervous
here. People who enable that debug stuff should know that this can be
NULL and just deal with it. If they get nervous about it they probably
should not try to debug that code in the first place.

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/