[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [livepatch-build-tools part2 v2 5/6] create-diff-object: Add new entries to special sections array array
- To: "Wieczorkiewicz, Pawel" <wipawel@xxxxxxxxx>
- From: Ross Lagerwall <ross.lagerwall@xxxxxxxxxx>
- Date: Fri, 16 Aug 2019 15:19:46 +0100
- Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=ross.lagerwall@xxxxxxxxxx; spf=Pass smtp.mailfrom=ross.lagerwall@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
- Cc: "Pohlack, Martin" <mpohlack@xxxxxxxxx>, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxx>
- Delivery-date: Fri, 16 Aug 2019 14:20:11 +0000
- Ironport-sdr: 1C0XmSjIvxD2xTAHbsyzRM3lLaIKShYRED0OXqkmdc66+BZYU9frypSuEwPb4AuCMQiNuy7QUB Jg+aHNebkPQQ0PnT5ywDIs2710wVh8Juk7J8wVFBhtULQbeSap6ao2HUbD0lcFa0dAezxs950u bJ2EWLEu0lGF/DHaaXHzw5gIKykuFRL0O/Wxi6unIlnjjKc2lThllNYCpL62IYtFK4VSzSh6Uh X3+M/6csZHOYJ7eWYZMHR/NB+e0xEj060iEghXcrPrM5jxf/N4Hr0zAYgAA/Pt+JCoFM8c4OYi t94=
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 8/16/19 1:06 PM, Wieczorkiewicz, Pawel wrote:
On 16. Aug 2019, at 11:40, Ross Lagerwall <ross.lagerwall@xxxxxxxxxx
<mailto:ross.lagerwall@xxxxxxxxxx>> wrote:
On 8/8/19 1:35 PM, Pawel Wieczorkiewicz wrote:
…snip...
* The rela groups in the .fixup section vary in size. The
beginning of each
* .fixup rela group is referenced by the .ex_table section. To find
the size
@@ -1072,6 +1090,18 @@ static struct special_section
special_sections[] = {
.name= ".altinstructions",
.group_size= altinstructions_group_size,
},
+{
+.name= ".altinstr_replacement",
+.group_size= undefined_group_size,
+},
+{
+.name= ".livepatch.hooks.load",
+.group_size= livepatch_hooks_group_size,
+},
+{
+.name= ".livepatch.hooks.unload",
+.group_size= livepatch_hooks_group_size,
+},
{},
};
Unless I'm misunderstanding something, I can't see how
kpatch_regenerate_special_section would work with
.altinstr_replacement having a group size of 0. It looks to me like
the for loop in that function would become an infinite loop (due to
incrementing by group_size) and should_keep_rela_group would always
return false.
AFAICS, the group_size 0 sections are never actually processed by the
kpatch_regenerate_special_section().
They are not RELA sections and the following check excludes them from
this processing:
OK, that makes sense.
Thanks,
--
Ross Lagerwall
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
|
