[PATCH v2] powerpc: increase MIN RMA size for CAS negotiation

2025-03-03 Thread Avnish Chouhan
Change RMA size from 512 MB to 768 MB which will result in more memory at boot time for PowerPC. When vTPM, Secure Boot or FADump are enabled on PowerPC, the 512 MB RMA memory is not sufficient for booting. With this 512 MB RMA, GRUB2 runs out of memory and fails to boot the machine. Sometimes even

Re: [SECURITY PATCH 18/73] fs/ntfs: Implement attribute verification

2025-03-03 Thread Glenn Washburn
On Sun, 02 Mar 2025 09:41:28 +0100 "Thomas Schmitt" wrote: > Hi, > > Glenn Washburn wrote: > > Seems like running them the tests before > > committing large patch series, like this security update, might be a > > good idea. > > I think that any change of the filesystem code should be tested as

[PATCH v2 5/6] tests/grub_cmd_cryptomount: Remove temporary directories if successful and debug is not on

2025-03-03 Thread Glenn Washburn
From: Thomas Schmitt grub_cmd_cryptomount creates a directory per subtest. If a subtest is successful and debugging is not on, the directory should be empty. So it can be deleted. Signed-off-by: Glenn Washburn --- tests/grub_cmd_cryptomount.in | 9 + 1 file changed, 9 insertions(+) di

GRUB Error: Couldn't Find Suitable Memory Target for 210MB Initramfs

2025-03-03 Thread Zhang, Wentao via Grub-devel
Hi GRUB-devel team I encountered an issue with a 210MB initramfs where GRUB prints the following error: error: couldn't find suitable memory target. Below are parts of my memory layout: .. mmap/efi/mmap.c:63: EFI memory region 0xdf6e000-0x17238000: 7 mmap/efi/mmap.c:63: EFI memory regio

Re: [SECURITY PATCH 18/73] fs/ntfs: Implement attribute verification

2025-03-03 Thread Thomas Schmitt via Grub-devel
Hi, i wrote: > > it is quite some hurdle that some of the tests need to be run as > > superuser. Glenn Washburn wrote: > Yes, not ideal. I have spent an embarrassing amount of time developing > scripts that do away with with root requirement, and it works. The way > they are able to run the root

[PATCH v2 4/6] tests/grub_cmd_cryptomount: Default TMPDIR to /tmp

2025-03-03 Thread Glenn Washburn
This fixes behavior where grub_cmd_cryptomount temporary files, which are some times not cleaned up, are left in the / directory. Set TMPDIR if your system does not have /tmp or it can not be used for some reason. Reported-by: Thomas Schmitt Signed-off-by: Glenn Washburn --- tests/grub_cmd_cryp

[PATCH v2 2/6] tests: Cleanup generated files on expected failure in grub_cmd_cryptomount

2025-03-03 Thread Glenn Washburn
grub-shell-luks-tester only cleans up generated files when the test it runs returns success. Sometimes tests are run that should fail. Add a --xfail argument to grub-shell-luks-tester and pass it from grub_cmd_cryptomount when invoking a test that is expected to fail. Reported-by: Thomas Schmitt

[PATCH v2 6/6] tests/util/grub-shell: Remove the work directory on successful run and debug is not on

2025-03-03 Thread Glenn Washburn
This removes alot of empty grub-shell working directories in the TMPDIR directory. Signed-off-by: Thomas Schmitt Signed-off-by: Glenn Washburn --- tests/util/grub-shell.in | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/util/grub-shell.in b/tests/util/grub-shell.in index 15c5f45a5c30.

[PATCH v2 1/6] tests/util/grub-shell-luks-tester: Add missing line to create RET variable in cleanup

2025-03-03 Thread Glenn Washburn
Set the RET variable to the exit status of the script, as was assumed in the cleanup() function. Reported-by: Thomas Schmitt Signed-off-by: Glenn Washburn --- tests/util/grub-shell-luks-tester.in | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/util/grub-shell-luks-tester.in b/tests/u

[PATCH v2 0/6] Various test fixes proposed by Thomas Schmitt

2025-03-03 Thread Glenn Washburn
Thomas Schmitt proposed variations on these fixes[1]. I've broken his patch into several patches with improvements. Glenn [1] https://lore.kernel.org/all/9956308756800479...@scdbackup.webframe.org/ v2: * Add patch #2 to allow grub-shell-luks-tester to cleanup properly on expected failure. *

[PATCH v2 3/6] tests/grub_cmd_cryptomount: Cleanup the cryptsetup script unless debug is enabled

2025-03-03 Thread Glenn Washburn
This fixes an issue where the grub_cmd_cryptomount test leaves a file with an ambiguous name in the / directory when TMPDIR is not set. Reported-by: Thomas Schmitt Signed-off-by: Glenn Washburn --- tests/grub_cmd_cryptomount.in | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/grub_c

Re: Feedback Request: Implement Fuzzers and Add to OSS-Fuzz

2025-03-03 Thread Lidong Chen via Grub-devel
> On Feb 28, 2025, at 7:57 AM, Andrew Hamilton wrote: > > Yes that sounds great, I am back from vacation just now so will be generally > free any time between 0800 and 2100 Central Standard Time. I’m mostly available from 9am PT onward. Thanks, Lidong > > Thanks, > Andrew > On Fri, Feb 28,

Re: [PATCH 1/1] lib: Fix clang Build - Remove Duplicate Typedef in TSS

2025-03-03 Thread Gary Lin via Grub-devel
On Sun, Mar 02, 2025 at 10:54:04AM -0600, Andrew Hamilton wrote: > tss2_structs.h contains a duplicate typedef as follows: > typedef TPMS_SCHEME_HASH_t TPMS_SCHEME_KDF2_t; > > This causes a build failure when compiling with clang. > Remove the duplicate typedef which allows successfully > building

Re: [PATCH 1/1] lib: Fix clang Build - Remove Duplicate Typedef in TSS

2025-03-03 Thread Ross Philipson via Grub-devel
On 3/2/25 8:54 AM, Andrew Hamilton wrote: tss2_structs.h contains a duplicate typedef as follows: typedef TPMS_SCHEME_HASH_t TPMS_SCHEME_KDF2_t; This causes a build failure when compiling with clang. Remove the duplicate typedef which allows successfully building GRUB with clang. Signed-off-by: