summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohn Kacur <jkacur@redhat.com>2009-10-07 20:19:32 +0200
committerH. Peter Anvin <hpa@zytor.com>2009-10-07 15:41:21 -0700
commit170a0bc3808909d8ea0f3f9c725c6565efe7f9c4 (patch)
tree9b685c6fdf3be7f4889790d607d8bc0c15c359a3
parentd6c304055b3cecd4ca865769ac7cea97a320727b (diff)
downloadop-kernel-dev-170a0bc3808909d8ea0f3f9c725c6565efe7f9c4.zip
op-kernel-dev-170a0bc3808909d8ea0f3f9c725c6565efe7f9c4.tar.gz
x86, cpuid: Remove the bkl from cpuid_open()
Most of the variables are local to the function. It IS possible that for struct cpuinfo_x86 *c c could point to the same area. However, this is used read only. Signed-off-by: John Kacur <jkacur@redhat.com> LKML-Reference: <alpine.LFD.2.00.0910072016190.15183@localhost.localdomain> Signed-off-by: H. Peter Anvin <hpa@zytor.com>
-rw-r--r--arch/x86/kernel/cpuid.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/arch/x86/kernel/cpuid.c b/arch/x86/kernel/cpuid.c
index b07af88..ef69284 100644
--- a/arch/x86/kernel/cpuid.c
+++ b/arch/x86/kernel/cpuid.c
@@ -118,8 +118,6 @@ static int cpuid_open(struct inode *inode, struct file *file)
struct cpuinfo_x86 *c;
int ret = 0;
- lock_kernel();
-
cpu = iminor(file->f_path.dentry->d_inode);
if (cpu >= nr_cpu_ids || !cpu_online(cpu)) {
ret = -ENXIO; /* No such CPU */
@@ -129,7 +127,6 @@ static int cpuid_open(struct inode *inode, struct file *file)
if (c->cpuid_level < 0)
ret = -EIO; /* CPUID not supported */
out:
- unlock_kernel();
return ret;
}
OpenPOWER on IntegriCloud