Re: [PATCH][next] dm vdo: Fix spelling mistake "postive" -> "positive"
From: Robert P. J. Day
Date: Thu Sep 03 2026 - 16:12:18 EST
On Thu, 3 Sep 2026, Matthew Sakai wrote:
> I had to look at this for quite a while to figure out what was going on.
> We made the same spelling error twice in the same function and somehow only
> noticed the first one previously? Anyway,
>
> Reviewed-by: Matthew Sakai <msakai@xxxxxxxxxx>
>
> On 9/3/26 11:23 AM, Colin Ian King wrote:
> > There is another spelling mistake in a vdo_log_error message that
> > I somehow overlooked last time. Fix it.
> >
> > Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx>
> > ---
> > drivers/md/dm-vdo/dm-vdo-target.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/md/dm-vdo/dm-vdo-target.c
> > b/drivers/md/dm-vdo/dm-vdo-target.c
> > index 1d8375cc3c3e..043549b1fc2e 100644
> > --- a/drivers/md/dm-vdo/dm-vdo-target.c
> > +++ b/drivers/md/dm-vdo/dm-vdo-target.c
> > @@ -443,7 +443,7 @@ static int __must_check parse_slab_size(const char
> > *slab_str, block_count_t *sla
> > result = kstrtoull(slab_str, 10, &value);
> > if (result) {
> > - vdo_log_error("optional parameter error: invalid slab size,
> > must be a postive integer");
> > + vdo_log_error("optional parameter error: invalid slab size,
> > must be a positive integer");
> > return -EINVAL;
> > }
My rule from years back was ... you find one typo, you "grep -rw"
the entire kernel tree.
rday