[PATCHv3 2/9] Xen: Fix the offset of the _domain field in struct page_info
ptesarik at suse.cz
Fri Aug 24 11:41:55 EDT 2012
The page_info structure was re-arranged in Xen-3.4 (with changeset
162cdb596b9a), and the _domain field moved from union "u" to union "v"
on x86 and x86_64.
By using OFFSET_IN_UNION_INIT we can cover all cases:
1. old placement in union "u"
2. new placement in union "v" on x86 and x86_64
3. unmodified placement in union "u" on ia64
Unfortunately, the re-arrangement was not reflected in xen/common/kexec.c,
so the page_info _domain offset was still set with:
VMCOREINFO_OFFSET_ALIAS(page_info, u, _domain);
The bug was spotted during Xen-4.1 development (changeset cb756381087c),
and all Xen-4.1 releases are fixed like this:
VMCOREINFO_OFFSET_SUB(page_info, u.inuse, _domain);
VMCOREINFO_OFFSET_SUB(page_info, v.inuse, _domain);
This changeset was also backported to Xen-4.0 branch as changeset
3b90a5353f20, which first went into Xen-4.0.2.
For Xen-3.4, this bug was never fixed.
To sum it up:
1. Xen versions up to and including Xen-3.3 output the correct offset
2. Xen-4.1 and above is also correct
3. Xen-4.0 is broken up to 4.0.2
4. Xen-3.4 is always broken
Some OS vendors may also have backported the patch to an otherwise broken
version, so the above summary may not be reliable.
Anyway, the VMCOREINFO produced by some Xen versions is wrong, and
makedumpfile will behave incorrectly. Adding a workaround to makedumpfile
for a bug in another package is wrong, because:
A. Self-reliant users should upgrade Xen.
B. Users of distro packages should report it as a bug to their vendor, and
the vendor should backport the patch.
C. Users who need an immediate workaround should generate a vmcoreinfo file
using makedumpfile, which will be correct after applying this patch.
Signed-off-by: Norbert Trapp <norbert.trapp at ts.fujitsu.com>
Signed-off-by: Petr Tesarik <ptesarik at suse.cz>
makedumpfile.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
@@ -5306,10 +5306,7 @@ get_structure_info_xen(void)
OFFSET_INIT(page_info.count_info, "page_info", "count_info");
- * _domain is the first member of union u
- OFFSET_INIT(page_info._domain, "page_info", "u");
+ OFFSET_IN_UNION_INIT(page_info._domain, "page_info", "_domain");
OFFSET_INIT(domain.domain_id, "domain", "domain_id");
More information about the kexec