[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 2/4] raw-format: add bdrv_max_size() support
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [RFC 2/4] raw-format: add bdrv_max_size() support |
Date: |
Tue, 07 Mar 2017 11:32:29 +0100 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Fri 03 Mar 2017 02:51:48 PM CET, Stefan Hajnoczi wrote:
> +static uint64_t raw_max_size(QemuOpts *opts, BlockDriverState *in_bs,
> + Error **errp)
> +{
> + if (in_bs) {
> + int64_t size = bdrv_nb_sectors(in_bs);
> + if (size < 0) {
> + error_setg_errno(errp, -size, "Unable to get image size");
> + return 0;
> + }
> + return (uint64_t)size * BDRV_SECTOR_SIZE;
> + }
Why not use bdrv_getlength() directly? It gives you the size in bytes.
Berto
- [Qemu-devel] [RFC 0/4] qemu-img: add max-size subcommand, Stefan Hajnoczi, 2017/03/03
- [Qemu-devel] [RFC 1/4] block: add bdrv_max_size() API, Stefan Hajnoczi, 2017/03/03
- [Qemu-devel] [RFC 2/4] raw-format: add bdrv_max_size() support, Stefan Hajnoczi, 2017/03/03
- Re: [Qemu-devel] [RFC 2/4] raw-format: add bdrv_max_size() support,
Alberto Garcia <=
- [Qemu-devel] [RFC 3/4] qemu-img: add max-size subcommand, Stefan Hajnoczi, 2017/03/03
- [Qemu-devel] [RFC 4/4] iotests: add test 178 for qemu-img max-size, Stefan Hajnoczi, 2017/03/03
- Re: [Qemu-devel] [RFC 0/4] qemu-img: add max-size subcommand, John Snow, 2017/03/03
- Re: [Qemu-devel] [RFC 0/4] qemu-img: add max-size subcommand, Nir Soffer, 2017/03/03