<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;"> <p>It comes from <span>amdgpu_query_gpu_info_init()</span></p> <p><br> </p> <p></p> <div><span style="font-family: "Courier New", monospace;"> for (i = 0; i < (int)dev->info.num_shader_engines; i++) {</span></div> <div><span style="font-family: "Courier New", monospace;"> unsigned instance = (i << AMDGPU_INFO_MMR_SE_INDEX_SHIFT) |</span></div> <div><span style="font-family: "Courier New", monospace;"> (</span><b><span style="font-family: "Courier New", monospace;">AMDGPU_INFO_MMR_SH_INDEX_MASK</span></b><span style="font-family: "Courier New", monospace;"> <<</span></div> <div><span style="font-family: "Courier New", monospace;"> AMDGPU_INFO_MMR_SH_INDEX_SHIFT);</span></div> <div><br> </div> <div><span style="font-family: "Courier New", monospace;"> r = amdgpu_read_mm_registers(dev, 0x263d, 1, instance, 0,</span></div> <div><span style="font-family: "Courier New", monospace;"> &dev->info.backend_disable[i]);</span></div> <div><br> </div> <div>This effectively reads from 0/* where the kernel adds the instance of * so it's 0/*/*. That line was last changed by Alex</div> <div><br> </div> <div> <div><span style="font-family: "Courier New", monospace; font-size: 10pt;"><b>0936139536380</b> (Alex Deucher 2015-04-20 12:04:22 -0400 174) (AMDGPU_INFO_MMR_SH_INDEX_MASK <<</span></div> </div> <div><br> </div> <div>I still don't get why this is a reason to hit pause on the patch(es) though. They're root only, you can still not use * if you don't want to and furthermore I'm already using the write debugfs content (to debug waveforms). </div> <div><br> </div> <div>Tom</div> <p></p> <br> <br> <div style="color: rgb(0, 0, 0);"> <div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Michel Dänzer <michel.daenzer@mailbox.org><br> <b>Sent:</b> Tuesday, October 11, 2016 20:34<br> <b>To:</b> StDenis, Tom; Christian König<br> <b>Cc:</b> amd-gfx@lists.freedesktop.org<br> <b>Subject:</b> Re: [PATCH] drm/amd/amdgpu: Allow broadcast on debugfs read (v2)</font> <div> </div> </div> </div> <font size="2"><span style="font-size:10pt;"> <div class="PlainText">On 11/10/16 09:32 PM, StDenis, Tom wrote:<br> > It's used by the UMD though they read from 0/*/* when reading the<br> > RASTER_CONFIG registers (which may be a bug...)<br> <br> We should probably clarify what userspace is trying to do there, and<br> whether the hardware actually does that.<br> <br> <br> -- <br> Earthling Michel Dänzer | <a href="http://www.amd.com" id="LPlnk115396" previewremoved="true"> http://www.amd.com</a> <div id="LPBorder_GT_14762864460090.3072699807620518" style="margin-bottom: 20px; overflow: auto; width: 100%; text-indent: 0px;"> <table id="LPContainer_14762864460060.8831490190015878" cellspacing="0" style="width: 90%; background-color: rgb(255, 255, 255); position: relative; overflow: auto; padding-top: 20px; padding-bottom: 20px; margin-top: 20px; border-top: 1px dotted rgb(200, 200, 200); border-bottom: 1px dotted rgb(200, 200, 200);"> <tbody> <tr valign="top" style="border-spacing: 0px;"> <td id="TextCell_14762864460070.24216894343271722" colspan="2" style="vertical-align: top; position: relative; padding: 0px; display: table-cell;"> <div id="LPRemovePreviewContainer_14762864460070.8888817369809654"></div> <div id="LPTitle_14762864460070.3011910420489814" style="top: 0px; color: rgb(59, 87, 119); font-weight: normal; font-size: 21px; font-family: wf_segoe-ui_light, "Segoe UI Light", "Segoe WP Light", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; line-height: 21px;"> <a id="LPUrlAnchor_14762864460080.3324236906153566" href="http://www.amd.com/" target="_blank" style="text-decoration: none;">Graphics, Processors and Immersive VR Solutions | AMD </a></div> <div id="LPMetadata_14762864460080.08593046515590075" style="margin: 10px 0px 16px; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; font-size: 14px; line-height: 14px;"> www.amd.com</div> <div id="LPDescription_14762864460080.9804859375779311" style="display: block; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; font-size: 14px; line-height: 20px; max-height: 100px; overflow: hidden;"> Explore a wide range of innovative next generation computing processors, graphics, and Immersive VR solutions by Advanced Micro Devices (AMD). Visit AMD.com now!</div> </td> </tr> </tbody> </table> </div> <br> <br> Libre software enthusiast | Mesa and X developer<br> </div> </span></font></div> </div> </body> </html>