Re: [PATCH 3/3] EDAC: amd64_edac: decide if driver can load successfully early.

From: Borislav Petkov
Date: Thu Mar 19 2015 - 11:36:51 EST


On Thu, Mar 19, 2015 at 11:29:57AM -0400, Tejun Heo wrote:
> This is a gloss layering violation. Please don't do things like this.
> I'm all for ripping out the hack even w/o considering the async probe
> issue.

And I don't want to leave the driver loaded when there's nothing to
be loaded for. One instance in this driver's specific case is one
northbridge and there are numascale boxes with hundreds of northbridges.

If you have a better idea about how to unload the driver, asynchronously
or not, after all probe() calls have failed, I'm all ears.

Thanks.

--
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/