| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sat, 22 Jun 2013 00:58:28 +0000 From: Shuah Khan <shuah.kh@...sung.com> To: "Rafael J. Wysocki" <rjw@...k.pl> Cc: "pavel@....cz" <pavel@....cz>, "len.brown@...el.com" <len.brown@...el.com>, "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>, "joe@...ches.com" <joe@...ches.com>, "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "shuahkhan@...il.com" <shuahkhan@...il.com>, Shuah Khan <shuah.kh@...sung.com> Subject: Re: [PATCHv v3] power: Include additional information in pm_print_times On 06/21/2013 06:15 PM, Rafael J. Wysocki wrote: > On Monday, June 17, 2013 01:36:35 PM Shuah Khan wrote: >> Change __device_suspend() path to include driver name and the ops that >> get run for a device. This additional information helps associate the >> driver and the type of pm_ops the device uses in the suspend path very >> quickly, which will aid in debugging problems in suspend and resume paths. >> Changed both start and end debug messages to include pm_ops information >> and use dev_info() instead of pr_info(). Changed the end message to include >> parent device information and have the same format as the start message. >> >> dmesg output before the change: >> >> [ 164.390032] calling 1-1+ @ 69, parent: usb1 >> [ 164.390035] call 1-1+ returned 0 after 0 usecs >> >> [ 164.390352] calling 00:0a+ @ 2457, parent: pnp0 >> [ 164.390357] call 00:0a+ returned 0 after 3 usecs >> >> [ 164.390361] calling 00:09+ @ 2457, parent: pnp0 >> [ 164.496458] call 00:09+ returned 0 after 103500 usecs >> >> [ 164.496494] calling 00:05+ @ 2457, parent: pnp0 >> [ 164.496511] call 00:05+ returned 0 after 14 usecs >> >> dmesg output after the change: >> >> [ 545.985394] usb 1-1: Start: type pm ops @ 68, parent: usb1 >> [ 545.987650] usb 1-1: End : type pm ops @ 68, parent: usb1 time(2184 usecs) err(0) >> >> [ 545.982544] system 00:0a: Start: legacy bus pm ops @ 2391, parent: pnp0 >> [ 545.982554] system 00:0a: End : legacy bus pm ops @ 2391, parent: pnp0 time(4 usecs) err(0) >> >> [ 545.982569] tpm_tis 00:09: Start: legacy bus pm ops @ 2391, parent: pnp0 >> [ 546.087017] tpm_tis 00:09: End : legacy bus pm ops @ 2391, parent: pnp0 time(101936 usecs) err(0) >> >> [ 546.087069] rtc_cmos 00:05: Start: legacy bus pm ops @ 2391, parent: pnp0 >> [ 546.087084] rtc_cmos 00:05: End : legacy bus pm ops @ 2391, parent: pnp0 time(11 usecs) err(0) >> > > I was about to apply your patch, but then I noticed something that might cause > problems to happen. > > Namely, there are tools that use these messages to create suspend/resume time > charts and they will stop working after the proposed changes. > > Please keep the existing formatting the way it is and only append the additional > information at the end of each line. > > Thanks, > Rafael > Hi Rafael, Yes changing the format would cause problems for scripts that rely on the exact format. Would you like to pick the v2 version of the patch that appends the additional information at the end, instead of changing the format? Here is the link. If you would like me to resend it, I can do that. https://lkml.org/lkml/2013/6/14/448 -- Shuah Shuah Khan, Linux Kernel Developer - Open Source Group Samsung Research America (Silicon Valley) shuah.kh@...sung.com | (970) 672-0658 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists