On Fri, Nov 27, 2020 at 8:12 PM <t...@redhat.com> wrote: > From: Tom Rix <t...@redhat.com> > > The macro use will already have a semicolon. > > Signed-off-by: Tom Rix <t...@redhat.com> > --- > fs/gfs2/util.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/gfs2/util.h b/fs/gfs2/util.h > index d7562981b3a0..493020393ceb 100644 > --- a/fs/gfs2/util.h > +++ b/fs/gfs2/util.h > @@ -162,7 +162,7 @@ void gfs2_io_error_bh_i(struct gfs2_sbd *sdp, struct > buffer_head *bh, > gfs2_io_error_bh_i((sdp), (bh), __func__, __FILE__, __LINE__, true); > > #define gfs2_io_error_bh(sdp, bh) \ > -gfs2_io_error_bh_i((sdp), (bh), __func__, __FILE__, __LINE__, false); > +gfs2_io_error_bh_i((sdp), (bh), __func__, __FILE__, __LINE__, false) > > > extern struct kmem_cache *gfs2_glock_cachep; > -- > 2.18.4
Yeah okay, there are two more instances of exactly the same pattern further below in the same header file. I'm adding fixes for those as well. Thanks, Andreas
