diff options
| author |   Adrian Bunk <bunk@stusta.de> | 2006-04-19 00:07:28 +0200 |
|---|---|---|
| committer |   Dave Jones <davej@redhat.com> | 2006-04-18 17:24:52 -0500 |
| commit | 9180053cacfec4aa233a6cabf1256960e75b0abd (patch) | |
| tree | f8378ac361b4ae393a579d2e94161251ff1ed328 /arch/i386 | |
| parent | [CPUFREQ] Remove duplicate check in powernow-k8 (diff) | |
| download | linux-dev-9180053cacfec4aa233a6cabf1256960e75b0abd.tar.xz linux-dev-9180053cacfec4aa233a6cabf1256960e75b0abd.zip | |
[CPUFREQ] powernow-k8.c: fix a check-after-use
This patch fixes a check-after-use introduced by commit
4211a30349e8d2b724cfb4ce2584604f5e59c299 and spotted by the Coverity checker.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'arch/i386')
| -rw-r--r-- | arch/i386/kernel/cpu/cpufreq/powernow-k8.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/arch/i386/kernel/cpu/cpufreq/powernow-k8.c b/arch/i386/kernel/cpu/cpufreq/powernow-k8.c index 30bb6fd5b2b2..71fffa174425 100644 --- a/arch/i386/kernel/cpu/cpufreq/powernow-k8.c +++ b/arch/i386/kernel/cpu/cpufreq/powernow-k8.c @@ -905,14 +905,17 @@ static int powernowk8_target(struct cpufreq_policy *pol, unsigned targfreq, unsi { cpumask_t oldmask = CPU_MASK_ALL; struct powernow_k8_data *data = powernow_data[pol->cpu]; - u32 checkfid = data->currfid; - u32 checkvid = data->currvid; + u32 checkfid; + u32 checkvid; unsigned int newstate; int ret = -EIO; if (!data) return -EINVAL; + checkfid = data->currfid; + checkvid = data->currvid; + /* only run on specific CPU from here on */ oldmask = current->cpus_allowed; set_cpus_allowed(current, cpumask_of_cpu(pol->cpu)); |