linux-kernel - Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

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]

Date:	Wed, 18 Mar 2015 12:56:18 -0400
From:	Tejun Heo <tj@...nel.org>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	"Luis R . Rodriguez" <mcgrof@...e.com>,
	linux-kernel@...r.kernel.org,
	Arjan van de Ven <arjan@...ux.intel.com>,
	Rusty Russell <rusty@...tcorp.com.au>,
	Olof Johansson <olof@...om.net>,
	Tetsuo Handa <penguin-kernel@...ove.SAKURA.ne.jp>
Subject: Re: [PATCH 6/8] amd64_edac: enforce synchronous probe

On Fri, Jan 16, 2015 at 03:33:15PM -0800, Dmitry Torokhov wrote:
> From: Luis R. Rodriguez <mcgrof@...e.com>
> 
> While testing asynchronous PCI probe on this driver I noticed it failed
> so enforce just synchronouse probe for now.  Asynchronous probe is not
> used by default and requires userepace intervention.  Patches for its
> support will be merged later.
> 
> The reason async probe fails is that the init call for this driver
> relies on probe to have finished for at least one device. This needs to
> be addressed before enabling async probe.

I'm still kinda uncomfortable with this both white and black list
behavior.  If we're gonna do this, let's please drop the debug options
and build proper blacklists; otherwise, this will never be complete
and we're gonna left with the in-between situation forever.

Thanks.

-- 
tejun
--
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/