<html><head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> </head> <body> <div class="moz-cite-prefix">It again fails with the same message! But this time it is different! Here's why: openat(AT_FDCWD, "/sys/class/drm/card0/device/pp_dpm_fclk", O_RDONLY) = 3 read(3, "0: 571Mhz \n1: 1274Mhz *\n2: 1221M"..., 8191) = 36 read(3, "", 8191) = 0 close(3) = 0 write(2, "python3: /home/ltuikov/proj/amd/"..., 220python3: /home/ltuikov/proj/amd/rocm_smi_lib/src/rocm_smi.cc:913: rsmi_status_t get_frequencies(amd::smi::DevInfoTypes, uint32_t, rsmi_frequencies_t*, uint32_t*): Assertion `f->frequency[i-1] <= f->frequency[i]' failed. ) = 220 mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f531f9bc000 rt_sigprocmask(SIG_UNBLOCK, [ABRT], NULL, 8) = 0 rt_sigprocmask(SIG_BLOCK, ~[RTMIN RT_1], [], 8) = 0 getpid() = 37861 gettid() = 37861 tgkill(37861, 37861, SIGABRT) = 0 rt_sigprocmask(SIG_SETMASK, [], NULL, 8) = 0 --- SIGABRT {si_signo=SIGABRT, si_code=SI_TKILL, si_pid=37861, si_uid=1000} --- +++ killed by SIGABRT (core dumped) +++ Aborted (core dumped) $cat /sys/class/drm/card0/device/pp_dpm_fclk 0: 571Mhz 1: 1274Mhz * 2: 1221Mhz $_ Why is the mid frequency larger than the last? Why does get_frequencies() insists that they be ordered when they're not? (Does the tool need fixing or the kernel?) The current patchset doesn't report 0, and doesn't report any current if 0 would've been reported as current. But anything else is reported as it would've been reported before the patch. And I tested it with vanilla amd-staging-drm-next--same thing. Regards, Luben On 2021-10-19 09:25, Russell, Kent wrote: </div> <blockquote type="cite" cite="mid:DM6PR12MB3324ECC96ADE558E587AAFF085BD9@DM6PR12MB3324.namprd12.prod.outlook.com"> <meta name="ProgId" content="Word.Document"> <meta name="Generator" content="Microsoft Word 15"> <meta name="Originator" content="Microsoft Word 15"> <link rel="File-List" href="cid:filelist.xml@01D7C4CB.4CFC7E60"> <link rel="Edit-Time-Data" href="cid:editdata.mso">  <style>@font-face {font-family:"Cambria Math"; panose-1:2 4 5 3 5 4 6 3 2 4; mso-font-charset:0; mso-generic-font-family:roman; mso-font-pitch:variable; mso-font-signature:-536869121 1107305727 33554432 0 415 0;}@font-face {font-family:Calibri; panose-1:2 15 5 2 2 2 4 3 2 4; mso-font-charset:0; mso-generic-font-family:swiss; mso-font-pitch:variable; mso-font-signature:-469750017 -1073732485 9 0 511 0;}@font-face {font-family:"Times New Roman \;color\:black"; panose-1:0 0 0 0 0 0 0 0 0 0; mso-font-charset:0; mso-generic-font-family:roman; mso-font-format:other; mso-font-pitch:auto; mso-font-signature:0 0 0 0 0 0;}p.MsoNormal, li.MsoNormal, div.MsoNormal {mso-style-unhide:no; mso-style-qformat:yes; mso-style-parent:""; margin:0in; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}a:link, span.MsoHyperlink {mso-style-noshow:yes; mso-style-priority:99; color:#0563C1; text-decoration:underline; text-underline:single;}a:visited, span.MsoHyperlinkFollowed {mso-style-noshow:yes; mso-style-priority:99; color:purple; text-decoration:underline; text-underline:single;}p.msonormal0, li.msonormal0, div.msonormal0 {mso-style-name:msonormal; mso-style-unhide:no; margin:0in; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}p.xmsonormal, li.xmsonormal, div.xmsonormal {mso-style-name:x_msonormal; mso-style-unhide:no; margin:0in; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}p.xmsipheadera4477989, li.xmsipheadera4477989, div.xmsipheadera4477989 {mso-style-name:x_msipheadera4477989; mso-style-unhide:no; mso-margin-top-alt:auto; margin-right:0in; mso-margin-bottom-alt:auto; margin-left:0in; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}p.xmsochpdefault, li.xmsochpdefault, div.xmsochpdefault {mso-style-name:x_msochpdefault; mso-style-unhide:no; mso-margin-top-alt:auto; margin-right:0in; mso-margin-bottom-alt:auto; margin-left:0in; mso-pagination:widow-orphan; font-size:10.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}p.msipheader9f8d35bc, li.msipheader9f8d35bc, div.msipheader9f8d35bc {mso-style-name:msipheader9f8d35bc; mso-style-unhide:no; mso-margin-top-alt:auto; margin-right:0in; mso-margin-bottom-alt:auto; margin-left:0in; mso-pagination:widow-orphan; font-size:11.0pt; font-family:"Calibri",sans-serif; mso-fareast-font-family:Calibri;}span.xmsohyperlink {mso-style-name:x_msohyperlink; mso-style-unhide:no; color:#0563C1; text-decoration:underline; text-underline:single;}span.xemailstyle18 {mso-style-name:x_emailstyle18; mso-style-unhide:no; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-hansi-font-family:Calibri; mso-bidi-font-family:Calibri; color:windowtext;}span.EmailStyle24 {mso-style-type:personal; mso-style-noshow:yes; mso-style-unhide:no; mso-ansi-font-size:11.0pt; mso-bidi-font-size:11.0pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-fareast-font-family:Calibri; mso-hansi-font-family:Calibri; mso-bidi-font-family:"Times New Roman"; color:windowtext;}span.EmailStyle25 {mso-style-type:personal; mso-style-noshow:yes; mso-style-unhide:no; mso-ansi-font-size:11.0pt; mso-bidi-font-size:11.0pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-fareast-font-family:Calibri; mso-hansi-font-family:Calibri; mso-bidi-font-family:"Times New Roman"; color:windowtext;}span.EmailStyle26 {mso-style-type:personal-reply; mso-style-noshow:yes; mso-style-unhide:no; mso-ansi-font-size:11.0pt; mso-bidi-font-size:11.0pt; font-family:"Calibri",sans-serif; mso-ascii-font-family:Calibri; mso-fareast-font-family:Calibri; mso-hansi-font-family:Calibri; mso-bidi-font-family:"Times New Roman"; color:windowtext;}.MsoChpDefault {mso-style-type:export-only; mso-default-props:yes; font-size:10.0pt; mso-ansi-font-size:10.0pt; mso-bidi-font-size:10.0pt;}div.WordSection1 {page:WordSection1;}</style> <div class="WordSection1"> [AMD Official Use Only]<o:p></o:p> <o:p> </o:p> It was the rocm-smi -c flag. Maybe some work was done to make it more robust, that would be nice. But the -c flag is supposed to show the current frequency for each clock type. -g would do the same, but just for SCLK.<o:p></o:p> <o:p> </o:p> Kent<o:p></o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Tuikov, Luben <a class="moz-txt-link-rfc2396E" href="mailto:Luben.Tuikov@amd.com"><Luben.Tuikov@amd.com></a> Sent: Tuesday, October 19, 2021 12:27 AM To: Russell, Kent <a class="moz-txt-link-rfc2396E" href="mailto:Kent.Russell@amd.com"><Kent.Russell@amd.com></a>; Deucher, Alexander <a class="moz-txt-link-rfc2396E" href="mailto:Alexander.Deucher@amd.com"><Alexander.Deucher@amd.com></a>; Quan, Evan <a class="moz-txt-link-rfc2396E" href="mailto:Evan.Quan@amd.com"><Evan.Quan@amd.com></a>; Lazar, Lijo <a class="moz-txt-link-rfc2396E" href="mailto:Lijo.Lazar@amd.com"><Lijo.Lazar@amd.com></a>; <a class="moz-txt-link-abbreviated moz-txt-link-freetext" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> Cc: Kasiviswanathan, Harish <a class="moz-txt-link-rfc2396E" href="mailto:Harish.Kasiviswanathan@amd.com"><Harish.Kasiviswanathan@amd.com></a> Subject: Re: [PATCH 0/5] 0 MHz is not a valid current frequency<o:p></o:p> </div> </div> <o:p> </o:p> <div> Kent, What is the command which fails? I can try to duplicate it here. So far, things I've tried, I cannot make rocm-smi fail. Command arguments? Regards, Luben On 2021-10-18 21:06, Russell, Kent wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> [AMD Official Use Only]<o:p></o:p> <o:p></o:p> The * is required for the rocm-smi’s functionality for showing what the current clocks are. We had a bug before where the * was removed, then the SMI died fantastically. Work could be done to try to handle that type of situation, but the SMI has a “show current clocks” and uses the * to determine which one is active<o:p></o:p> <o:p></o:p> Kent<o:p></o:p> <o:p></o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: amd-gfx <a href="mailto:amd-gfx-bounces@lists.freedesktop.org" moz-do-not-send="true"><amd-gfx-bounces@lists.freedesktop.org></a> On Behalf Of Russell, Kent Sent: Monday, October 18, 2021 9:05 PM To: Tuikov, Luben <a href="mailto:Luben.Tuikov@amd.com" moz-do-not-send="true"><Luben.Tuikov@amd.com></a>; Deucher, Alexander <a href="mailto:Alexander.Deucher@amd.com" moz-do-not-send="true"><Alexander.Deucher@amd.com></a>; Quan, Evan <a href="mailto:Evan.Quan@amd.com" moz-do-not-send="true"><Evan.Quan@amd.com></a>; Lazar, Lijo <a href="mailto:Lijo.Lazar@amd.com" moz-do-not-send="true"> <Lijo.Lazar@amd.com></a>; <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext">amd-gfx@lists.freedesktop.org</a> Cc: Kasiviswanathan, Harish <a href="mailto:Harish.Kasiviswanathan@amd.com" moz-do-not-send="true"> <Harish.Kasiviswanathan@amd.com></a> Subject: RE: [PATCH 0/5] 0 MHz is not a valid current frequency<o:p></o:p> </div> </div> <o:p></o:p> [AMD Official Use Only]<o:p></o:p> <o:p></o:p> +Harish, rocm-smi falls under his purview now.<o:p></o:p> <o:p></o:p> Kent<o:p></o:p> <o:p></o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Tuikov, Luben <<a href="mailto:Luben.Tuikov@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Luben.Tuikov@amd.com</a>> Sent: Monday, October 18, 2021 4:30 PM To: Deucher, Alexander <<a href="mailto:Alexander.Deucher@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Alexander.Deucher@amd.com</a>>; Quan, Evan <<a href="mailto:Evan.Quan@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Evan.Quan@amd.com</a>>; Lazar, Lijo <<a href="mailto:Lijo.Lazar@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Lijo.Lazar@amd.com</a>>; <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext">amd-gfx@lists.freedesktop.org</a>; Russell, Kent <<a href="mailto:Kent.Russell@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Kent.Russell@amd.com</a>> Subject: Re: [PATCH 0/5] 0 MHz is not a valid current frequency<o:p></o:p> </div> </div> <o:p></o:p> <div> I think Kent is already seen these patches as he did comment on 1/5 patch. The v3 version of the patch, posted last week, removes the asterisk to report the lowest frequency as the current frequency, when the current frequency is 0, i.e. when the block is in low power state. Does the tool rely on the asterisk? If this information is necessary could it not use amdgpu_pm_info? Regards, Luben On 2021-10-18 16:19, Deucher, Alexander wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> [Public]<o:p></o:p> <o:p></o:p> <div> <div> We the current behavior (0 for clock) already crashes the tool, so I don't think we can really make things worse.<o:p></o:p> </div> <div> <o:p></o:p> </div> <div> Alex<o:p></o:p> </div> <div> <o:p></o:p> </div> <div class="MsoNormal" style="text-align:center" align="center"> <hr width="98%" size="1" align="center"> </div> <div id="divRplyFwdMsg"> From: Quan, Evan <a href="mailto:Evan.Quan@amd.com" moz-do-not-send="true"><Evan.Quan@amd.com></a> Sent: Thursday, October 14, 2021 10:25 PM To: Lazar, Lijo <a href="mailto:Lijo.Lazar@amd.com" moz-do-not-send="true"><Lijo.Lazar@amd.com></a>; Tuikov, Luben <a href="mailto:Luben.Tuikov@amd.com" moz-do-not-send="true"><Luben.Tuikov@amd.com></a>; <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext"> amd-gfx@lists.freedesktop.org</a> <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true"> <amd-gfx@lists.freedesktop.org></a>; Russell, Kent <a href="mailto:Kent.Russell@amd.com" moz-do-not-send="true"> <Kent.Russell@amd.com></a> Cc: Deucher, Alexander <a href="mailto:Alexander.Deucher@amd.com" moz-do-not-send="true"><Alexander.Deucher@amd.com></a> Subject: RE: [PATCH 0/5] 0 MHz is not a valid current frequency <o:p></o:p> <div> <o:p></o:p> </div> </div> <div> <div> [AMD Official Use Only]<o:p></o:p> <o:p></o:p> +Kent who maintains the Rocm tool<o:p></o:p> <o:p></o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: amd-gfx <a href="mailto:amd-gfx-bounces@lists.freedesktop.org" moz-do-not-send="true"> <amd-gfx-bounces@lists.freedesktop.org></a> On Behalf Of Lazar, Lijo Sent: Thursday, October 14, 2021 1:07 AM To: Tuikov, Luben <a href="mailto:Luben.Tuikov@amd.com" moz-do-not-send="true"><Luben.Tuikov@amd.com></a>; <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext">amd-gfx@lists.freedesktop.org</a> Cc: Deucher, Alexander <a href="mailto:Alexander.Deucher@amd.com" moz-do-not-send="true"><Alexander.Deucher@amd.com></a> Subject: Re: [PATCH 0/5] 0 MHz is not a valid current frequency<o:p></o:p> </div> </div> <o:p></o:p> [AMD Official Use Only]<o:p></o:p> <o:p></o:p> <div> [AMD Official Use Only]<o:p></o:p> <o:p></o:p> <div> <div> >Or maybe just a list without default hint, i.e. no asterisk?<o:p></o:p> </div> <div> <o:p></o:p> </div> <div> I think this is also fine meaning we are having trouble in determining the current frequency or DPM level. Evan/Alex? Don't know if this will crash the tools.<o:p></o:p> </div> <div id="x_ms-outlook-mobile-signature"> <div> <o:p></o:p> </div> Thanks, Lijo<o:p></o:p> </div> <div class="MsoNormal" style="text-align:center" align="center"> <hr width="98%" size="1" align="center"> </div> <div id="x_divRplyFwdMsg"> From: Tuikov, Luben <<a href="mailto:Luben.Tuikov@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Luben.Tuikov@amd.com</a>> Sent: Wednesday, October 13, 2021 9:52:09 PM To: Lazar, Lijo <<a href="mailto:Lijo.Lazar@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Lijo.Lazar@amd.com</a>>; <a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext">amd-gfx@lists.freedesktop.org</a> <<a href="mailto:amd-gfx@lists.freedesktop.org" moz-do-not-send="true" class="moz-txt-link-freetext">amd-gfx@lists.freedesktop.org</a>> Cc: Deucher, Alexander <<a href="mailto:Alexander.Deucher@amd.com" moz-do-not-send="true" class="moz-txt-link-freetext">Alexander.Deucher@amd.com</a>> Subject: Re: [PATCH 0/5] 0 MHz is not a valid current frequency <o:p> </o:p> <div> <o:p></o:p> </div> </div> <div> <div> On 2021-10-13 00:14, Lazar, Lijo wrote: > > On 10/13/2021 8:40 AM, Luben Tuikov wrote: >> Some ASIC support low-power functionality for the whole ASIC or just >> an IP block. When in such low-power mode, some sysfs interfaces would >> report a frequency of 0, e.g., >> >> $cat /sys/class/drm/card0/device/pp_dpm_sclk >> 0: 500Mhz >> 1: 0Mhz * >> 2: 2200Mhz >> $_ >> >> An operating frequency of 0 MHz doesn't make sense, and this interface >> is designed to report only operating clock frequencies, i.e. non-zero, >> and possibly the current one. >> >> When in this low-power state, round to the smallest >> operating frequency, for this interface, as follows, >> > Would rather avoid this - > > 1) It is manipulating FW reported value. If at all there is an uncaught > issue in FW reporting of frequency values, that is masked here. > 2) Otherwise, if 0MHz is described as GFX power gated case, this > provides a convenient interface to check if GFX is power gated. > > If seeing a '0' is not pleasing, consider changing to something like > "NA" - not available (frequency cannot be fetched at the moment). There's a ROCm tool which literally asserts if the values are not ordered in increasing order. Now since 0 < 550, but 0 is listed as the second entry, the tool simply asserts and crashes. It is not clear what you'd rather see here: $cat /sys/class/drm/card0/device/pp_dpm_sclk 0: 550Mhz 1: N/A * 2: 2200MHz $_ Is this what you want to see? (That'll crash other tools which expect %uMhz.) Or maybe just a list without default hint, i.e. no asterisk? $cat /sys/class/drm/card0/device/pp_dpm_sclk 0: 550Mhz 1: 2200MHz $_ What should the output be? We want to avoid showing 0, but still show numbers. Regards, Luben > > Thanks, > Lijo > >> $cat /sys/class/drm/card0/device/pp_dpm_sclk >> 0: 500Mhz * >> 1: 2200Mhz >> $_ >> >> Luben Tuikov (5): >> drm/amd/pm: Slight function rename >> drm/amd/pm: Rename cur_value to curr_value >> drm/amd/pm: Rename freq_values --> freq_value >> dpm/amd/pm: Sienna: 0 MHz is not a current clock frequency >> dpm/amd/pm: Navi10: 0 MHz is not a current clock frequency >> >> .../gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 60 +++++++++------ >> .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 73 ++++++++++++------- >> 2 files changed, 86 insertions(+), 47 deletions(-) >><o:p></o:p> </div> </div> </div> </div> </div> </div> </div> </div> </blockquote> <o:p></o:p> </div> </div> </blockquote> <o:p> </o:p> </div> </div> </blockquote> </body> </html>