patina_smbios: Fix protocol installation and cleanup implementation

[kat-perez]

Description

Fixes SMBIOS protocol initialization to prevent allocation failures during SmbiosAdd() operations.

What changed

Fixed initialization order:

• Allocate SMBIOS buffers before installing protocol (was failing when SmbiosAdd() called during install)
• Publish initial table with Type 127 (End-Of-Table) marker immediately after protocol install
• Subsequent SmbiosAdd() calls update buffer in-place

C protocol compatibility improvements:

• GetNext now returns pointers into the published table (not internal buffer), ensuring RecordA != RecordB for different records
• Added checksum-based table integrity verification to detect direct modifications
• Returns TableDirectlyModified error if published table was modified directly instead of using protocol APIs
• Error message guides users to use Remove() + Add() or UpdateString() instead

Safety improvements:

• Scoped manager locks to prevent TplMutex re-entrant lock panic during republish
• Use write_unaligned() for FFI pointer writes
• Narrowed unsafe scopes in protocol layer

Code cleanup:

• Use MemoryManager service for allocations
• Replace hard-coded sizes with SIZE_(n)KB constants
• Enhanced error handling and debug_asserts

---

• Impacts functionality?
• Impacts security?
• Breaking change?
• Includes tests?
• Includes documentation?

How This Was Tested

• Integrated with Tiano DXE Core to test C FFI protocol functions
• Integrated with patched branch in patina-dxe-core-qemu to ensure it also works with Patina DXE Core
• Verified direct modification detection with test in PatinaSmbiosDxe driver

Integration Instructions

N/A

[codecov]

Codecov Report

❌ Patch coverage is 95.38462% with 27 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
components/patina_smbios/src/manager/core.rs	93.41%	24 Missing ⚠️
components/patina_smbios/src/manager/protocol.rs	76.92%	3 Missing ⚠️

📢 Thoughts on this report? Let us know!