[CSRSS] Do not give IOPL to the Client/Server native process if ran on a AMD64 system

ProcessUserModeIOPL is strictly implemented for 32-bit architecture so it doesn't make any sense for CSRSS to gather user mode I/O privilege when that won't work anyway.

[NTOS:PS] Fix several issues within info classes in AMD64 build

ProcessUserModeIOPL, ProcessWow64Information and ThreadZeroTlsCell classes fail on AMD64 build because of wrong IQS values assigned to them. Also explicitly tell the compiler that ProcessUserModeIOPL is strictly for x86 as user mode I/O privilege level is 32-bit stuff.
In addition to that, implement IQS_NO_TYPE_LENGTH macro which it'll be used for classes such as ProcessUserModeIOPL where type length is not required and that it should be 0. With that said, we indirectly fix a size length mismatch issue with ProcessUserModeIOPL on 32-bit of ReactOS as well.

[NTOS:MM] Implement MmFlushImageSection(MmFlushForDelete)

CORE-17544

[NTOS:MM] Simplify refcounting when adding/deleting a page in a shared segment

CORE-17544

[NTOS:MM] Do not call ExAllocatePool* with PFN lock acquired

Unsurprisingly, it actually might acquire it

[ATL] Close m_hKey in CRegKey destructor and when attaching to a new key (#3764)

[NTOSKRNL] Enforce -fno-common on GCC & CLang builds

[NTOSKRNL] Deduplicate some symbols

[NTOS:MM] Fix-up call site of traced PFNs when serving a page fault

[NTOS:MM] Save caller address when tracing PFNs

[CMD] Do not use an intermediate buffer when reading lines from batch files

This is easier on the heap and improves cmd:batch winetest nicely

[CMD] Improve performance a bit

Do not use custom alloc/free functions by default.
Do not free resources at process exit.
 - You're doing it brick by brick while the OS  & CRT do that with a bulldozer better than you do.

[WIN32K] CreateDIBPalette addendum to #3758 CORE-17626

Fix compilation on older GCC4.7.2 toolchain
to not warn about universal zero initializer.
See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53119
https://stackoverflow.com/questions/13746033/how-to-repair-warning-missing-braces-around-initializer

[WIN32K] CreateDIBPalette SEH Simplification #3758 CORE-17626

Enter an SEH2_TRY one time and do all testing within it instead of entering the SEH_TRY multiple times.

The commit is an addendum to 0.4.15-dev-2734-g 514147776a7e70636911033ab6c89779b2c8ee1e

Thanks to patches author Doug-Lyons

This is 1:1 the final approved content of #3758, but I committed
by hand and closed the PR to avoid squash+rebase.

[NTOS:MM] Fix one occurrence of MiDeletePte instead of MiDeletePde

[NTOS:CC] CcRosFlushVacb(): Fix Iosb annotation

Addendum to 2ba1926.

[NTOS:MM] MmFlushSegment(): Fix Iosb annotation

Addendum to 90c6a65.

[XBOX][FREELDR] Initialize PCI-AGP bridge BAR values

This fixes problem with Xbox GPU not being detected as a children device.

CORE-17600 CORE-16216

[UMPNPMGR] PNP_GetDeviceStatus: Add status checks as documented in MS-PNPR

CORE-17527

[SAMSRV] Fix call to SamrCloseHandle in SamrGetDomainPasswordInformation

[UMPNPMGR] Add missing parameter checks to PNP_GetDeviceStatus

[NTOS:KE] Do not allow waiting at IRQL >= DISPATCH_LEVEL when providing a timeout in KeWaitForMultipleObjects

CORE-6473

[TCPIP][IP] Use an ERESOURCE as mutex for address files & connection end points

Spinlocks are not reentrant (and this is done a lot), using them forces us to have
an horrible hack in the kernel, which unschedules threads which are at DISPATCH_LEVEL
thus allowing another thread to take ownership of the spinlock while the unscheduled
thread should already hold it.

CORE-6473

[USETUP] Update de-DE

Translate the Copying-bootloader status bar text
that was added during 0.4.15-dev'ing

[NE2000] Add description for Realtek RTL8019AS (emulated by 86Box)

[NE2000] Combine two INF files into one for ISA and PCI cards

[INF] machine.inf: Improve Russian translation

[INF] machine.inf: Minor whitespace fixes

[ISAPNP] Start the driver early in the boot

Because our child devices can have boot-start drivers

[ISAPNP] Implement device configuration

[ISAPNP] Disable bus scan on unsupported platforms

This speeds up boot a bit and also decreases binary size.

- Introduce a new CMake define ISAPNP_ENABLE that defined only in x86 platform
  when ISA PnP is supported.

[ISAPNP] Disable I/O range check before activation

See Table A-2 of "Plug and Play ISA Specification 1.0a"

[ISAPNP] Map configuration states to PnP states

Rework the hardware support code to avoid unneeded transitions
between configuration states.
- Only call WaitForKey when needed.
- Each PnP state transition causes the bus to move to WaitForKey state.

Besides, deactivate logical device during processing remove request.

[ISAPNP] Reduce software delay

[ISAPNP] Fix descriptors and support alternative configurations

- Fix empty resource descriptors being created.
- Properly support IRQ descriptors.
- Introduce four helpers made to help search descriptors
  in the logical device's requirements.
- Implement support for memory descriptors and alternative configurations.
- DMA descriptors are always DMA_8.

[ISAPNP] Read all resources when detecting devices

[ISAPNP] Rewrite the tag parser

- Support all resource descriptors.
- Optimize card identification.
- Detect cards that is no longer present on the bus.
- Deactivate cards after the identification phase; they will be activated
  by start device IRP.
- Provide a device description and compatible IDs to the device manager.
- Prevent duplicate IDs across multiple logical devices.
- Suppress warning about the usage of literals in port addresses.

[ISAPNP] Use the correct types for helper functions

[ISAPNP] Implement the Read Port resource management

Currently disabled until the kernel is ready

[ISAPNP] Fixes and improvements for IRP handling

- Implement device removal.
- Finish the Read Port special handling
  introduced in 21514e473f5.
- Gracefully handle low memory situations.
- Implement IRQ translator.
- Correctly stub power IRP handling.
- Remove unneeded dispatch routines; implement WMI IRP handling.
- Do not handle requests like WRITE_CONFIG.
- Add a signature member to device extensions to make checks clearer.

[ISAPNP] Implement querying bus information

[ISAPNP] Properly count descriptors

Fixes random pool corruptions

[ISAPNP] Fix device capabilities

- The bus does not supply an address.
- Read Port ID is always unique.

This reverts commit fc0a0b9f84e1f284914b4576305cbf11c9015ad6.
See also commit 21514e473f5.

CORE-17034

[ISAPNP] Make Read Data Port PDO unique

This PDO is created only once during start of first FDO.
Other buses will remain in an inactive state until
the first FDO receives a remove request.

CORE-17034

[ISAPNP] Refactor string handling

- Don't allocate string buffers twice.

[ISAPNP] Make the driver pageable

[ISAPNP] Use pool tagging

[ISAPNP] Code formatting

- Add SAL2 annotations.
- Use RTL_NUMBER_OF macro in kernel mode.
- Use better function/member names.
- Less hardcoded values.
- Add license information. See commit
  d9face83c693cb5e3fad155df8609d9c13316061 for details.

No functional changes.

[UNIATA] Workaround random crashes on Virtual PC (#3377)

I am not sure why this line was added.
I don't know how I could improve that case.
Then, disable it, for the time being.

Addendum to 4b9cf2e (r71252). CORE-12441 CORE-17371

[UNIATA] BusMasterAdapters[]: 8086:7111 is really PIIX4, not PIIX3

Reverts a part of deaba45 (r57265).

[SAMLIB] Check the server name in the binding routine

[SAMLIB] SamConnect: Pass a zero-terminated server name to SamrConnect

[PCI] Filter out devices with null vendor and device ID

Some virtual machines errorneously expose null PCI device function
on PIIX4 chipset where an USB controller should normally reside.

Windows pci.sys driver does not enumerate these devices.

Affected virtual machines:
- Connectix Virtual PC 5.1.370
- Connectix Virtual PC 5.2.418
- Microsoft Virtual PC 2004 SP1 (5.3.582.27)
- Microsoft Virtual PC 2007 SP1 (6.0.156.0)

Non-affected virtual machines:
- Connectix Virtual PC 4.0
- Microsoft Hyper-V

Fixes CORE-17636.

[WIN32K] Revert incorrect part of R50928 for RealizePalette. (#3678)

Revert incorrect code for Realize Palette
Revert part of SVN R50928 that causes Durak card suites to have wrong colors.
SVN r50928 == git 5de8339cd196e2bc01ac631e677010a939b2d083

JIRA issue: CORE-13748 <= Durak Example
JIRA issue: CORE-16510 <= GDIProg Example

The patch also fixes 2 crashes on each bot, although
it was not targeted for that explicitly:
KVM: https://reactos.org/testman/compare.php?ids=77304,77308 LGTM
VBox: https://reactos.org/testman/compare.php?ids=77305,77309 LGTM

[NTGDI] Fix potential BSOD 0x1E CORE-17626

in CreateDIBPalette() when passing invalid arguments to CreateDIBSection.
This could be triggered by using the broken test-application "GDIProg".

After this patch not only the BSOD is fixed but also the app does
properly start up, like it is the case on 2k3sp2.

Thanks to the patches author Doug Lyons.

[SDK] Add script to check includes with modified struct packing

[NTOS][SETUP] Add x64 SMP kernel and make SMP the default boot mode on x64

[HAL] Add smp component and build x64 smp hal

Like the up component it contains the spinlock code and now also the HalpBuildType variable.

[NTOS:MM] Improve an ASSERT

[NTOS:MM] Fix PFN lock usage

[NTOS:KE] Fix usage of KTHREAD::SwapBusy

[NTOS:KE] Fix some locking issues

[NTOS:KE] Fix KxReleaseSpinLock for 64 bit SMP builds

[NTOS] Fix build with CONFIG_SMP

[HAL] Fix build with CONFIG_SMP

[FREELDR/x64] Allow using memory above 4GB on x64

This splits MM_MAX_PAGE into the maximum addressable PFN (which is 2^20-1 on x86, 2^26-1 on x86PAE and 2^36-1 on x64) and the maximum mapped/accessible virtual memory MM_MAX_PAGE_LOADER (which is 4 GB on x86, since paging is disabled, but only 1 GB on x64, since only that much is identity-mapped).

[NTOS:MM/x64] Fix Interlocked(Compare)ExchangePte and PAGE_FROM_SSE for x64

This fixes using PFNs with more than 20 bits.

[NTOS:SE] Minor refactor on NtOpenThreadTokenEx

- Remove a redundant call of ObReferenceObjectByHandle. Not only it didn't make much sense (we reference the object from thread handle and the new thread object referencing the same handle!), specifying a request access of THREAD_ALL_ACCESS for the thread object is kind of suspicious and all of these access rights are unwanted.
- Add some failure checks involving the CopyOnOpen code paths
- Add some DPRINT1 debug prints (concerning the CopyOnOpen code paths as usual)

[HAL] Add APIC hals to setup and livecd

[HAL:APIC] Fix interrupt assignment

This is probably not how it is supposed to work, but the results are somewhat similar to what Windows has.

[HAL:APIC] Remove obsolete hacks

These hacks were used to work around broken VBox. VBox works fine without the hacks these days.

[HAL:APIC] Code fixes

* Fix some broken code
* Add some ASSERTs
* Use ApicWriteIORedirectionEntry where appropriate
* Use KeQueryInterruptHandler/KeRegisterInterruptHandler to save/restore the old handler instead of saving the KIDTENTRY
* Move HalpProfileInterruptHandler to apictimer.c and implement it
* Use READ/WRITE_REGISTER macros
* Add some symbolic names

[MINIHAL] Compile less stuff

[HAL] Rename apic.h to apicp.h and move it back to the apic folder

[NTOS:IOPNP] Both PNP_PROPERTY_REMOVAL_POLICY_OVERRIDE and PNP_PROPERTY_LOCATION_PATHS are unimplemented, return such status.

Fixes CID 1441387: Uninitialized scalar variable (UNINIT),
in the case of PNP_PROPERTY_LOCATION_PATHS.

[NTOS:IO] Comment out (and explain why it is there) an unused LdrEntry assignment in IopInitializeBootDrivers(). CID 1237114

[NTOS:IO] Revert part of commit a82ff90b2: don't do direct ULONG data access: it might be unaligned.

Requested by Victor.

[NTOS:IO] Fix driverName.Buffer leak in some failure paths in IopGetDriverNames().

driverName.Buffer leaked when the "(!NT_SUCCESS(status) || ServiceName != NULL)"
case is taken because ServiceName != NULL, and some of the functions fail.

[NTOS:IO] Fix basicInfo handling in IopGetDriverNames().

- Fix CID 1477246: Uninitialized pointer read (UNINIT) (happens in
  the last ExFreePoolWithTag(basicInfo, TAG_IO) call when the
  "(!NT_SUCCESS(status) || ServiceName != NULL)" case is not taken).

- Centralize all the ExFreePoolWithTag(basicInfo, TAG_IO) cleanups
  at the end of the function.

- Both cases "(driverName.Buffer == NULL)" and "(ServiceName != NULL)"
  can only be taken when basicInfo != NULL, so assert on this fact.

[NTOS:IO] Fix similar bugs in IopGetDriverNames(), IopInitializeBuiltinDriver() and IopLoadDriver().

- As they are initialized, some registry string values must be NULL-terminated.
- When retrieving REG_DWORD registry values, validate their length.

[NTOS:IO] Minor code style changes...

- Fix/add comments;
- Reduce indentation level;
- Direct copy for registry integer values;
- Use for-loops for linked lists;
- Use ULONG when the API uses it (sizes for Ob, or REG_DWORD data in registry).

[LDR] Restore Logging when images overlap CORE-17635

This reverts
0.4.13-dev-284-g 74be88ac473425fc99e9a363ba8cd604702262bf

The logging is helpful to alert us when we should really
recalculate the base addresses for performance reasons.
Spams a bit sometimes, sure, but it's worth it!

[NTOS:MM] Fix placing entries into MmNonPagedPoolFreeListHead

When freeing pages, free page entries with pages num == 3 were
incorrectly treated as entries with pages num >= 4 and thus
their re-insertion was not triggered. That lead to non paged pool
fragmentation (can be triggered by kmtest:ExPools, for example)

Also, altered the index acquisition code for MmNonPagedPoolFreeList
entries so it looks more clear

[COMCTL32] Strip EOL whitespace within ifdef __REACTOS__ section

no functional change.
Make that look beautiful before porting it back further.

Addendum to 0.4.15-dev-534-g 887764e6075fdde7fb81d2a54d5255164c046511
(CORE-17062)

[MSI] Enumerate only type DRIVE_FIXED CORE-17623 (& CORE-14432)

MSI on 2k3sp2 hides empty CD-ROM, and hides CD-ROM with iso mounted.
MSI on 2k3sp2 hides VBox shares, both: writable and read-only-ones.
MSI on 2k3sp2 hides floppy drives with a floppy img mounted.
MSI on XPSP3 hides a mounted writable USB stick.

Thanks to the patches author Doug Lyons.

This patch supersedes
0.4.15-dev-2651-g 864e20b881130cbc9668b8e253a31b8c325e9b99 (CORE-14432)
and we are also better than Wines
https://source.winehq.org/git/wine.git/commit/0dd0d879a1c487dcefedd44762d4ea0fcebd98fe

[GITIGNORE] Add compile_commands.json and .cache folder for clangd cache

[WIN32K] Add x64/arm exports

Checked against Windows XP x64 and Windows 8 ARM.

[FORMATTING][NTOS][HAL][DRIVERS][SDK] Fix trailing spaces

[NTOS:IO] Modify when 'PartitionBuffer' and how 'DriveLayout' are freed in IopCreateArcNamesDisk().

- Manage the lifetime of the temporary 'PartitionBuffer' buffer where
  it is locally used only, and free it as soon as possible, just after
  calculating the sector checksum. No need to then free it outside of
  the main for-loop.

- When the 'DriveLayout' buffer is freed, ensure the pointer is NULL-ed
  (and assert this at the top of the main for-loop), since it can also
  be freed at cleanup outside this for-loop, and in this case a NULL
  check is performed.
  This will avoid the scenario of possibly double-freeing a pointer,
  in the case the 'DriveLayout' was previously freed (when e.g. reading
  the sector for checksum calculation failed), then the for-loop goes to
  the next disk and stops early.

[NTOS:IO/FSTUB] Fix the determination of 'SingleDisk' in IoGetBootDiskInformation().

The purpose of 'SingleDisk' is the same as in the IopCreateArcNames()
function. It is an optimization for that when looking up the
firmware-recognized ARC disks list, in order to match one of these with
the current NT disk being analysed (see e.g. also in IopCreateArcNamesDisk()),
we avoid a possible IopVerifyDiskSignature() call and directly build a
corresponding ARC name NT symbolic link for it.

'SingleDisk' will actually be TRUE, whether the DiskSignatureListHead
list is empty or contains only one element: Indeed in only both these
cases, 'DiskSignatureListHead.Flink->Flink' will refer to the list head.
(If the list is empty but 'SingleDisk' is TRUE, this does not matter,
because the DiskSignatureListHead looking-up loop never starts.)

[NTOS:IO] Avoid hardcoding variable types in sizeofs when possible.

[NTOS:IO] Minor code style fixes (typos; improve comments/DPRINT; IN vs. OUT parameter).

[SHELL32_APITEST] Improve ShellExecCmdLine testcase more (#3736)

- Simplify code.
- Loose the condition of success.
- Add some missing tests.
- Close newly opened windows gracefully.
CORE-17351, CORE-16898, CORE-17612

[CMAKE] host-tools: Define ROS_SAVED_* for MSVC_IDE only

Addendum to 18e95f5.
ROSBE-174

[SOUNDS] Add new Recycle Bin system sound
Add new sound for Recycle Bin sound event.
Since this sound event is now working properly after some fixes in shell32 code, it's now possible for user to set the sound for this sound event manually.
But don't enable it by default, since it is also disabled on Windows Server 2003.
CORE-15868

[RAPPS] Fix double free when closing rapps

[HIDPARSER] Don't include pshpack1.h before including other headers

NEVER DO THIS! It is guaranteed to be wrong. Instead always individually pack single structures that need packing.
This fixes USB mouse on 64 bit builds.

[MORE] Use positional elements to build the MORE prompt, whose state is controlled by flags.

[MORE] Update and complete MORE's help usage.

And add french translation.

[WIN32K:ENG] In EngLockDriverObj, properly return NULL when locking failed

CORE-15958