使用Nsight查找CE3的渲染bug

工作临时的接的一个小任务，查找ce3引擎修改后在绘制上出的一点bug

在代码的底层调用代码做了一些修改后，场景里的绘制的问题，因为也是刚接触CE3代码，也只能通过Nsight来查找问题了。

 

首先用Nsight对正确和错误两个版本做次对比

下图是正确的版本

这个是错误版本

对比后，先是定位出问题绘制事件

 

定位到应该是FOG绘制的问题

 

再看一下当前API设置的Pixel Shader的Source code

pixout FogPassPS(vert2fragFog IN)
{
    pixout OUT;
    float sceneDepth;
    half4 localFogColor;
    float3 worldPos,cameraToWorldPos;
    FogPassCommon(IN,sceneDepth,localFogColor,worldPos,cameraToWorldPos);
    localFogColor.a=1.0-localFogColor.a;
    localFogColor.xyz*=HDRParams2.y;
    HDROutput(OUT,localFogColor,);
    return OUT;
}

从shader的名字看，确实是FogPass的问题，先把代码回滚回正确的版本后，对两个版本fogpass的shader代码做比对，没有任何区别，排除了shader source code 出错的可能性。

然后再对两个版本的IS, VS, XFORM/RS, PS, OT逐一比较

正确版本的Output

错误版本的Output

发现了SrcBlendAlpha的设置错误，绘制fog时的AlphaBelnd设置也出了问题，另外Stencil的输出也不对，不过因为这个draw event的时候，SeperateAlphaBlendEnable和StencilEnable都是FALSE，所以画面绘制问题并不是这个造成的，不过还是需要记录下来以后进行修改。

然后，这时候发现绘制fog时，Ps时时使用的Texture Sample，两个版本也有区别

一个是SceneDiffuseAcc，也就是deferred light的漫反射累积光照的问题

另外就是在，ZPass阶段的ZMap的深度输出信息，ZTarget也不一样，因为ZTarget的输出在更前面，那么就先解决它的问题

 

正确版本的ZTraget

错误版本的ZTarget

从上图可以看到SceneNormalsMap和ZTarget的信息是不一样的，而且错误版本好像是这两个rt输出的信息互换了，SceneNormalsMap的RT格式是A8R8G8B8,

而ZTarget的RT格式是R32F。

 

为验证这个猜测，跟踪到计算ZTarget的Draw Event附近，对比一下上下文的RT的输出差别

正确版本

 

错误版本

可以确定了，最后gbuffer的RT_0输出的到的目标surface错误的，本应该是A8R8G8B8格式的SceneNormalsMap，变成了R32F格式的ZTarget。

定位到错误后，就是到代码里去查找BUG了=v=。

 

一开始的想法比较简单，认为应该就是renderTarget的目标被错误设置的问题，那么，先根据Perf Marker信息，定位到处理ZPASS阶段的函数

1. PROFILE_LABEL_PUSH("ZPASS");
   FX_ProcessZPassRenderLists();
   PROFILE_LABEL_POP("ZPASS");

   CD3D9Renderer::FX_ProcessZPassRenderLists，而它通过调用CD3D9Renderer::FX_ZScene，来设置rendertarget

1. if( bRenderNormalsOnly )
   {
       FX_PushRenderTarget(,CTexture::s_ptexSceneNormalsMap,&m_DepthBufferOrigMSAA,false,-,true);
   }
   else
   {
       FX_PushRenderTarget(,CTexture::s_ptexZTarget,&m_DepthBufferOrigMSAA,false,-,true);
       FX_PushRenderTarget(,CTexture::s_ptexSceneNormalsMap, NULL);
   }

   这里FX_PushRenderTarget并不会立即调用d3d的api设置rendertarget，而是把信息保存在RT的stack里，等commit时再统一进行设置，条件语句if成功的分支，就是把ZTarget设置到RT0的阶段，但从

但通过runtime时debug代码来看，这部分的指针传递都是没问题的。

