Re: [PATCH v12 2/5] soc: qcom: Add Sleep stats driver

Shawn Guo
2021-10-19T09:47:36Z

On Mon, Oct 18, 2021 at 07:45:30PM +0530, Maulik Shah wrote:
> > > +static void qcom_create_soc_sleep_stat_files(struct dentry *root, void __iomem *reg,
> > > + struct stats_data *d,
> > > + const struct stats_config *config)
> > > +{
> > > + char stat_type[sizeof(u32) + 1] = {0};
> > > + size_t stats_offset = config->stats_offset;
> > > + u32 offset = 0, type;
> > > + int i, j;
> > > +
> > > + /*
> > > + * On RPM targets, stats offset location is dynamic and changes from target
> > > + * to target and sometimes from build to build for same target.
> > > + *
> > > + * In such cases the dynamic address is present at 0x14 offset from base
> > > + * address in devicetree. The last 16bits indicates the stats_offset.
> > > + */
> > > + if (config->dynamic_offset) {
> > > + stats_offset = readl(reg + RPM_DYNAMIC_ADDR);
> > > + stats_offset &= RPM_DYNAMIC_ADDR_MASK;
> > > + }
> > > +
> > > + for (i = 0; i < config->num_records; i++) {
> > > + d[i].base = reg + offset + stats_offset;
> > > +
> > > + /*
> > > + * Read the low power mode name and create debugfs file for it.
> > > + * The names read could be of below,
> > > + * (may change depending on low power mode supported).
> > > + * For rpmh-sleep-stats: "aosd", "cxsd" and "ddr".
> > > + * For rpm-sleep-stats: "vmin" and "vlow".
> >
> > It reports 'vmin' and 'xosd' on MSM8939, 'vmin' and 'vlow' on SDM660.
> > I know that 'vmin' is VDD Minimization mode, and 'xosd' is XO Shutdown
> > mode. But I'm not sure about 'vlow' mode. Could you share some
> > information regarding what this low power mode is, and how it differs
> > from 'vmin' and 'xosd'?
>
> vlow and xosd are same.
> vmin is xosd plus voltage minimization of chip, memory rails.

Thanks much for the info, Maulik!

I'm running your driver on qcm2290 and trying to reach vlow mode.

# cat /sys/kernel/debug/qcom_sleep_stats/vlow
Count: 0
Last Entered At: 0
Last Exited At: 0
Accumulated Duration: 0
Client Votes: 0x81
# echo mem > /sys/power/state
[ 551.446603] PM: suspend entry (s2idle)
[ 551.450948] Filesystems sync: 0.000 seconds
[ 551.462828] Freezing user space processes ... (elapsed 0.002 seconds) done.
[ 551.472276] OOM killer disabled.
[ 551.475556] Freezing remaining freezable tasks ... (elapsed 0.001 seconds) done.
[ 551.484461] printk: Suspending console(s) (use no_console_suspend to debug)
[ 551.561280] OOM killer enabled.
[ 551.564461] Restarting tasks ... done.
[ 551.569652] PM: suspend exit
# cat /sys/kernel/debug/qcom_sleep_stats/vlow
Count: 0
Last Entered At: 0
Last Exited At: 0
Accumulated Duration: 0
Client Votes: 0x818081

The count doesn't increases along with suspend/resume cycle at the
moment. But as you can see, 'Client Votes' field changes. If possible,
could you shed some light on what this means?

As the comparison, I'm also running the downstream 'rpm_master_stats'
driver in the same kernel, and the 'xo_count' field of APSS does
increase along with suspend/resume cycle. May I ask some advices what
I'm possibly missing and thus getting different result between 'vlow'
and 'rpm_master_stats' report?

# cat /sys/kernel/debug/rpm_master_stats
APSS
shutdown_req:0x37EA3CC74
wakeup_ind:0x0
bringup_req:0x37F041958
bringup_ack:0x37F042D54
xo_last_entered_at:0x286FF36AC
xo_last_exited_at:0x28AF94178
xo_accumulated_duration:0x3EDD55B
last_sleep_transition_duration:0x122f
last_wake_transition_duration:0x11f8
xo_count:0x1
wakeup_reason:0x0
numshutdowns:0x641
active_cores:0x1
core0
MPSS
shutdown_req:0x0
wakeup_ind:0x0
bringup_req:0x0
bringup_ack:0x0
xo_last_entered_at:0x0
xo_last_exited_at:0x0
xo_accumulated_duration:0x0
last_sleep_transition_duration:0x0
last_wake_transition_duration:0x0
xo_count:0x0
wakeup_reason:0x0
numshutdowns:0x0
active_cores:0x1
core0
ADSP
shutdown_req:0x0
wakeup_ind:0x0
bringup_req:0x0
bringup_ack:0x0
xo_last_entered_at:0x0
xo_last_exited_at:0x0
xo_accumulated_duration:0x0
last_sleep_transition_duration:0x0
last_wake_transition_duration:0x0
xo_count:0x0
wakeup_reason:0x0
numshutdowns:0x0
active_cores:0x1
core0
CDSP
shutdown_req:0x0
wakeup_ind:0x0
bringup_req:0x0
bringup_ack:0x0
xo_last_entered_at:0x0
xo_last_exited_at:0x0
xo_accumulated_duration:0x0
last_sleep_transition_duration:0x0
last_wake_transition_duration:0x0
xo_count:0x0
wakeup_reason:0x0
numshutdowns:0x0
active_cores:0x0
TZ
shutdown_req:0x0
wakeup_ind:0x0
bringup_req:0x0
bringup_ack:0x0
xo_last_entered_at:0x0
xo_last_exited_at:0x0
xo_accumulated_duration:0x0
last_sleep_transition_duration:0x0
last_wake_transition_duration:0x0
xo_count:0x0
wakeup_reason:0x0
numshutdowns:0x0
active_cores:0x0

Any comment or suggestion would be much appreciated!

Shawn