Re: [RFC PATCH V4 5/5] cpuidle: cpuidle driver for apm
From: Stephen Rothwell
Date: Tue Mar 22 2011 - 21:15:36 EST
Hi,
On Tue, 22 Mar 2011 18:03:40 +0530 Trinabh Gupta <trinabh@xxxxxxxxxxxxxxxxxx> wrote:
>
> +static int apm_setup_cpuidle(int cpu)
> +{
> + struct cpuidle_device *dev = kzalloc(sizeof(struct cpuidle_device),
> + GFP_KERNEL);
Same as xen comments: no NULL check.
> + int count = CPUIDLE_DRIVER_STATE_START;
> + dev->cpu = cpu;
> + dev->drv = &apm_idle_driver;
Also wondering why you would ever have a different idle routine on
different cpus?
> +
> + dev->states[count] = state_apm_idle;
> + count++;
> +
> + dev->state_count = count;
> +
> + if (cpuidle_register_device(dev))
> + return -EIO;
And we leak dev.
> +static void apm_idle_exit(void)
> +{
> + cpuidle_unregister_driver(&apm_idle_driver);
Do we leak the per cpu device structure here?
> + return;
Unnecessary return statement.
--
Cheers,
Stephen Rothwell sfr@xxxxxxxxxxxxxxxx
http://www.canb.auug.org.au/~sfr/
Attachment:
pgp00000.pgp
Description: PGP signature