Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-06-05 17:30:32 +00:00
Code Issues Projects Releases Packages Wiki Activity

[SETUPLIB] Silence noisy DPRINTs.

Browse source
This commit is contained in:
Hermès Bélusca-Maïto 2018-10-28 23:33:29 +01:00
parent 3d68c01664
commit 9016b5f3ef
No known key found for this signature in database
GPG key ID: 3B2539C65E7B93D0
3 changed files with 70 additions and 66 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
base/setup/lib/bootsup.c
View file

@ -356,9 +356,9 @@ EnumerateReactOSEntries(
RTL_FIELD_SIZE(NTOS_OPTIONS, Signature)) RTL_FIELD_SIZE(NTOS_OPTIONS, Signature))
{ {
/* This is not a ReactOS entry */ /* This is not a ReactOS entry */
// DPRINT1(" An installation '%S' of unsupported type '%S'\n", // DPRINT(" An installation '%S' of unsupported type '%S'\n",
// BootEntry->FriendlyName, BootEntry->Version ? BootEntry->Version : L"n/a"); // BootEntry->FriendlyName, BootEntry->Version ? BootEntry->Version : L"n/a");
DPRINT1(" An installation '%S' of unsupported type %lu\n", DPRINT(" An installation '%S' of unsupported type %lu\n",
BootEntry->FriendlyName, BootEntry->OsOptionsLength); BootEntry->FriendlyName, BootEntry->OsOptionsLength);
/* Continue the enumeration */ /* Continue the enumeration */
goto SkipThisEntry; goto SkipThisEntry;
@ -388,12 +388,12 @@ EnumerateReactOSEntries(
} }
} }
DPRINT1(" Found a candidate Win2k3 install '%S' with ARC path '%S'\n", DPRINT(" Found a candidate Win2k3 install '%S' with ARC path '%S'\n",
BootEntry->FriendlyName, Options->OsLoadPath); BootEntry->FriendlyName, Options->OsLoadPath);
// DPRINT1(" Found a Win2k3 install '%S' with ARC path '%S'\n", // DPRINT(" Found a Win2k3 install '%S' with ARC path '%S'\n",
// BootEntry->FriendlyName, Options->OsLoadPath); // BootEntry->FriendlyName, Options->OsLoadPath);
DPRINT1("EnumerateReactOSEntries: OsLoadPath: '%S'\n", Options->OsLoadPath); DPRINT("EnumerateReactOSEntries: OsLoadPath: '%S'\n", Options->OsLoadPath);
Data->UseExistingEntry = TRUE; Data->UseExistingEntry = TRUE;
RtlStringCchCopyW(Data->OsName, ARRAYSIZE(Data->OsName), BootEntry->FriendlyName); RtlStringCchCopyW(Data->OsName, ARRAYSIZE(Data->OsName), BootEntry->FriendlyName);

12
base/setup/lib/utils/bldrsup.c
View file

