[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190726164823.GB9224@smile.fi.intel.com>
Date: Fri, 26 Jul 2019 19:48:23 +0300
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Shaohua Li <shli@...nel.org>, linux-raid@...r.kernel.org
Cc: Jens Axboe <axboe@...nel.dk>, Mike Snitzer <snitzer@...hat.com>,
Coly Li <colyli@...e.de>,
Mikulas Patocka <mpatocka@...hat.com>,
Song Liu <songliubraving@...com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] md: Convert to use int_pow()
On Tue, Jul 23, 2019 at 11:41:55PM +0300, Andy Shevchenko wrote:
> Instead of linear approach to calculate power of 10, use generic int_pow()
> which does it better.
I took into Cc drivers/dm guys as they might have known something about md raid
state of affairs. Sorry if I mistakenly added somebody.
Who is doing this?
Should it be orphaned?
(I got a bounce from Shaohua address)
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> ---
> drivers/md/md.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/drivers/md/md.c b/drivers/md/md.c
> index 24638ccedce4..3f1252440ad0 100644
> --- a/drivers/md/md.c
> +++ b/drivers/md/md.c
> @@ -3664,11 +3664,7 @@ int strict_strtoul_scaled(const char *cp, unsigned long *res, int scale)
> return -EINVAL;
> if (decimals < 0)
> decimals = 0;
> - while (decimals < scale) {
> - result *= 10;
> - decimals ++;
> - }
> - *res = result;
> + *res = result * int_pow(10, scale - decimals);
> return 0;
> }
>
> --
> 2.20.1
>
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists