Re: (mostly) Arch-independent livepatch

From: Jiri Kosina
Date: Tue Mar 29 2016 - 19:24:19 EST


On Tue, 29 Mar 2016, Jessica Yu wrote:

> > v6:
> > - Since we hard-code the field widths for the objname and symbol name
> > for the sscanf() calls, which are supposed to correspond to the values
> > of MODULE_NAME_LEN and KSYM_NAME_LEN, use BUILD_BUG_ON() to detect when
> > the values of these constants deviate from the expected values.
> > - Squash the sample livepatch module patch into patch 4
> > ("livepatch: reuse module loader code to write relocations") so
> > git bisects don't break
> > - Don't need the klp_buf struct, just use plain char arrays to hold the
> > output of sscanf(). Also, no need to clear the bufs after every
> > invocation, as sscanf() takes care to put a null byte at the end of
> > the bufs.
> > - Fix compiler kbuild errors for the !CONFIG_LIVEPATCH case
> > - Fixed some small module.c nits
> >
>
> Pinging Rusty, just in case this thread got buried :-)
> How do the module.c changes look?

Plus there are (admittedly indeed rather small and trivial) changes to
s390 module loader, so I'd prefer to have Heiko's / Martin's Ack before
merging this.

Hence, let me piggy back on this ping to Rusty, and let me ping Heiko and
Martin as well (adding to CC explicitly to make sure this doesn't get lost
in general noise).

Thanks,

--
Jiri Kosina
SUSE Labs