上层API没有问题，那么就只能从底层d3ddevice的SetRenderTarget来查找问题了

1. if(NewRenderTargets== NULL ||!RTs[i]->GetRenderTargetView())
   {
       mNativeD3D9Device->SetRenderTarget(i, NULL);
   }
   else
   {
       IDirect3DSurface9* pSurf =(IDirect3DSurface9*)RTs[i]->GetRenderTargetView()->GetNativeResource();
       mNativeD3D9Device->SetRenderTarget( i, pSurf );
   }

   上面是同事修改过后的SetRednerTarget函数，在这里设置断点，结果发现传递进去的pSurf指针也是正确的，那么说明，这部分API的调用是没问题的，很有可能是SetRenderTarget失败，导致前一个pass里，设置的RT0 surface，也就是R32F ZTarget被保留到了下一个pass里继续使用了。

重新单步调试，遍历这部分流程，发现调用CD3D9Renderer::FX_PopRenderTarget()函数的时候，也就是把当前的RT保存到texutre的操作，也会有SetRednerTarget的操作。恰好也就在前面切换RT0的绑定surface之间调用的。也就是说，如果我们已经把SceneNormalsMap设置到RT1后，如果不把RT1清空，就立即把它设置到RT0,那么这部操作是不会成功的。

 

也就是

1. D3D9Device->SetRenderTarget(,ZTarget);
   D3D9Device->SetRenderTarget(,SceneNormal);
   
   D3D9Device->SetRenderTarget(,SceneNormal);//错误

   D3D9Device->SetRenderTarget(,ZTarget);
   D3D9Device->SetRenderTarget(,SceneNormal);
   
   D3D9Device->SetRenderTarget(,);
   D3D9Device->SetRenderTarget(,SceneNormal);//正确

   而我们修改过的代码，因为某些逻辑问题，设置RT1为NULL的操作被跳过了

打开Nsight的Event list

可以看到正确版本的Eevnt99部分有设置RT1为NULL的操作，而如果跳过这部操作的话，Event135的的设置会返回错误代码。也就是之前的ZTarget会被继续设置在RT0上。

同事修改了代码后，这部分总算是正确了=0=。

在查找bug过程中中，还发现了绘制纹理的亮度有差别

正确版本

错误版本

排查后在读取贴图的地方发现，是SRGBTexture设置的问题。

CE老版本的设置方法，而DX11风格变成了统一设置，而我们把这个参数忘记了。

1. LPDIRECT3DDEVICE9 dv = gcpRendD3D->GetD3DDevice();
   bool sRGBRead = gRenDev->IsLinearSpaceShadingEnabled()&&( m_bIsSRGB || s_TexStates[nTS].m_bSRGBLookup &&(m_nFlags& FT_USAGE_ALLOWREADSRGB));
   STexState*pDTS =&TexStages[nTUnit].m_State;
   if(pDTS->m_bSRGBLookup != sRGBRead )
   {
       pDTS->m_bSRGBLookup = sRGBRead;
       dv->SetSamplerState(nTUnit + nVtxTexOffSet, D3DSAMP_SRGBTEXTURE, sRGBRead ? TRUE : FALSE);
   }

   然后，又回到了SceneDiffuseAcc渲染错误的问题上了。这个问题略坑，还没找到正确解决方法，
   具体现象是，在正确版本里，把法线，UV，以及其他定值作为output输出

Normal

UV.x

可以看到上面部分都被clip掉了，而查看了所以可能的状态，设置都为false

而我们的新版本，不设置这些状态，上面部分也要绘制，所以才会出现ScenediffuseAcc错误信息

错误的版本，红框部分因为深度过大，本来是不应该绘制的。

解决办法，只有暂时在shader里添加一个clip函数来把深度过大的地方去掉。

 

11月7日添加: 最后发现是同事把nv的depthboundTest实现的代码注释掉了，也难怪nsight和pix都差不到= =