diff options
author | Jeff Layton <jlayton@redhat.com> | 2017-07-24 06:22:15 -0400 |
---|---|---|
committer | Jeff Layton <jlayton@redhat.com> | 2017-07-26 12:24:36 -0400 |
commit | 3acdfd280fe7d807237f2cb7a09d6f8f7f1b484f (patch) | |
tree | 80a7c666a0055b2c21b72736a14cc10d58af2e81 | |
parent | 5771a8c08880cdca3bfb4a3fc6d309d6bba20877 (diff) |
errseq: rename __errseq_set to errseq_set
Nothing calls this wrapper anymore, so just remove it and rename the
old function to get rid of the double underscore prefix.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
-rw-r--r-- | include/linux/errseq.h | 9 | ||||
-rw-r--r-- | lib/errseq.c | 17 | ||||
-rw-r--r-- | mm/filemap.c | 2 |
3 files changed, 9 insertions, 19 deletions
diff --git a/include/linux/errseq.h b/include/linux/errseq.h index 9e0d444ac88d..784f0860527b 100644 --- a/include/linux/errseq.h +++ b/include/linux/errseq.h | |||
@@ -5,14 +5,7 @@ | |||
5 | 5 | ||
6 | typedef u32 errseq_t; | 6 | typedef u32 errseq_t; |
7 | 7 | ||
8 | errseq_t __errseq_set(errseq_t *eseq, int err); | 8 | errseq_t errseq_set(errseq_t *eseq, int err); |
9 | static inline void errseq_set(errseq_t *eseq, int err) | ||
10 | { | ||
11 | /* Optimize for the common case of no error */ | ||
12 | if (unlikely(err)) | ||
13 | __errseq_set(eseq, err); | ||
14 | } | ||
15 | |||
16 | errseq_t errseq_sample(errseq_t *eseq); | 9 | errseq_t errseq_sample(errseq_t *eseq); |
17 | int errseq_check(errseq_t *eseq, errseq_t since); | 10 | int errseq_check(errseq_t *eseq, errseq_t since); |
18 | int errseq_check_and_advance(errseq_t *eseq, errseq_t *since); | 11 | int errseq_check_and_advance(errseq_t *eseq, errseq_t *since); |
diff --git a/lib/errseq.c b/lib/errseq.c index 841fa24e6e00..7b900c2a277a 100644 --- a/lib/errseq.c +++ b/lib/errseq.c | |||
@@ -41,23 +41,20 @@ | |||
41 | #define ERRSEQ_CTR_INC (1 << (ERRSEQ_SHIFT + 1)) | 41 | #define ERRSEQ_CTR_INC (1 << (ERRSEQ_SHIFT + 1)) |
42 | 42 | ||
43 | /** | 43 | /** |
44 | * __errseq_set - set a errseq_t for later reporting | 44 | * errseq_set - set a errseq_t for later reporting |
45 | * @eseq: errseq_t field that should be set | 45 | * @eseq: errseq_t field that should be set |
46 | * @err: error to set | 46 | * @err: error to set (must be between -1 and -MAX_ERRNO) |
47 | * | 47 | * |
48 | * This function sets the error in *eseq, and increments the sequence counter | 48 | * This function sets the error in *eseq, and increments the sequence counter |
49 | * if the last sequence was sampled at some point in the past. | 49 | * if the last sequence was sampled at some point in the past. |
50 | * | 50 | * |
51 | * Any error set will always overwrite an existing error. | 51 | * Any error set will always overwrite an existing error. |
52 | * | 52 | * |
53 | * Most callers will want to use the errseq_set inline wrapper to efficiently | 53 | * We do return the latest value here, primarily for debugging purposes. The |
54 | * handle the common case where err is 0. | 54 | * return value should not be used as a previously sampled value in later calls |
55 | * | 55 | * as it will not have the SEEN flag set. |
56 | * We do return an errseq_t here, primarily for debugging purposes. The return | ||
57 | * value should not be used as a previously sampled value in later calls as it | ||
58 | * will not have the SEEN flag set. | ||
59 | */ | 56 | */ |
60 | errseq_t __errseq_set(errseq_t *eseq, int err) | 57 | errseq_t errseq_set(errseq_t *eseq, int err) |
61 | { | 58 | { |
62 | errseq_t cur, old; | 59 | errseq_t cur, old; |
63 | 60 | ||
@@ -107,7 +104,7 @@ errseq_t __errseq_set(errseq_t *eseq, int err) | |||
107 | } | 104 | } |
108 | return cur; | 105 | return cur; |
109 | } | 106 | } |
110 | EXPORT_SYMBOL(__errseq_set); | 107 | EXPORT_SYMBOL(errseq_set); |
111 | 108 | ||
112 | /** | 109 | /** |
113 | * errseq_sample - grab current errseq_t value | 110 | * errseq_sample - grab current errseq_t value |
diff --git a/mm/filemap.c b/mm/filemap.c index a49702445ce0..e1cca770688f 100644 --- a/mm/filemap.c +++ b/mm/filemap.c | |||
@@ -589,7 +589,7 @@ EXPORT_SYMBOL(filemap_write_and_wait_range); | |||
589 | 589 | ||
590 | void __filemap_set_wb_err(struct address_space *mapping, int err) | 590 | void __filemap_set_wb_err(struct address_space *mapping, int err) |
591 | { | 591 | { |
592 | errseq_t eseq = __errseq_set(&mapping->wb_err, err); | 592 | errseq_t eseq = errseq_set(&mapping->wb_err, err); |
593 | 593 | ||
594 | trace_filemap_set_wb_err(mapping, eseq); | 594 | trace_filemap_set_wb_err(mapping, eseq); |
595 | } | 595 | } |