summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2017-07-24 06:22:15 -0400
committerJeff Layton <jlayton@redhat.com>2017-07-26 12:24:36 -0400
commit3acdfd280fe7d807237f2cb7a09d6f8f7f1b484f (patch)
tree80a7c666a0055b2c21b72736a14cc10d58af2e81
parent5771a8c08880cdca3bfb4a3fc6d309d6bba20877 (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.h9
-rw-r--r--lib/errseq.c17
-rw-r--r--mm/filemap.c2
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
6typedef u32 errseq_t; 6typedef u32 errseq_t;
7 7
8errseq_t __errseq_set(errseq_t *eseq, int err); 8errseq_t errseq_set(errseq_t *eseq, int err);
9static 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
16errseq_t errseq_sample(errseq_t *eseq); 9errseq_t errseq_sample(errseq_t *eseq);
17int errseq_check(errseq_t *eseq, errseq_t since); 10int errseq_check(errseq_t *eseq, errseq_t since);
18int errseq_check_and_advance(errseq_t *eseq, errseq_t *since); 11int 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 */
60errseq_t __errseq_set(errseq_t *eseq, int err) 57errseq_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}
110EXPORT_SYMBOL(__errseq_set); 107EXPORT_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
590void __filemap_set_wb_err(struct address_space *mapping, int err) 590void __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}