linux-kernel - Re: [PATCH 3/3] x86, perf, uncore: Don't make MSR uncore depend on PCI uncore

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] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.11.1506190051540.4107@nanos>
Date:	Fri, 19 Jun 2015 00:57:58 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Andi Kleen <ak@...ux.intel.com>
cc:	Andi Kleen <andi@...stfloor.org>, peterz@...radead.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 3/3] x86, perf, uncore: Don't make MSR uncore depend on
 PCI uncore

On Thu, 18 Jun 2015, Andi Kleen wrote:
> > One possible solution is to split the initcall and have one
> > for uncore_pci and one for uncode_msr, but that does not work well if
> > you want to make it a module.
> > 
> > But we should at least have some indication, what worked and what went
> > wrong instead of unconditionally returning success.
> 
> Nobody uses the return value for builtin drivers (short of one
> debug printk).

Which is a valuable tool to figure out WHY stuff does not work, which
you broke for a particular case.

> It would not load the module, but right now we don't have a module.

Though you keep the exit function around for making this modular. So
your argumentation does not make any sense at all.

Make it sloppy first, so it's more work to convert it to a module.
 
> Generally it's a bad idea to print something when a probe doesn't work,
> as that just leads to lots of dmesg spam for large monolithic kernels.

Generally it's bad to just hack stuff into submission and leave the
mess created for others to clean up.

You've been told that a gazillion times in the past, and I tell it
another time, even if I realized long ago, that you are completely
advisory resistant.

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/