[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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.