J Arges <christopherar...@gmail.com>
---
drivers/net/ethernet/intel/igb/e1000_phy.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/intel/igb/e1000_phy.c
b/drivers/net/ethernet/intel/igb/e1000_phy.c
index 5b54254..93ec2d0 100644
--- a/drivers/net/ethernet/int
J Arges
---
drivers/net/ethernet/intel/igb/e1000_phy.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/intel/igb/e1000_phy.c
b/drivers/net/ethernet/intel/igb/e1000_phy.c
index 5b54254..93ec2d0 100644
--- a/drivers/net/ethernet/intel/igb/e1000_phy.c
+++ b/drivers/net
right after disabling functions.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 33 +++--
kernel/livepatch/core.c | 29 +
2 files changed, 56 insertions(+), 6 deletions(-)
diff --git a/include
right after disabling functions.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 33 +++--
kernel/livepatch/core.c | 29 +
2 files changed, 56 insertions(+), 6 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux
hooks
- handles properly when inserting/removing dependent modules
- can handle load_hook for vmlinux as well
The idea of these hooks are based off of the kpatch project.
Chris J Arges (1):
livepatch: add load/unload hooks to objects
include/linux/livepatch.h | 33
hooks
- handles properly when inserting/removing dependent modules
- can handle load_hook for vmlinux as well
The idea of these hooks are based off of the kpatch project.
Chris J Arges (1):
livepatch: add load/unload hooks to objects
include/linux/livepatch.h | 33
Noticed some minor spelling errors when looking through the code.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
fs/ecryptfs/crypto.c | 8
fs/ecryptfs/file.c | 4 ++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/fs/ecryptfs/crypto.c b/fs/ec
Noticed some minor spelling errors when looking through the code.
Signed-off-by: Chris J Arges
---
fs/ecryptfs/crypto.c | 8
fs/ecryptfs/file.c | 4 ++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
index 0d8eb34..e5e29f8
for bogus values and filters them in order to prevent crashes.
Signed-off-by: Chris J Arges <christopherar...@gmail.com>
---
drivers/input/touchscreen/wacom_w8001.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/input/touchscreen/wacom_w8001.c
b/drivers/input/touch
for bogus values and filters them in order to prevent crashes.
Signed-off-by: Chris J Arges
---
drivers/input/touchscreen/wacom_w8001.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/input/touchscreen/wacom_w8001.c
b/drivers/input/touchscreen/wacom_w8001.c
index bab3c6a
On Wed, Apr 27, 2016 at 10:08:08PM +0200, Jiri Kosina wrote:
> On Tue, 26 Apr 2016, Chris J Arges wrote:
>
> [ ... snip ... ]
> > > + + Kretprobes using the ftrace framework conflict with the patched
> >
> > + + Kretprobes using the ftrace framework conflic
On Wed, Apr 27, 2016 at 10:08:08PM +0200, Jiri Kosina wrote:
> On Tue, 26 Apr 2016, Chris J Arges wrote:
>
> [ ... snip ... ]
> > > + + Kretprobes using the ftrace framework conflict with the patched
> >
> > + + Kretprobes using the ftrace framework conflic
On Mon, Apr 25, 2016 at 05:14:35PM +0200, Petr Mladek wrote:
> LivePatch framework deserves some documentation, definitely.
> This is an attempt to provide some basic info. I hope that
> it will be useful for both LivePatch producers and also
> potential developers of the framework itself.
>
>
On Mon, Apr 25, 2016 at 05:14:35PM +0200, Petr Mladek wrote:
> LivePatch framework deserves some documentation, definitely.
> This is an attempt to provide some basic info. I hope that
> it will be useful for both LivePatch producers and also
> potential developers of the framework itself.
>
>
On Tue, Apr 12, 2016 at 12:16:00PM -0500, Josh Poimboeuf wrote:
> On Tue, Apr 12, 2016 at 09:44:43AM -0500, Chris J Arges wrote:
> > On Fri, Mar 25, 2016 at 02:34:55PM -0500, Josh Poimboeuf wrote:
> > > Once we have a consistency model, patches and their objects will be
> >
On Tue, Apr 12, 2016 at 12:16:00PM -0500, Josh Poimboeuf wrote:
> On Tue, Apr 12, 2016 at 09:44:43AM -0500, Chris J Arges wrote:
> > On Fri, Mar 25, 2016 at 02:34:55PM -0500, Josh Poimboeuf wrote:
> > > Once we have a consistency model, patches and their objects will be
> >
On Fri, Mar 25, 2016 at 02:34:55PM -0500, Josh Poimboeuf wrote:
> Once we have a consistency model, patches and their objects will be
> enabled and disabled at different times. For example, when a patch is
> disabled, its loaded objects' funcs can remain registered with ftrace
> indefinitely
On Fri, Mar 25, 2016 at 02:34:55PM -0500, Josh Poimboeuf wrote:
> Once we have a consistency model, patches and their objects will be
> enabled and disabled at different times. For example, when a patch is
> disabled, its loaded objects' funcs can remain registered with ftrace
> indefinitely
On Wed, Apr 06, 2016 at 02:09:01PM +0200, Miroslav Benes wrote:
> On Wed, 6 Apr 2016, Chris J Arges wrote:
>
> > I think this approach needs more thought and my code has bug(s).
>
> And indeed there is...
>
> long (*__kvm_arch_vm_ioctl)(struct file *filp, unsigned long
On Wed, Apr 06, 2016 at 02:09:01PM +0200, Miroslav Benes wrote:
> On Wed, 6 Apr 2016, Chris J Arges wrote:
>
> > I think this approach needs more thought and my code has bug(s).
>
> And indeed there is...
>
> long (*__kvm_arch_vm_ioctl)(struct file *filp, unsigned long
On Wed, Apr 06, 2016 at 11:09:04AM +0200, Miroslav Benes wrote:
> On Wed, 6 Apr 2016, Chris J Arges wrote:
>
> > On Tue, Apr 05, 2016 at 03:07:13PM +0200, Miroslav Benes wrote:
> > > On Mon, 4 Apr 2016, Josh Poimboeuf wrote:
> > >
> > > > So I th
On Wed, Apr 06, 2016 at 11:09:04AM +0200, Miroslav Benes wrote:
> On Wed, 6 Apr 2016, Chris J Arges wrote:
>
> > On Tue, Apr 05, 2016 at 03:07:13PM +0200, Miroslav Benes wrote:
> > > On Mon, 4 Apr 2016, Josh Poimboeuf wrote:
> > >
> > > > So I th
On Tue, Apr 05, 2016 at 03:07:13PM +0200, Miroslav Benes wrote:
> On Mon, 4 Apr 2016, Josh Poimboeuf wrote:
>
> > So I think this doesn't fix the problem. Dynamic relocations are
> > applied to the "patch module", whereas the above code deals with the
> > initialization order of the "patched
On Tue, Apr 05, 2016 at 03:07:13PM +0200, Miroslav Benes wrote:
> On Mon, 4 Apr 2016, Josh Poimboeuf wrote:
>
> > So I think this doesn't fix the problem. Dynamic relocations are
> > applied to the "patch module", whereas the above code deals with the
> > initialization order of the "patched
On Fri, Apr 01, 2016 at 05:46:52PM +0200, Miroslav Benes wrote:
> On Fri, 1 Apr 2016, Jiri Kosina wrote:
>
> > On Tue, 29 Mar 2016, Jiri Kosina wrote:
> >
> > > Agreed; I think we should be safe applying all the alternatives (with
> > > paravirt being really just a special case of those) to the
On Fri, Apr 01, 2016 at 05:46:52PM +0200, Miroslav Benes wrote:
> On Fri, 1 Apr 2016, Jiri Kosina wrote:
>
> > On Tue, 29 Mar 2016, Jiri Kosina wrote:
> >
> > > Agreed; I think we should be safe applying all the alternatives (with
> > > paravirt being really just a special case of those) to the
On Fri, Apr 01, 2016 at 05:46:52PM +0200, Miroslav Benes wrote:
> On Fri, 1 Apr 2016, Jiri Kosina wrote:
>
> > On Tue, 29 Mar 2016, Jiri Kosina wrote:
> >
> > > Agreed; I think we should be safe applying all the alternatives (with
> > > paravirt being really just a special case of those) to the
On Fri, Apr 01, 2016 at 05:46:52PM +0200, Miroslav Benes wrote:
> On Fri, 1 Apr 2016, Jiri Kosina wrote:
>
> > On Tue, 29 Mar 2016, Jiri Kosina wrote:
> >
> > > Agreed; I think we should be safe applying all the alternatives (with
> > > paravirt being really just a special case of those) to the
Paravirtualized ops and livepatching currently don't mix very well and can
cause undefined behavor such as oops, invalid opcodes or corrupted stacks.
The original discussion of this issue can be found here [1].
I've written an example livepatch module that reproduces the issue [2].
In order to
Paravirtualized ops and livepatching currently don't mix very well and can
cause undefined behavor such as oops, invalid opcodes or corrupted stacks.
The original discussion of this issue can be found here [1].
I've written an example livepatch module that reproduces the issue [2].
In order to
, objname);
> > + pr_err("unresolvable ambiguity on symbol '%s' in object '%s'\n",
>
> While you're at it, can you improve the grammar: s/on/for/ ?
>
> "unresolvable ambiguity *for* symbol..."
>
> Either way,
>
> Acked-by: Josh Poimboeuf <jpoim...@redhat.com>
>
> --
> Josh
>
I agree that the 'matches' error message doesn't added anything helpful.
With the word change above:
Acked-by: Chris J Arges <chris.j.ar...@canonical.com>
--chris
+ pr_err("unresolvable ambiguity on symbol '%s' in object '%s'\n",
>
> While you're at it, can you improve the grammar: s/on/for/ ?
>
> "unresolvable ambiguity *for* symbol..."
>
> Either way,
>
> Acked-by: Josh Poimboeuf
>
> --
> Josh
>
I agree that the 'matches' error message doesn't added anything helpful.
With the word change above:
Acked-by: Chris J Arges
--chris
Commit-ID: 3f62de5f6f369b67b7ac709e3c942c9130d2c51a
Gitweb: http://git.kernel.org/tip/3f62de5f6f369b67b7ac709e3c942c9130d2c51a
Author: Chris J Arges <chris.j.ar...@canonical.com>
AuthorDate: Fri, 22 Jan 2016 15:44:38 -0600
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate:
Commit-ID: 3f62de5f6f369b67b7ac709e3c942c9130d2c51a
Gitweb: http://git.kernel.org/tip/3f62de5f6f369b67b7ac709e3c942c9130d2c51a
Author: Chris J Arges
AuthorDate: Fri, 22 Jan 2016 15:44:38 -0600
Committer: Ingo Molnar
CommitDate: Wed, 24 Feb 2016 08:35:44 +0100
x86/kvm: Add output
Commit-ID: f05058c4d652b619adfda6c78d8f5b341169c264
Gitweb: http://git.kernel.org/tip/f05058c4d652b619adfda6c78d8f5b341169c264
Author: Chris J Arges <chris.j.ar...@canonical.com>
AuthorDate: Thu, 21 Jan 2016 16:49:25 -0600
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate:
Commit-ID: f05058c4d652b619adfda6c78d8f5b341169c264
Gitweb: http://git.kernel.org/tip/f05058c4d652b619adfda6c78d8f5b341169c264
Author: Chris J Arges
AuthorDate: Thu, 21 Jan 2016 16:49:25 -0600
Committer: Ingo Molnar
CommitDate: Wed, 24 Feb 2016 08:35:43 +0100
x86/uaccess: Add stack
Commit-ID: 5e947b38f8ead33bf6aaecc2af20a0a3a988fd02
Gitweb: http://git.kernel.org/tip/5e947b38f8ead33bf6aaecc2af20a0a3a988fd02
Author: Chris J Arges <chris.j.ar...@canonical.com>
AuthorDate: Thu, 21 Jan 2016 16:49:25 -0600
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate:
Commit-ID: 5e947b38f8ead33bf6aaecc2af20a0a3a988fd02
Gitweb: http://git.kernel.org/tip/5e947b38f8ead33bf6aaecc2af20a0a3a988fd02
Author: Chris J Arges
AuthorDate: Thu, 21 Jan 2016 16:49:25 -0600
Committer: Ingo Molnar
CommitDate: Tue, 23 Feb 2016 09:03:58 +0100
x86/uaccess: Add stack
Commit-ID: 3aff5415db7e5a5e50f9b05ce1b0b92a4a55e169
Gitweb: http://git.kernel.org/tip/3aff5415db7e5a5e50f9b05ce1b0b92a4a55e169
Author: Chris J Arges <chris.j.ar...@canonical.com>
AuthorDate: Fri, 22 Jan 2016 15:44:38 -0600
Committer: Ingo Molnar <mi...@kernel.org>
CommitDate:
Commit-ID: 3aff5415db7e5a5e50f9b05ce1b0b92a4a55e169
Gitweb: http://git.kernel.org/tip/3aff5415db7e5a5e50f9b05ce1b0b92a4a55e169
Author: Chris J Arges
AuthorDate: Fri, 22 Jan 2016 15:44:38 -0600
Committer: Ingo Molnar
CommitDate: Tue, 23 Feb 2016 09:04:01 +0100
x86/kvm: Add output
a noreturn attribute. Add this
to the global list because this particular attribute cannot be determined from
reading the ELF object.
Signed-off-by: Chris J Arges
---
tools/stacktool/builtin-check.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/stacktool/builtin-check.c b/tools
assmebly statement.
Signed-off-by: Chris J Arges
---
arch/x86/kvm/vmx.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
index e2951b6..e153522 100644
--- a/arch/x86/kvm/vmx.c
+++ b/arch/x86/kvm/vmx.c
@@ -8356,6 +8356,7 @@ static void
These patches fix a few warnings I saw testing stacktool v16.
I've done light testing on these by booting it on a machine and running
kvm-unit-tests on it.
Chris J Arges (2):
tools/stacktool: Add __reiserfs_panic to global_noreturns list
x86/kvm: Add output operand in vmx_handle_external_intr
On Fri, Jan 22, 2016 at 01:14:47PM -0600, Josh Poimboeuf wrote:
> On Fri, Jan 22, 2016 at 11:43:48AM -0600, Chris J Arges wrote:
> > On Thu, Jan 21, 2016 at 04:49:04PM -0600, Josh Poimboeuf wrote:
> > > This is v16 of the compile-time stack metadata validation patch set,
> &g
ay
>
> v14:
> - make tools/include/linux/list.h self-sufficient
> - create FRAME_OFFSET to allow 32-bit code to be able to access function
> arguments on the stack
> - add FRAME_OFFSET usage in crypto patch 14/24: "Create stack frames in
> aesni-intel_asm.S"
> - r
These patches fix a few warnings I saw testing stacktool v16.
I've done light testing on these by booting it on a machine and running
kvm-unit-tests on it.
Chris J Arges (2):
tools/stacktool: Add __reiserfs_panic to global_noreturns list
x86/kvm: Add output operand in vmx_handle_external_intr
assmebly statement.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
arch/x86/kvm/vmx.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
index e2951b6..e153522 100644
--- a/arch/x86/kvm/vmx.c
+++ b/arch/x86/kvm
a noreturn attribute. Add this
to the global list because this particular attribute cannot be determined from
reading the ELF object.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
tools/stacktool/builtin-check.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/sta
On Fri, Jan 22, 2016 at 01:14:47PM -0600, Josh Poimboeuf wrote:
> On Fri, Jan 22, 2016 at 11:43:48AM -0600, Chris J Arges wrote:
> > On Thu, Jan 21, 2016 at 04:49:04PM -0600, Josh Poimboeuf wrote:
> > > This is v16 of the compile-time stack metadata validation patch set,
> &g
ay
>
> v14:
> - make tools/include/linux/list.h self-sufficient
> - create FRAME_OFFSET to allow 32-bit code to be able to access function
> arguments on the stack
> - add FRAME_OFFSET usage in crypto patch 14/24: "Create stack frames in
> aesni-intel_asm.S"
> - r
disambiguation for relocations is added in the
next patch in this series.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(+), 50 deletions(-)
diff --git
be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git
- amend/remove commit messages/comments to be more precise
Chris J Arges (3):
livepatch: add old_sympos as disambiguator field to klp_func
livepatch: add sympos as disambiguator field to klp_reloc
livepatch: function,sympos scheme in livepatch sysfs directory
Documentation/ABI/testi
ned-off-by: Chris J Arges
---
include/linux/livepatch.h | 5 ++-
kernel/livepatch/core.c | 84 +++
2 files changed, 21 insertions(+), 68 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h
index b60e8ab..a882865 100
disambiguation for relocations is added in the
next patch in this series.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(
tching multiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files
- amend/remove commit messages/comments to be more precise
Chris J Arges (3):
livepatch: add old_sympos as disambiguator field to klp_func
livepatch: add sympos as disambiguator field to klp_reloc
livepatch: function,sympos scheme in livepatch sysfs directory
Documentation/ABI/testi
ned-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 5 ++-
kernel/livepatch/core.c | 84 +++
2 files changed, 21 insertions(+), 68 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatc
On Mon, Nov 23, 2015 at 10:52:23AM +0100, Miroslav Benes wrote:
> On Fri, 20 Nov 2015, Chris J Arges wrote:
>
> [...]
>
> > @@ -272,6 +233,7 @@ static int klp_write_object_relocations(struct module
> > *pmod,
> > struct klp_objec
On Mon, Nov 23, 2015 at 10:52:23AM +0100, Miroslav Benes wrote:
> On Fri, 20 Nov 2015, Chris J Arges wrote:
>
> [...]
>
> > @@ -272,6 +233,7 @@ static int klp_write_object_relocations(struct module
> > *pmod,
> > struct klp_objec
/comments to be more precise
Chris J Arges (3):
livepatch: add old_sympos as disambiguator field to klp_func
livepatch: add sympos as disambiguator field to klp_reloc
livepatch: function,sympos scheme in livepatch sysfs directory
Documentation/ABI/testing/sysfs-kernel-livepatch | 6
ned-off-by: Chris J Arges
---
include/linux/livepatch.h | 5 ++-
kernel/livepatch/core.c | 77 +++
2 files changed, 20 insertions(+), 62 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h
index b60e8ab..a882865 100
disambiguation for relocations is added in the
next patch in this series.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(+), 50 deletions(-)
diff --git
be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git
/comments to be more precise
Chris J Arges (3):
livepatch: add old_sympos as disambiguator field to klp_func
livepatch: add sympos as disambiguator field to klp_reloc
livepatch: function,sympos scheme in livepatch sysfs directory
Documentation/ABI/testing/sysfs-kernel-livepatch | 6
ned-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 5 ++-
kernel/livepatch/core.c | 77 +++
2 files changed, 20 insertions(+), 62 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatc
disambiguation for relocations is added in the
next patch in this series.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(
tching multiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files
On 11/18/2015 10:37 AM, Petr Mladek wrote:
> On Mon 2015-11-16 11:03:06, Chris J Arges wrote:
>> In cases of duplicate symbols, sympos will be used to disambiguate instead
>> of val. By default sympos will be 0, and patching will only succeed if
>> the symbol is unique. Specif
On 11/18/2015 10:37 AM, Petr Mladek wrote:
> On Mon 2015-11-16 11:03:06, Chris J Arges wrote:
>> In cases of duplicate symbols, sympos will be used to disambiguate instead
>> of val. By default sympos will be 0, and patching will only succeed if
>> the symbol is unique. Specif
On 11/16/2015 03:59 PM, Jiri Kosina wrote:
> On Mon, 16 Nov 2015, Chris J Arges wrote:
>
>> In cases of duplicate symbols, old_sympos will be used to disambiguate
>> instead of old_addr. By default old_sympos will be 0, and patching will
>> only succeed if the symb
On 11/16/2015 03:59 PM, Jiri Kosina wrote:
> On Mon, 16 Nov 2015, Chris J Arges wrote:
>
>> In cases of duplicate symbols, old_sympos will be used to disambiguate
>> instead of old_addr. By default old_sympos will be 0, and patching will
>> only succeed if the symb
be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges
---
include/li
patch in this series.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(+), 50 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux
patch in this series.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 19 +++--
kernel/livepatch/core.c | 72 ---
2 files changed, 41 insertions(+), 50 deletions(-)
diff --git a/include
tching multiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges <chris.j.ar...@canonic
be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges
---
include/li
for patching if it is valid.
In addition, make old_addr an internal structure field not to be specified
by the user. Finally, remove klp_find_verify_func_addr as it can be
replaced by klp_find_object_symbol directly.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 19
for patching if it is valid.
In addition, make old_addr an internal structure field not to be specified
by the user. Finally, remove klp_find_verify_func_addr as it can be
replaced by klp_find_object_symbol directly.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include
tching multiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges <chris.j.ar...@canonic
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges
---
include/li
be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git
the search for the static symbols within
the livepatch module. It makes it easier to understand
the meaning of the external flag and klp_find_external_symbol()
function.
Signed-off-by: Petr Mladek
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 3 ++-
kernel/livepatch/core.c | 12
for patching if it is valid.
In addition, make old_addr an internal structure field not to be specified
by the user. Finally, remove klp_find_verify_func_addr as it can be
replaced by klp_find_object_symbol directly.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 20
if it is valid.
Remove klp_verify_callback, klp_verify_args and klp_verify_vmlinux_symbol
as they are no longer used.
>From the klp_reloc structure remove val, as it can be refactored as a
local variable in klp_write_object_relocations.
Signed-off-by: Chris J Arges <chris.j.ar...@canonic
tching multiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files
files.
This patch removes the search for the static symbols within
the livepatch module. It makes it easier to understand
the meaning of the external flag and klp_find_external_symbol()
function.
Signed-off-by: Petr Mladek <pmla...@suse.com>
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.co
for patching if it is valid.
In addition, make old_addr an internal structure field not to be specified
by the user. Finally, remove klp_find_verify_func_addr as it can be
replaced by klp_find_object_symbol directly.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include
On 11/11/2015 10:29 AM, Chris J Arges wrote:
> In cases of duplicate symbols, sympos will be used to disambiguate instead
> of val. By default old_sympos will be 0, and patching will only succeed if
Minor typo. old_sympos, should just be sympos.
--chris
> the symbol is unique. S
-by: Chris J Arges
---
include/linux/livepatch.h | 5 ++--
kernel/livepatch/core.c | 74 ++-
2 files changed, 18 insertions(+), 61 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h
index df7b752..fb968a2 100644
here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/Documentation
. Finally, old_addr is now an internal structure element and not to
be specified by the user.
Signed-off-by: Chris J Arges
---
include/linux/livepatch.h | 20 ++
kernel/livepatch/core.c | 53 +++
2 files changed, 37 insertions(+), 36
On 11/11/2015 10:29 AM, Chris J Arges wrote:
> In cases of duplicate symbols, sympos will be used to disambiguate instead
> of val. By default old_sympos will be 0, and patching will only succeed if
Minor typo. old_sympos, should just be sympos.
--chris
> the symbol is unique. S
. Finally, old_addr is now an internal structure element and not to
be specified by the user.
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 20 ++
kernel/livepatch/core.c | 53 +++
2 files c
ltiple symbols can be found here:
https://github.com/dynup/kpatch/issues/493
Signed-off-by: Chris J Arges <chris.j.ar...@canonical.com>
---
Documentation/ABI/testing/sysfs-kernel-livepatch | 6 +-
kernel/livepatch/core.c | 10 --
2 files changed, 1
-by: Chris J Arges <chris.j.ar...@canonical.com>
---
include/linux/livepatch.h | 5 ++--
kernel/livepatch/core.c | 74 ++-
2 files changed, 18 insertions(+), 61 deletions(-)
diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h
index d
1 - 100 of 284 matches
Mail list logo