From cee893be99b5cd159a580a5dec9eef1ccef93dc5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Herm=C3=A8s=20B=C3=A9lusca-Ma=C3=AFto?= Date: Mon, 9 Jan 2023 16:08:07 +0100 Subject: [PATCH] [NTOS:KD] Simplify min-values calculations in KdpPrintToLogFile and KdpScreenPrint. --- ntoskrnl/kd/kdio.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/ntoskrnl/kd/kdio.c b/ntoskrnl/kd/kdio.c index 45733e0b3ca..5f2c7f966c7 100644 --- a/ntoskrnl/kd/kdio.c +++ b/ntoskrnl/kd/kdio.c @@ -174,10 +174,7 @@ KdpPrintToLogFile(PCHAR String, OldIrql = KdbpAcquireLock(&KdpDebugLogSpinLock); beg = KdpCurrentPosition; - num = KdpFreeBytes; - if (Length < num) - num = Length; - + num = min(Length, KdpFreeBytes); if (num != 0) { end = (beg + num) % KdpBufferSize; @@ -470,16 +467,14 @@ KdpScreenPrint(PCHAR String, return; if (KdpDmesgBuffer == NULL) - return; + return; /* Acquire the printing spinlock without waiting at raised IRQL */ OldIrql = KdbpAcquireLock(&KdpDmesgLogSpinLock); - /* Invariant: always_true(KdpDmesgFreeBytes == KdpDmesgBufferSize); - * set num to min(KdpDmesgFreeBytes, Length). - */ - num = (Length < KdpDmesgFreeBytes) ? Length : KdpDmesgFreeBytes; beg = KdpDmesgCurrentPosition; + /* Invariant: always_true(KdpDmesgFreeBytes == KdpDmesgBufferSize); */ + num = min(Length, KdpDmesgFreeBytes); if (num != 0) { end = (beg + num) % KdpDmesgBufferSize;