@ -156,18 +156,18 @@ FindBootStore( // By handle
if (DoesFileExist(PartitionDirectoryHandle, LoaderExecutable)) if (DoesFileExist(PartitionDirectoryHandle, LoaderExecutable))
{ {
/* A loader was found, stop there */ /* A loader was found, stop there */
DPRINT1("Found loader executable '%S'\n", LoaderExecutable); DPRINT("Found loader executable '%S'\n", LoaderExecutable);
break; break;
} }
/* The loader does not exist, continue with another one */ /* The loader does not exist, continue with another one */
DPRINT1("Loader executable '%S' does not exist, continue with another one...\n", LoaderExecutable); DPRINT("Loader executable '%S' does not exist, continue with another one...\n", LoaderExecutable);
LoaderExecutable += wcslen(LoaderExecutable) + 1; LoaderExecutable += wcslen(LoaderExecutable) + 1;
} }
if (!*LoaderExecutable) if (!*LoaderExecutable)
{ {
/* No loader was found */ /* No loader was found */
DPRINT1("No loader executable was found\n"); DPRINT("No loader executable was found\n");
return STATUS_NOT_FOUND; return STATUS_NOT_FOUND;
} }
@ -1308,7 +1308,7 @@ FreeLdrEnumerateBootEntries(
InstallName = Buffer; InstallName = Buffer;
} }
DPRINT1("Boot entry '%S' in OS section '%S'\n", InstallName, SectionName); DPRINT("Boot entry '%S' in OS section '%S'\n", InstallName, SectionName);
BootEntry->Version = FreeLdr; BootEntry->Version = FreeLdr;
BootEntry->BootEntryKey = MAKESTRKEY(SectionName); BootEntry->BootEntryKey = MAKESTRKEY(SectionName);
@ -1337,7 +1337,7 @@ FreeLdrEnumerateBootEntries(
/* BootType is Windows2003 */ /* BootType is Windows2003 */
PNTOS_OPTIONS Options = (PNTOS_OPTIONS)&BootEntry->OsOptions; PNTOS_OPTIONS Options = (PNTOS_OPTIONS)&BootEntry->OsOptions;
DPRINT1("This is a '%S' boot entry\n", KeyData); DPRINT("This is a '%S' boot entry\n", KeyData);
BootEntry->OsOptionsLength = sizeof(NTOS_OPTIONS); BootEntry->OsOptionsLength = sizeof(NTOS_OPTIONS);
RtlCopyMemory(Options->Signature, RtlCopyMemory(Options->Signature,
@ -1368,7 +1368,7 @@ FreeLdrEnumerateBootEntries(
/* BootType is BootSector */ /* BootType is BootSector */
PBOOT_SECTOR_OPTIONS Options = (PBOOT_SECTOR_OPTIONS)&BootEntry->OsOptions; PBOOT_SECTOR_OPTIONS Options = (PBOOT_SECTOR_OPTIONS)&BootEntry->OsOptions;
DPRINT1("This is a '%S' boot entry\n", KeyData); DPRINT("This is a '%S' boot entry\n", KeyData);
BootEntry->OsOptionsLength = sizeof(BOOT_SECTOR_OPTIONS); BootEntry->OsOptionsLength = sizeof(BOOT_SECTOR_OPTIONS);
RtlCopyMemory(Options->Signature, RtlCopyMemory(Options->Signature,

48
base/setup/lib/utils/osdetect.c
View file

@ -96,9 +96,9 @@ EnumerateInstallations(
RTL_FIELD_SIZE(NTOS_OPTIONS, Signature)) RTL_FIELD_SIZE(NTOS_OPTIONS, Signature))
{ {
/* This is not a ReactOS entry */ /* This is not a ReactOS entry */
// DPRINT1(" An installation '%S' of unsupported type '%S'\n", // DPRINT(" An installation '%S' of unsupported type '%S'\n",
// BootEntry->FriendlyName, BootEntry->Version ? BootEntry->Version : L"n/a"); // BootEntry->FriendlyName, BootEntry->Version ? BootEntry->Version : L"n/a");
DPRINT1(" An installation '%S' of unsupported type %lu\n", DPRINT(" An installation '%S' of unsupported type %lu\n",
BootEntry->FriendlyName, BootEntry->OsOptionsLength); BootEntry->FriendlyName, BootEntry->OsOptionsLength);
/* Continue the enumeration */ /* Continue the enumeration */
return STATUS_SUCCESS; return STATUS_SUCCESS;
@ -113,9 +113,9 @@ EnumerateInstallations(
return STATUS_SUCCESS; return STATUS_SUCCESS;
} }
DPRINT1(" Found a candidate Win2k3 install '%S' with ARC path '%S'\n", DPRINT(" Found a candidate Win2k3 install '%S' with ARC path '%S'\n",
BootEntry->FriendlyName, Options->OsLoadPath); BootEntry->FriendlyName, Options->OsLoadPath);
// DPRINT1(" Found a Win2k3 install '%S' with ARC path '%S'\n", // DPRINT(" Found a Win2k3 install '%S' with ARC path '%S'\n",
// BootEntry->FriendlyName, Options->OsLoadPath); // BootEntry->FriendlyName, Options->OsLoadPath);
// TODO: Normalize the ARC path. // TODO: Normalize the ARC path.
@ -127,7 +127,7 @@ EnumerateInstallations(
NtOsInstall = FindExistingNTOSInstall(Data->List, Options->OsLoadPath, NULL); NtOsInstall = FindExistingNTOSInstall(Data->List, Options->OsLoadPath, NULL);
if (NtOsInstall) if (NtOsInstall)
{ {
DPRINT1(" An NTOS installation with name \"%S\" from vendor \"%S\" already exists in SystemRoot '%wZ'\n", DPRINT(" An NTOS installation with name \"%S\" from vendor \"%S\" already exists in SystemRoot '%wZ'\n",
NtOsInstall->InstallationName, NtOsInstall->VendorName, &NtOsInstall->SystemArcPath); NtOsInstall->InstallationName, NtOsInstall->VendorName, &NtOsInstall->SystemArcPath);
/* Continue the enumeration */ /* Continue the enumeration */
return STATUS_SUCCESS; return STATUS_SUCCESS;
@ -146,7 +146,7 @@ EnumerateInstallations(
return STATUS_SUCCESS; return STATUS_SUCCESS;
} }
DPRINT1("ArcPathToNtPath() succeeded: '%S' --> '%wZ'\n", DPRINT("ArcPathToNtPath() succeeded: '%S' --> '%wZ'\n",
Options->OsLoadPath, &SystemRootPath); Options->OsLoadPath, &SystemRootPath);
/* /*
@ -162,7 +162,7 @@ EnumerateInstallations(
return STATUS_SUCCESS; return STATUS_SUCCESS;
} }
DPRINT1("EnumerateInstallations: SystemRootPath: '%wZ'\n", &SystemRootPath); DPRINT("EnumerateInstallations: SystemRootPath: '%wZ'\n", &SystemRootPath);
/* Check if this is a valid NTOS installation; stop there if it isn't one */ /* Check if this is a valid NTOS installation; stop there if it isn't one */
RtlInitEmptyUnicodeString(&VendorName, VendorNameBuffer, sizeof(VendorNameBuffer)); RtlInitEmptyUnicodeString(&VendorName, VendorNameBuffer, sizeof(VendorNameBuffer));
@ -172,13 +172,13 @@ EnumerateInstallations(
return STATUS_SUCCESS; return STATUS_SUCCESS;
} }
DPRINT1("Found a valid NTOS installation in SystemRoot ARC path '%S', NT path '%wZ'\n", DPRINT("Found a valid NTOS installation in SystemRoot ARC path '%S', NT path '%wZ'\n",
Options->OsLoadPath, &SystemRootPath); Options->OsLoadPath, &SystemRootPath);
/* From the NT path, compute the disk, partition and path components */ /* From the NT path, compute the disk, partition and path components */
if (NtPathToDiskPartComponents(SystemRootPath.Buffer, &DiskNumber, &PartitionNumber, &PathComponent)) if (NtPathToDiskPartComponents(SystemRootPath.Buffer, &DiskNumber, &PartitionNumber, &PathComponent))
{ {
DPRINT1("SystemRootPath = '%wZ' points to disk #%d, partition #%d, path '%S'\n", DPRINT("SystemRootPath = '%wZ' points to disk #%d, partition #%d, path '%S'\n",
&SystemRootPath, DiskNumber, PartitionNumber, PathComponent); &SystemRootPath, DiskNumber, PartitionNumber, PathComponent);
/* Retrieve the corresponding disk and partition */ /* Retrieve the corresponding disk and partition */
@ -307,7 +307,7 @@ CheckForValidPEAndVendor(
if (NT_SUCCESS(Status) /*&& pvData*/) if (NT_SUCCESS(Status) /*&& pvData*/)
{ {
/* BufLen includes the NULL terminator count */ /* BufLen includes the NULL terminator count */
DPRINT1("Found version vendor: \"%S\" for file '%S'\n", pvData, PathNameToFile); DPRINT("Found version vendor: \"%S\" for file '%S'\n", pvData, PathNameToFile);
RtlStringCbCopyNW(VendorName->Buffer, VendorName->MaximumLength, RtlStringCbCopyNW(VendorName->Buffer, VendorName->MaximumLength,
pvData, BufLen * sizeof(WCHAR)); pvData, BufLen * sizeof(WCHAR));
@ -318,7 +318,7 @@ CheckForValidPEAndVendor(
} }
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
DPRINT1("No version vendor found for file '%S'\n", PathNameToFile); DPRINT("No version vendor found for file '%S'\n", PathNameToFile);
UnmapCloseFile: UnmapCloseFile:
/* Finally, unmap and close the file */ /* Finally, unmap and close the file */
@ -417,7 +417,7 @@ IsValidNTOSInstallationByHandle(
if (Success) if (Success)
{ {
/* We have found a correct vendor combination */ /* We have found a correct vendor combination */
DPRINT1("IsValidNTOSInstallation: We've got an NTOS installation from %S !\n", KnownVendors[i]); DPRINT("IsValidNTOSInstallation: We've got an NTOS installation from %S !\n", KnownVendors[i]);
break; break;
} }
} }
@ -445,7 +445,7 @@ IsValidNTOSInstallationByHandle(
if (!!FindSubStrI(LocalVendorName.Buffer, KnownVendors[i])) if (!!FindSubStrI(LocalVendorName.Buffer, KnownVendors[i]))
{ {
/* We have found a correct vendor combination */ /* We have found a correct vendor combination */
DPRINT1("IsValidNTOSInstallation: The user-mode DLL '%S' is from %S\n", PathName, KnownVendors[i]); DPRINT("IsValidNTOSInstallation: The user-mode DLL '%S' is from %S\n", PathName, KnownVendors[i]);
break; break;
} }
} }
@ -498,6 +498,7 @@ IsValidNTOSInstallation(
return Success; return Success;
} }
#ifndef NDEBUG
static VOID static VOID
DumpNTOSInstalls( DumpNTOSInstalls(
IN PGENERIC_LIST List) IN PGENERIC_LIST List)
@ -506,7 +507,7 @@ DumpNTOSInstalls(
PNTOS_INSTALLATION NtOsInstall; PNTOS_INSTALLATION NtOsInstall;
ULONG NtOsInstallsCount = GetNumberOfListEntries(List); ULONG NtOsInstallsCount = GetNumberOfListEntries(List);
DPRINT1("There %s %d installation%s detected:\n", DPRINT("There %s %d installation%s detected:\n",
NtOsInstallsCount >= 2 ? "are" : "is", NtOsInstallsCount >= 2 ? "are" : "is",
NtOsInstallsCount, NtOsInstallsCount,
NtOsInstallsCount >= 2 ? "s" : ""); NtOsInstallsCount >= 2 ? "s" : "");
@ -515,13 +516,14 @@ DumpNTOSInstalls(
{ {
NtOsInstall = (PNTOS_INSTALLATION)GetListEntryData(Entry); NtOsInstall = (PNTOS_INSTALLATION)GetListEntryData(Entry);
DPRINT1(" On disk #%d, partition #%d: Installation \"%S\" in SystemRoot '%wZ'\n", DPRINT(" On disk #%d, partition #%d: Installation \"%S\" in SystemRoot '%wZ'\n",
NtOsInstall->DiskNumber, NtOsInstall->PartitionNumber, NtOsInstall->DiskNumber, NtOsInstall->PartitionNumber,
NtOsInstall->InstallationName, &NtOsInstall->SystemNtPath); NtOsInstall->InstallationName, &NtOsInstall->SystemNtPath);
} }
DPRINT1("Done.\n"); DPRINT("Done.\n");
} }
#endif
static PNTOS_INSTALLATION static PNTOS_INSTALLATION
FindExistingNTOSInstall( FindExistingNTOSInstall(
@ -661,7 +663,7 @@ FindNTOSInstallations(
L"\\Device\\Harddisk%lu\\Partition%lu\\", L"\\Device\\Harddisk%lu\\Partition%lu\\",
DiskNumber, PartitionNumber); DiskNumber, PartitionNumber);
RtlInitUnicodeString(&PartitionRootPath, PathBuffer); RtlInitUnicodeString(&PartitionRootPath, PathBuffer);
DPRINT1("FindNTOSInstallations: PartitionRootPath: '%wZ'\n", &PartitionRootPath); DPRINT("FindNTOSInstallations: PartitionRootPath: '%wZ'\n", &PartitionRootPath);
/* Open the partition */ /* Open the partition */
InitializeObjectAttributes(&ObjectAttributes, InitializeObjectAttributes(&ObjectAttributes,
@ -691,13 +693,13 @@ FindNTOSInstallations(
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
/* The loader does not exist, continue with another one */ /* The loader does not exist, continue with another one */
DPRINT1("Loader type '%d' does not exist, or an error happened (Status 0x%08lx), continue with another one...\n", DPRINT("Loader type '%d' does not exist, or an error happened (Status 0x%08lx), continue with another one...\n",
Type, Status); Type, Status);
continue; continue;
} }
/* The loader exists, try to enumerate its boot entries */ /* The loader exists, try to enumerate its boot entries */
DPRINT1("Analyse the OS installations for loader type '%d' in disk #%d, partition #%d\n", DPRINT("Analyze the OS installations for loader type '%d' in disk #%d, partition #%d\n",
Type, DiskNumber, PartitionNumber); Type, DiskNumber, PartitionNumber);
Status = OpenBootStoreByHandle(&BootStoreHandle, PartitionDirectoryHandle, Type, FALSE); Status = OpenBootStoreByHandle(&BootStoreHandle, PartitionDirectoryHandle, Type, FALSE);
@ -750,7 +752,7 @@ CreateNTOSInstallationsList(
DiskEntry = CONTAINING_RECORD(Entry, DISKENTRY, ListEntry); DiskEntry = CONTAINING_RECORD(Entry, DISKENTRY, ListEntry);
Entry = Entry->Flink; Entry = Entry->Flink;
DPRINT1("Disk #%d\n", DiskEntry->DiskNumber); DPRINT("Disk #%d\n", DiskEntry->DiskNumber);
/* ... and for each disk, loop each available partition */ /* ... and for each disk, loop each available partition */
@ -763,7 +765,7 @@ CreateNTOSInstallationsList(
ASSERT(PartEntry->DiskEntry == DiskEntry); ASSERT(PartEntry->DiskEntry == DiskEntry);
DPRINT1(" Primary Partition #%d, index %d - Type 0x%02x, IsLogical = %s, IsPartitioned = %s, IsNew = %s, AutoCreate = %s, FormatState = %lu -- Should I check it? %s\n", DPRINT(" Primary Partition #%d, index %d - Type 0x%02x, IsLogical = %s, IsPartitioned = %s, IsNew = %s, AutoCreate = %s, FormatState = %lu -- Should I check it? %s\n",
PartEntry->PartitionNumber, PartEntry->PartitionIndex, PartEntry->PartitionNumber, PartEntry->PartitionIndex,
PartEntry->PartitionType, PartEntry->LogicalPartition ? "TRUE" : "FALSE", PartEntry->PartitionType, PartEntry->LogicalPartition ? "TRUE" : "FALSE",
PartEntry->IsPartitioned ? "TRUE" : "FALSE", PartEntry->IsPartitioned ? "TRUE" : "FALSE",
@ -785,7 +787,7 @@ CreateNTOSInstallationsList(
ASSERT(PartEntry->DiskEntry == DiskEntry); ASSERT(PartEntry->DiskEntry == DiskEntry);
DPRINT1(" Logical Partition #%d, index %d - Type 0x%02x, IsLogical = %s, IsPartitioned = %s, IsNew = %s, AutoCreate = %s, FormatState = %lu -- Should I check it? %s\n", DPRINT(" Logical Partition #%d, index %d - Type 0x%02x, IsLogical = %s, IsPartitioned = %s, IsNew = %s, AutoCreate = %s, FormatState = %lu -- Should I check it? %s\n",
PartEntry->PartitionNumber, PartEntry->PartitionIndex, PartEntry->PartitionNumber, PartEntry->PartitionIndex,
PartEntry->PartitionType, PartEntry->LogicalPartition ? "TRUE" : "FALSE", PartEntry->PartitionType, PartEntry->LogicalPartition ? "TRUE" : "FALSE",
PartEntry->IsPartitioned ? "TRUE" : "FALSE", PartEntry->IsPartitioned ? "TRUE" : "FALSE",
@ -799,8 +801,10 @@ CreateNTOSInstallationsList(
} }
} }
#ifndef NDEBUG
/**** Debugging: List all the collected installations ****/ /**** Debugging: List all the collected installations ****/
DumpNTOSInstalls(List); DumpNTOSInstalls(List);
#endif
return List; return List;
} }