lkml.org 
[lkml]   [2020]   [Jun]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 39/41] regset: kill ->get()
Date
From: Al Viro <viro@zeniv.linux.org.uk>

no instances left

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---
include/linux/regset.h | 22 ----------------------
kernel/regset.c | 24 +++++-------------------
2 files changed, 5 insertions(+), 41 deletions(-)

diff --git a/include/linux/regset.h b/include/linux/regset.h
index 567a4b63f469..eeabf5cbbbd7 100644
--- a/include/linux/regset.h
+++ b/include/linux/regset.h
@@ -82,27 +82,6 @@ static inline int membuf_write(struct membuf *s, const void *v, size_t size)
typedef int user_regset_active_fn(struct task_struct *target,
const struct user_regset *regset);

-/**
- * user_regset_get_fn - type of @get function in &struct user_regset
- * @target: thread being examined
- * @regset: regset being examined
- * @pos: offset into the regset data to access, in bytes
- * @count: amount of data to copy, in bytes
- * @kbuf: if not %NULL, a kernel-space pointer to copy into
- * @ubuf: if @kbuf is %NULL, a user-space pointer to copy into
- *
- * Fetch register values. Return %0 on success; -%EIO or -%ENODEV
- * are usual failure returns. The @pos and @count values are in
- * bytes, but must be properly aligned. If @kbuf is non-null, that
- * buffer is used and @ubuf is ignored. If @kbuf is %NULL, then
- * ubuf gives a userland pointer to access directly, and an -%EFAULT
- * return value is possible.
- */
-typedef int user_regset_get_fn(struct task_struct *target,
- const struct user_regset *regset,
- unsigned int pos, unsigned int count,
- void *kbuf, void __user *ubuf);
-
typedef int user_regset_get2_fn(struct task_struct *target,
const struct user_regset *regset,
struct membuf to);
@@ -235,7 +214,6 @@ typedef unsigned int user_regset_get_size_fn(struct task_struct *target,
* omitted when there is an @active function and it returns zero.
*/
struct user_regset {
- user_regset_get_fn *get;
user_regset_get2_fn *get2;
user_regset_set_fn *set;
user_regset_active_fn *active;
diff --git a/kernel/regset.c b/kernel/regset.c
index c548fb9f0943..592d90fc13f9 100644
--- a/kernel/regset.c
+++ b/kernel/regset.c
@@ -11,7 +11,7 @@ static int __regset_get(struct task_struct *target,
void *p = *data, *to_free = NULL;
int res;

- if (!regset->get && !regset->get2)
+ if (!regset->get2)
return -EOPNOTSUPP;
if (size > regset->n * regset->size)
size = regset->n * regset->size;
@@ -20,28 +20,14 @@ static int __regset_get(struct task_struct *target,
if (!p)
return -ENOMEM;
}
- if (regset->get2) {
- res = regset->get2(target, regset,
- (struct membuf){.p = p, .left = size});
- if (res < 0) {
- kfree(to_free);
- return res;
- }
- *data = p;
- return size - res;
- }
- res = regset->get(target, regset, 0, size, p, NULL);
- if (unlikely(res < 0)) {
+ res = regset->get2(target, regset,
+ (struct membuf){.p = p, .left = size});
+ if (res < 0) {
kfree(to_free);
return res;
}
*data = p;
- if (regset->get_size) { // arm64-only kludge, will go away
- unsigned max_size = regset->get_size(target, regset);
- if (size > max_size)
- size = max_size;
- }
- return size;
+ return size - res;
}

int regset_get(struct task_struct *target,
--
2.11.0
\
 
 \ /
  Last update: 2020-06-29 23:26    [W:0.261 / U:1.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site