mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] partitions: efi: print GPT offset in debug message
@ 2022-03-19  7:24 Ahmad Fatoum
  2022-03-28  8:54 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2022-03-19  7:24 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

This debug message is printed for the alternative GPT header as well,
which may not exist at that location, because the medium has a different
length than what's in the GPT header. Make it easier to differentiate,
whether the message refers to the primary or alternate GPT header by
printing the LBA the header was expected at.

Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
---
 common/partitions/efi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/partitions/efi.c b/common/partitions/efi.c
index 0787b93f120b..848bed22610f 100644
--- a/common/partitions/efi.c
+++ b/common/partitions/efi.c
@@ -154,8 +154,8 @@ static int is_gpt_valid(struct block_device *blk, u64 lba,
 
 	/* Check the GPT header signature */
 	if (le64_to_cpu((*gpt)->signature) != GPT_HEADER_SIGNATURE) {
-		dev_dbg(blk->dev, "GUID Partition Table Header signature is wrong:"
-			"0x%llX != 0x%llX\n",
+		dev_dbg(blk->dev, "GUID Partition Table Header signature at LBA"
+			"%llu is wrong: 0x%llX != 0x%llX\n", lba,
 			(unsigned long long)le64_to_cpu((*gpt)->signature),
 			(unsigned long long)GPT_HEADER_SIGNATURE);
 		goto fail;
-- 
2.34.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] partitions: efi: print GPT offset in debug message
  2022-03-19  7:24 [PATCH] partitions: efi: print GPT offset in debug message Ahmad Fatoum
@ 2022-03-28  8:54 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2022-03-28  8:54 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox

On Sat, Mar 19, 2022 at 08:24:27AM +0100, Ahmad Fatoum wrote:
> This debug message is printed for the alternative GPT header as well,
> which may not exist at that location, because the medium has a different
> length than what's in the GPT header. Make it easier to differentiate,
> whether the message refers to the primary or alternate GPT header by
> printing the LBA the header was expected at.
> 
> Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
> ---
>  common/partitions/efi.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied, thanks

Sascha

> 
> diff --git a/common/partitions/efi.c b/common/partitions/efi.c
> index 0787b93f120b..848bed22610f 100644
> --- a/common/partitions/efi.c
> +++ b/common/partitions/efi.c
> @@ -154,8 +154,8 @@ static int is_gpt_valid(struct block_device *blk, u64 lba,
>  
>  	/* Check the GPT header signature */
>  	if (le64_to_cpu((*gpt)->signature) != GPT_HEADER_SIGNATURE) {
> -		dev_dbg(blk->dev, "GUID Partition Table Header signature is wrong:"
> -			"0x%llX != 0x%llX\n",
> +		dev_dbg(blk->dev, "GUID Partition Table Header signature at LBA"
> +			"%llu is wrong: 0x%llX != 0x%llX\n", lba,
>  			(unsigned long long)le64_to_cpu((*gpt)->signature),
>  			(unsigned long long)GPT_HEADER_SIGNATURE);
>  		goto fail;
> -- 
> 2.34.1
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-03-28  8:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-19  7:24 [PATCH] partitions: efi: print GPT offset in debug message Ahmad Fatoum
2022-03-28  8:54 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox