[PATCH] include/ddk: Update MEMORY_CACHING_TYPE*

[Serge Gautherie]

Signed-off-by: Serge Gautherie 
---
 include/ddk/wdm.h | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/include/ddk/wdm.h b/include/ddk/wdm.h
index 447833c..2b988b0 100644
--- a/include/ddk/wdm.h
+++ b/include/ddk/wdm.h
@@ -1314,16 +1314,22 @@ typedef struct _KUSER_SHARED_DATA {
 #define SHARED_GLOBAL_FLAGS_QPC_BYPASS_A73_ERRATA 0x40
 #define SHARED_GLOBAL_FLAGS_QPC_BYPASS_USE_RDTSCP 0x80

+typedef enum _MEMORY_CACHING_TYPE_ORIG {
+    MmFrameBufferCached = 2
+} MEMORY_CACHING_TYPE_ORIG;
+
 typedef enum _MEMORY_CACHING_TYPE {
-    MmNonCached = 0,
-    MmCached = 1,
-    MmWriteCombined = 2,
-    MmHardwareCoherentCached = 3,
-    MmNonCachedUnordered = 4,
-    MmUSWCCached = 5,
-    MmMaximumCacheType = 6
+    MmNonCached = FALSE,
+    MmCached = TRUE,
+    MmWriteCombined = MmFrameBufferCached,
+    MmHardwareCoherentCached,
+    MmNonCachedUnordered,
+    MmUSWCCached,
+    MmMaximumCacheType,
+    MmNotMapped = -1
 } MEMORY_CACHING_TYPE;

+
 typedef enum _MM_PAGE_PRIORITY {
     LowPagePriority,
     NormalPagePriority = 16,

[Zebediah Figura (she/her)]

On 5/20/21 7:57 PM, Serge Gautherie wrote:
> Signed-off-by: Serge Gautherie
>
> include/ddk/wdm.h | 20 +++++++++++++-------
>   1 file changed, 13 insertions(+), 7 deletions(-)
>
> diff --git a/include/ddk/wdm.h b/include/ddk/wdm.h
> index 447833c..2b988b0 100644
> --- a/include/ddk/wdm.h
> +++ b/include/ddk/wdm.h
> @@ -1314,16 +1314,22 @@ typedef struct _KUSER_SHARED_DATA {
>   #define SHARED_GLOBAL_FLAGS_QPC_BYPASS_A73_ERRATA 0x40
>   #define SHARED_GLOBAL_FLAGS_QPC_BYPASS_USE_RDTSCP 0x80
>
> +typedef enum _MEMORY_CACHING_TYPE_ORIG {
> - MmFrameBufferCached = 2
>
> +} MEMORY_CACHING_TYPE_ORIG;
> - typedef enum _MEMORY_CACHING_TYPE {
> - MmNonCached = 0,
> - MmCached = 1,
> - MmWriteCombined = 2,
> - MmHardwareCoherentCached = 3,
> - MmNonCachedUnordered = 4,
> - MmUSWCCached = 5,
> - MmMaximumCacheType = 6
> - MmNonCached = FALSE,
> - MmCached = TRUE,
> - MmWriteCombined = MmFrameBufferCached,
> - MmHardwareCoherentCached,
> - MmNonCachedUnordered,
> - MmUSWCCached,
> - MmMaximumCacheType,
> - MmNotMapped = -1
> } MEMORY_CACHING_TYPE;

Setting aside that this is now almost identical to the Microsoft 
headers, is this really an improvement?
> - typedef enum _MM_PAGE_PRIORITY {
>     LowPagePriority,
>     NormalPagePriority = 16,