Enabling Hardware Acceleration on RDP

Author: Kuldeep-MS

src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/Media/MediaSystem.cs

@@ -79,6 +79,17 @@ public static bool Startup(MediaContext mc)
                 }
                 s_refCount++;
             }
+
+             /// <summary>
+            /// Set renderOption if Hardware acceleration is enabled for RDP
+            // </summary>
+
+            bool isHardwareAccelerationEnabledForRDP = false;
+            AppContext.TryGetSwitch("System.Windows.Media.IsHardwareAccelerationEnabledForRDP", out isHardwareAccelerationEnabledForRDP);
+            if(isHardwareAccelerationEnabledForRDP)
+            {
+                UnsafeNativeMethods.RenderOptions_EnableGraphicHwAccelerationForRDP(isHardwareAccelerationEnabledForRDP);
+            }
             // Consider making MediaSystem.ConnectTransport return the state of transport connectedness so
             // that we can initialize the media system to a disconnected state.
 

src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/Media/UnsafeNativeMethodsMilCoreApi.cs

@@ -208,7 +208,10 @@ internal unsafe static extern void RenderOptions_ForceSoftwareRenderingModeForPr
                 bool fForce);
 
             [DllImport(DllImport.MilCore, EntryPoint = "RenderOptions_IsSoftwareRenderingForcedForProcess")]
-            internal unsafe static extern bool RenderOptions_IsSoftwareRenderingForcedForProcess();            
+            internal unsafe static extern bool RenderOptions_IsSoftwareRenderingForcedForProcess();
+
+            [DllImport(DllImport.MilCore, EntryPoint = "RenderOptions_EnableGraphicHwAccelerationForRDP")]
+            internal unsafe static extern bool RenderOptions_EnableGraphicHwAccelerationForRDP(bool value);                 
 
             [DllImport(DllImport.MilCore, EntryPoint = "MilResource_CreateCWICWrapperBitmap")]
             internal unsafe static extern int /* HRESULT */ CreateCWICWrapperBitmap(

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/common/d3dutils.cpp

@@ -232,10 +232,19 @@ GetMinimalTextureDesc(
         }
 
         // If format is ok, restore HR from width/height check
-        if (hr == S_OK)
+        if (RenderOptions::IsGraphicHWAccelerationForRDPEnabled())
         {
-            hr = hrWH;
+            if(SUCCEEDED(hr))
+            {
+                hr = hrWH;
+            }
+        } else {
+            if (hr == S_OK)
+            {
+                hr = hrWH;
+            }
         }
+        
     }
 
 #if DBG
@@ -315,8 +324,3 @@ GetSuperiorSurfaceFormat(
 
     return D3DFMT_UNKNOWN;
 }
-
-
-
-
-

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/common/display.cpp

@@ -2125,7 +2125,7 @@ CDisplaySet::GetGraphicsAccelerationCaps(
         *pulDisplayUniqueness = m_ulDisplayUniquenessLoader;
     }
 
-    if (m_rgpDisplays.GetCount() == 0 || m_fNonLocalDevicePresent)
+    if (m_rgpDisplays.GetCount() == 0 || (!RenderOptions::IsGraphicHWAccelerationForRDPEnabled() && m_fNonLocalDevicePresent))
     {
         //
         // No display - no acceleration

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/common/renderoptions.cpp

@@ -23,10 +23,17 @@ RenderOptions_IsSoftwareRenderingForcedForProcess()
     return RenderOptions::IsSoftwareRenderingForcedForProcess();
 }
 
+void WINAPI
+RenderOptions_EnableGraphicHwAccelerationForRDP(BOOL value)
+{
+    RenderOptions::EnableGraphicHwAccelerationForRDP(value);
+}
+
 // m_cs must be entered before accessing m_fForceSoftware because multiple
 // managed threads plus the render thread could try to access it
 static CCriticalSection m_cs;
 static bool m_fForceSoftware;
+static bool m_fHwAccelerationForRDPEnabled;
 
 //+---------------------------------------------------------------------------------
 //
