Re: [PATCH] PM / devfreq: Rewrite devfreq_update_status() to fix multiple bugs

From: Saravana Kannan
Date: Mon Mar 03 2014 - 19:36:03 EST


On 02/27/2014 07:38 PM, Saravana Kannan wrote:
The current devfreq_update_status() has the following bugs:
- If previous frequency doesn't have a valid level, it does an out of bounds
access into the trans_table and causes memory corruption.
- When the new frequency doesn't have a valid level, the time spent in the
new frequency is counted towards the next valid frequency switch instead of
being ignored.
- The time spent on the previous frequency is added to the new frequency's
stats instead of the previous frequency's stats.

This patch fixes all of this.

Signed-off-by: Saravana Kannan <skannan@xxxxxxxxxxxxxx>
---

MyungJoo/Kyungmin,

Can you please take a look and let me know if this looks ok? Any possibility of getting this into 3.14?

Thanks,
Saravana

--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/