summaryrefslogtreecommitdiff
path: root/drivers/platform
diff options
context:
space:
mode:
authorThomas Gleixner <tglx@linutronix.de>2017-02-15 13:11:50 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-02-15 19:49:58 +0300
commit47512cfd0d7a8bd6ab71d01cd89fca19eb2093eb (patch)
treeaefa65bf27020a76f1dc3ab883e809165672513f /drivers/platform
parent8d12f88257cd9039d9556585475deb46db055050 (diff)
downloadlinux-47512cfd0d7a8bd6ab71d01cd89fca19eb2093eb.tar.xz
x86/platform/goldfish: Prevent unconditional loading
The goldfish platform code registers the platform device unconditionally which causes havoc in several ways if the goldfish_pdev_bus driver is enabled: - Access to the hardcoded physical memory region, which is either not available or contains stuff which is completely unrelated. - Prevents that the interrupt of the serial port can be requested - In case of a spurious interrupt it goes into a infinite loop in the interrupt handler of the pdev_bus driver (which needs to be fixed seperately). Add a 'goldfish' command line option to make the registration opt-in when the platform is compiled in. I'm seriously grumpy about this engineering trainwreck, which has seven SOBs from Intel developers for 50 lines of code. And none of them figured out that this is broken. Impressive fail! Fixes: ddd70cf93d78 ("goldfish: platform device for x86") Reported-by: Gabriel C <nix.or.die@gmail.com> Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Cc: stable@vger.kernel.org Acked-by: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/platform')
0 files changed, 0 insertions, 0 deletions