diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-10-04 09:19:02 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-10-04 09:19:02 -0700 |
| commit | ce57e981f2b996aaca2031003b3f866368307766 (patch) | |
| tree | 7ab6b29e6b0013519f3418bbaf8f661e064861ba /virt/kvm/irq_comm.c | |
| parent | e1cc485262846dcad931bf85ee655cbbb815bfe6 (diff) | |
| download | lwn-ce57e981f2b996aaca2031003b3f866368307766.tar.gz lwn-ce57e981f2b996aaca2031003b3f866368307766.zip | |
firmware: use 'kernel_read()' to read firmware into kernel buffer
Fengguang correctly points out that the firmware reading should not use
vfs_read(), since the buffer is in kernel space.
The vfs_read() just happened to work for kernel threads, but sparse
warns about the incorrect address spaces, and it's definitely incorrect
and could fail for other users of the firmware loading.
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'virt/kvm/irq_comm.c')
0 files changed, 0 insertions, 0 deletions