| Message ID | 5889efe35e0b300e69dba97216b1288d9c2428a8.1585233617.git.andreyknvl@google.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | eeb91f9a2e3e9766ae9fd1117bd19d87538f21bf |
| Headers | show |
| Series | kcov: collect coverage from usb soft interrupts | expand |
On Thu, Mar 26, 2020 at 3:44 PM Andrey Konovalov <andreyknvl@google.com> wrote: > > Move t->kcov_sequence assignment before assigning t->kcov_mode > for consistency. > > Signed-off-by: Andrey Konovalov <andreyknvl@google.com> Reviewed-by: Dmitry Vyukov <dvyukov@google.com> > --- > kernel/kcov.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kernel/kcov.c b/kernel/kcov.c > index 888d0a236b04..b985b7a72870 100644 > --- a/kernel/kcov.c > +++ b/kernel/kcov.c > @@ -318,10 +318,10 @@ static void kcov_start(struct task_struct *t, struct kcov *kcov, > /* Cache in task struct for performance. */ > t->kcov_size = size; > t->kcov_area = area; > + t->kcov_sequence = sequence; > /* See comment in check_kcov_mode(). */ > barrier(); > WRITE_ONCE(t->kcov_mode, mode); > - t->kcov_sequence = sequence; > } > > static void kcov_stop(struct task_struct *t) > -- > 2.26.0.rc2.310.g2932bb562d-goog >
diff --git a/kernel/kcov.c b/kernel/kcov.c index 888d0a236b04..b985b7a72870 100644 --- a/kernel/kcov.c +++ b/kernel/kcov.c @@ -318,10 +318,10 @@ static void kcov_start(struct task_struct *t, struct kcov *kcov, /* Cache in task struct for performance. */ t->kcov_size = size; t->kcov_area = area; + t->kcov_sequence = sequence; /* See comment in check_kcov_mode(). */ barrier(); WRITE_ONCE(t->kcov_mode, mode); - t->kcov_sequence = sequence; } static void kcov_stop(struct task_struct *t)
Move t->kcov_sequence assignment before assigning t->kcov_mode for consistency. Signed-off-by: Andrey Konovalov <andreyknvl@google.com> --- kernel/kcov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)