Your message dated Fri, 25 Apr 2014 12:55:02 -0500
with message-id <[email protected]>
and subject line Re: Bug#741235: efibootmgr patch
has caused the Debian Bug report #741235,
regarding extra padding in HARDDRIVE_DEVICE_PATH structure breaks UEFI on some 
systems
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
741235: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=741235
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: efibootmgr
Version: 0.5.4-7
X-Debbugs-CC: [email protected]
Tags: patch
Severity: important

In reference to Ubuntu bug #1272664
(https://bugs.launchpad.net/bugs/1272664), efibootmgr adds 6 bytes of
padding to the boot entry. This breaks at lest Hyper-V, possible other
systems as well. Comparing the Debian sources with
https://github.com/vathpela/efibootmgr/blob/master/src/include/efi.h,
shows that the padding is apparently only needed on IA64 architecture.
The attached patch applies this limitation to the Debian source tree.

I ask the maintainer to apply the attached patch and release a new
version. Thanks.
diff --git a/src/include/efi.h b/src/include/efi.h
index be667ae..c2ac853 100644
--- a/src/include/efi.h
+++ b/src/include/efi.h
@@ -294,7 +294,9 @@ typedef struct {
 	uint8_t  signature[16];
 	uint8_t  mbr_type;
 	uint8_t  signature_type;
+#ifdef __ia64
 	uint8_t  padding[6]; /* Emperically needed */
+#endif
 } __attribute__((packed)) HARDDRIVE_DEVICE_PATH;
 
 typedef struct {

Attachment: signature.asc
Description: OpenPGP digital signature


--- End Message ---
--- Begin Message ---
On Fri, Apr 25, 2014 at 01:08:48AM -0500, Jan Henke wrote:
The patch in question is part of git commit de0ca44637, which has been
merged upstream on November 29th 2012, according to GitHub.

So a new upstream release should include this patch as well.
--
Best Regards
Jan

Correct. However, you stated an issue with Hyper-V, which I can't test, so I was wanting to know if you can test to make sure that the issue is resolved for you.

Based on your statement, I'll mark this bug as done. Please reopen it if you continue to have issues.


--
Jared Domínguez
Server OS Engineering
Dell | Enterprise Solutions Group

--- End Message ---

Reply via email to