On Wed, Dec 4, 2024 at 11:05 PM Colin Ian King <[email protected]> wrote: > > A recent change added return 0 before an existing return statement > at the end of function blk_trace_setup. The final return is now > redundant, so remove it. > > Fixes: 64d124798244 ("blktrace: move copy_[to|from]_user() out of > ->debugfs_lock") > > Signed-off-by: Colin Ian King <[email protected]> > --- > kernel/trace/blktrace.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c > index 18c81e6aa496..3679a6d18934 100644 > --- a/kernel/trace/blktrace.c > +++ b/kernel/trace/blktrace.c > @@ -639,8 +639,6 @@ int blk_trace_setup(struct request_queue *q, char *name, > dev_t dev, > return -EFAULT; > } > return 0; > - > - return ret; > } > EXPORT_SYMBOL_GPL(blk_trace_setup);
Reviewed-by: Ming Lei <[email protected]> Thanks,
