Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>
To:     Daniel Wagner <dwagner@suse.de>
CC:     "linux-nvme@lists.infradead.org" <linux-nvme@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
        Chaitanya Kulkarni <kch@nvidia.com>,
        Shin'ichiro Kawasaki <shinichiro@fastmail.com>,
        Hannes Reinecke <hare@suse.de>
Subject: Re: [PATCH blktests v3 01/12] nvme/rc: Auto convert test device size
 info
Thread-Topic: [PATCH blktests v3 01/12] nvme/rc: Auto convert test device size
 info
Thread-Index: AQHZgTYxLd3TVfPtPU6X2GSB8ju+gQ==
Date:   Sun, 7 May 2023 22:49:34 +0000
Message-ID: <6bu6fbyvltx74goc6ul5ao6srvcl44wwq5c4v5trk6ioj2nwhj@cbhod6tkucex>
References: <20230503080258.14525-1-dwagner@suse.de>
 <20230503080258.14525-2-dwagner@suse.de>
In-Reply-To: <20230503080258.14525-2-dwagner@suse.de>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?UsoMtsIne/6Q2sHMyGsxIZYlGALcX1lD/Brx+6CBmVJk+8hT1Lf1zr1bVDsS?=
 =?us-ascii?Q?d8WCeXg9aPUL3sbS0NMZETc7Rh3kv3uttch4iZGoL22nV22oZBe7ViF8OHcb?=
 =?us-ascii?Q?VZcJZ9E1y1YcTsNl9FuPm8OsiwUS5/ol0QBOTt2JPHyXMAI1UHNw2ByFwCOU?=
 =?us-ascii?Q?nQvt2PXHcZ/jFf+6U0CJKuTQCfpzlw7mw1szoea/EUAP3jQ2B7E44bgFEJx4?=
 =?us-ascii?Q?hRPNgepSpuTbjh88TPVgVcoEfmXuK24athYNcCNjp6ZUeKm0nLD9fOboZF9A?=
 =?us-ascii?Q?WvJRe9kgBlDXbLDdI9Yzx3KKfQzG02Q3A41MCegXnvzFh1N/zA85rJ32mJ2H?=
 =?us-ascii?Q?dkksNSdb8pfI0kl5YdDVxCh/MdV6JnKCULmX8TsJdY+fRGGR11CtgyGymNZu?=
 =?us-ascii?Q?NbXXdm0F2d5QZVDuxOZMVC6QMsF3B0loYvqKkETBqWKgGFiJuF5fazrBWH70?=
 =?us-ascii?Q?tqA5yK4GJlDa9u0g1ChxqlWurw5b3NaF19M8GEHM5ZdYgcPPIhP0x+8cRkgJ?=
 =?us-ascii?Q?ow1yyQX8iuKBVa3iN8qrR2E0bBZVCOn4GAd7YR5jNK2Xak1BBdLaQgG+j7zc?=
 =?us-ascii?Q?YIPT3ZFd1OAbb+eGzFSzwUKs8b+wVd4DkUczyultYtE48yQRJWWZmJPx5TJt?=
 =?us-ascii?Q?VnS9ZMQBWyA+eJPxoP0cFZJfjoGDfUNtYx/OnNjJw2DqlZJk46/CBM8exPi9?=
 =?us-ascii?Q?YO+Vq2K0Jg+sM/U5qq/xG3C7Y9UDJ1fUsdWTloKGW9Fu9nrqJE+ZSTR54Fzm?=
 =?us-ascii?Q?rCE1mlc7mKbd1nf62d9hdQFr/pnfVJCwBE2eIEHHwoGRjajGj/EzC/rhZxbx?=
 =?us-ascii?Q?bwfxRpgA4JG3e4OrRa9iCMScTOzCTitNszJj1vVjsiP9cVde6y1kzyw0Mahn?=
 =?us-ascii?Q?VCq9eZSyr4hXPdPzFsppzjpiiP4iO3OZkRfaoSJmxC1D0D212k0Wosx3BokT?=
 =?us-ascii?Q?sNUayNGl3GmY+Paue2sWgxvIeAPgI8TMJu3nh4dB+3sVls8CHVASiUeIbw2R?=
 =?us-ascii?Q?eNMGtf4h/i8poyOeWFp32G2b8FgT1F3olZPoHs3R6Al4x/sctERPcAcmz81N?=
 =?us-ascii?Q?KhIVI4biePEUo7F3jyHYHrJJ7NpSAZ94QEAu6X6rdfKn0xvF0/WGNqFUQnpq?=
 =?us-ascii?Q?nc76/kcfbBhExJDi+gXFO9g2EzqzvBfk3lyxY7UMjUPD6TrpSDh7FP2nN0XT?=
 =?us-ascii?Q?5OuE+gDznc5Edg4QInNVFY5PTsMAC0w4cAeg+rZdn3MBo8eO5f9EtJVOYXGo?=
 =?us-ascii?Q?VmmBeuciv6LjHw/VTe3EkA/yMcoMMyCAqABI4yq/FcSsZzzlN/oAflpQ88U0?=
 =?us-ascii?Q?l0vcfhnUCJrKctKJ4soAphv5UpIyVd3f7iZVGgAdLLMseJAeJhB7wzEhCawf?=
 =?us-ascii?Q?pnib98x/HEp4IMJd2V2Nl69wgT1p2qw1eJnIxRHo76ra+VxE20Ym2+8Bizhf?=
 =?us-ascii?Q?md36m+YemuSiwF0WFFRy7pb4w+xIp5HO88wzpdeY/AOCJTl6GwkOBBk1qbKq?=
 =?us-ascii?Q?sZYZ45UPKSkfSe+gCcZOiFtQVXs8sLhOn+eeOmHTDqAs3rukHuCbTJJYmyDI?=
 =?us-ascii?Q?pGkQirkqLklod0sr11B4bwNeAP4G0iGwQpXkqPoo8c339n2sUfa4bjXSqDME?=
 =?us-ascii?Q?7w=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <1CFF6B9743663B429BE30F754B85D894@namprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: m0135vr1Uv0ytM/Mu/xhspoJy/CeoibZZRgfQOVWDseW7Lkc60UY6Nbh2wr9mLqCbyomCdJ9REZaWvi6rK6qYuXdMzEQSW5iqb/2a/ZzcrZ2Gt+Jg6oXk1hrTDBqj6KHPJc5xE2HJtTjciIlmzl+Ul3P/5sV/WqipKw85dRk4Yl3cWNkb+e11dfJElFM/yAq18aUBCQ5NdF0AK1mnJaB5iu+6jOoz5R40Wch/rHX1ZjteQ7pIheh395t4xeFHBOL5yuhxnHX4aLlAw5Nm/+myKhzFIIrVtj2Kbnm5bkY6oLq8E5x4hZVDNmFO+ZSk7MQyKNoTb2MHNbq0PcVreVXZ/zkFBixsGaij/f5+UZhp0GIMzoAIfv+ZePYZl80UogHDBteGQYEzOYR88PeRPNT6DIr3f9wMxFDCLtPF1YqBb9AmJAkqfEPpUuAT43GskY8SQkJv4SUh4t77bH/CV8seGZK9Y8qIpEFLwsz9rY2zmFAscmsfB97tUqhQoFsNVVmZOaK3PK5qJg5c0WDq+9S7NQRhse4oUA4lGfzJOky7J8Xo3k1nIkn4HRMnM3cXTOSDJeNFjlKvpWMys+XPASoT+bKk8cxmltW9wtmzMiAPzxJxtLzrFxAUiod47l9XB3Yy8tX3qblIuHTLS3OuNpyKte1y4wXssVzDoSoQ6/pfRjVuyTlMS8F3sHmx/GdtKOT6dqAzOKEuCJiXkZ2y7TVnDtts0dieHBPCMOwUCU1zHlpNF9xUbqNcZp8cTlrbvtCgxVIE+TVi9ZiYe3z98vaMdHAHbnHCvU1NeeSBFRrgswp2FKE4sqH36yawMesqCY4coBocqMbx2qdabx2UXhZ+3ukPHw6BXyZf2pnWOeEbGzEV0kP3HRSjZW9kTbB7KTJb8Bse+lxQCWtS/O4YJgGABkHPink1f29HU22VnVsR5g=
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM8PR04MB8037.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ee5a1dd9-b6ac-45fb-0943-08db4f4d544e
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 May 2023 22:49:34.8245
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: eBstwYXr4+Soq8Xz18FZVzoExi6/7DSENjy/jwzKmN/KQPtKFbnv0GsBhCbFpctabRg4P/swqP3GikYpxTNMLf8wRZFnGO2ubQqk2pHgWf8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR04MB7561
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On May 03, 2023 / 10:02, Daniel Wagner wrote:
[...]
> +convert_to_mb()
> +{
> +	local str=3D$1
> +	local res
> +
> +	res=3D$(echo "${str}" | sed -n 's/^\([0-9]\+\)$/\1/p')
> +	if [[ -n "${res}" ]]; then
> +		echo "$((res / 1024 / 1024))"
> +	fi
> +
> +	res=3D$(echo "${str}" | sed -n 's/^\([0-9]\+\)[mM]$/\1/p')
> +	if [[ -n "${res}" ]]; then
> +		echo "$((res))"
> +	fi
> +
> +	res=3D$(echo "${str}" | sed -n 's/^\([0-9]\+\)[gG]$/\1/p')
> +	if [[ -n "${res}" ]]; then
> +        	echo "$((res * 1024))"

Here are some spaces still  left.

> +	fi
> +}=