@@ -41,6 +48,7 @@ HRESULT
 RenderOptions::Init()
 {
     m_fForceSoftware = false;
+    m_fHwAccelerationForRDPEnabled = false;
     RRETURN(m_cs.Init());
 }
 
@@ -94,4 +102,30 @@ RenderOptions::IsSoftwareRenderingForcedForProcess()
     return m_fForceSoftware;
 }
 
+//+---------------------------------------------------------------------------------
+//
+//  RenderOptions::EnableGraphicHwAccelerationForRDP
+//
+//  Synopsis:   Sets whether or not Hardware Acceleration should be enabled for RDP.
+//
+//----------------------------------------------------------------------------------
+void
+RenderOptions::EnableGraphicHwAccelerationForRDP(BOOL value)
+{
+    CGuard<CCriticalSection> guard(m_cs);
+    m_fHwAccelerationForRDPEnabled = !!value;
+}
+
+//+---------------------------------------------------------------------------------
+//
+//  RenderOptions::IsGraphicHWAccelerationForRDPEnabled
+//
+//  Synopsis:   return whether or not Hardware Acceleration for RDP is enabled.
+//
+//----------------------------------------------------------------------------------
+BOOL
+RenderOptions::IsGraphicHWAccelerationForRDPEnabled()
+{
+    return m_fHwAccelerationForRDPEnabled;
+}
 

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/common/renderoptions.h

@@ -18,6 +18,10 @@ namespace RenderOptions
     void ForceSoftwareRenderingForProcess(BOOL fForce);
 
     BOOL IsSoftwareRenderingForcedForProcess();
+
+    void EnableGraphicHwAccelerationForRDP(BOOL value);
+
+    BOOL IsGraphicHWAccelerationForRDPEnabled();
 };
 
 

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/dll/wpfgfx.def

@@ -137,4 +137,5 @@ EXPORTS
 
     RenderOptions_ForceSoftwareRenderingModeForProcess
     RenderOptions_IsSoftwareRenderingForcedForProcess
+    RenderOptions_EnableGraphicHwAccelerationForRDP
 

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/hw/hwbitmapcolorsource.cpp

@@ -1421,7 +1421,20 @@ CHwBitmapColorSource::ComputeRealizationParameters(
             &uLevels
             );
 
-        Assert(pDevice->GetMinimalTextureDesc(
+        if(RenderOptions::IsGraphicHWAccelerationForRDPEnabled())
+        {
+            Assert(SUCCEEDED(pDevice->GetMinimalTextureDesc(
+            &d3dsdRequired,
+            TRUE,
+            (GMTD_CHECK_ALL |
+             (TextureAddressingAllowsConditionalNonPower2Usage(
+                 oRealizationParams.dlU.d3dta,
+                 oRealizationParams.dlV.d3dta) ?
+              GMTD_NONPOW2CONDITIONAL_OK : 0)
+            )
+            )));
+        } else {
+            Assert(pDevice->GetMinimalTextureDesc(
             &d3dsdRequired,
             TRUE,
             (GMTD_CHECK_ALL |
@@ -1431,6 +1444,8 @@ CHwBitmapColorSource::ComputeRealizationParameters(
               GMTD_NONPOW2CONDITIONAL_OK : 0)
             )
             ) == S_OK);
+        }
+        
     }
     #endif
 

src/Microsoft.DotNet.Wpf/src/WpfGfx/core/meta/desktoprt.cpp

@@ -93,7 +93,7 @@ HRESULT CDesktopRenderTarget::Create(
     // check whether any adapters don't support Hw acceleration or D3D is not
     // available.
     //
-    if (   pDisplaySet->IsNonLocalDisplayPresent()
+    if (   (!(RenderOptions::IsGraphicHWAccelerationForRDPEnabled()) && pDisplaySet->IsNonLocalDisplayPresent())
         || !pDisplaySet->D3DObject())
     {
         // If possible, just revert to Sw.  This simply prevents trying Hw and