From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51646) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ayfVC-0006xB-Jb for qemu-devel@nongnu.org; Fri, 06 May 2016 09:09:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ayfV0-0006kK-UR for qemu-devel@nongnu.org; Fri, 06 May 2016 09:09:01 -0400 Date: Fri, 6 May 2016 15:08:04 +0200 From: Kevin Wolf Message-ID: <20160506130804.GG5093@noname.redhat.com> References: <1462406126-22946-1-git-send-email-eblake@redhat.com> <1462406126-22946-18-git-send-email-eblake@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1462406126-22946-18-git-send-email-eblake@redhat.com> Subject: Re: [Qemu-devel] [PATCH v6 17/20] nbd: Switch to byte-based block access List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-devel@nongnu.org, qemu-block@nongnu.org, Paolo Bonzini Am 05.05.2016 um 01:55 hat Eric Blake geschrieben: > Sector-based blk_read() should die; switch to byte-based > blk_pread() instead. > > Signed-off-by: Eric Blake > --- > qemu-nbd.c | 11 +++++++---- > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/qemu-nbd.c b/qemu-nbd.c > index c55b40f..c07ceef 100644 > --- a/qemu-nbd.c > +++ b/qemu-nbd.c > @@ -159,12 +159,13 @@ static int find_partition(BlockBackend *blk, int partition, > off_t *offset, off_t *size) > { > struct partition_record mbr[4]; > - uint8_t data[512]; > + uint8_t data[BDRV_SECTOR_SIZE]; I like 512 better, actually. This is not the size of the arbitrary unit that some block layer functions use, but the size of an MBR. If you don't like the magic number, a new #define would probably be best. > int i; > int ext_partnum = 4; > int ret; > > - if ((ret = blk_read(blk, 0, data, 1)) < 0) { > + ret = blk_pread(blk, 0, data, sizeof(data)); > + if (ret < 0) { > error_report("error while reading: %s", strerror(-ret)); > exit(EXIT_FAILURE); > } > @@ -182,10 +183,12 @@ static int find_partition(BlockBackend *blk, int partition, > > if (mbr[i].system == 0xF || mbr[i].system == 0x5) { > struct partition_record ext[4]; > - uint8_t data1[512]; > + uint8_t data1[BDRV_SECTOR_SIZE]; Same here. > int j; > > - if ((ret = blk_read(blk, mbr[i].start_sector_abs, data1, 1)) < 0) { > + ret = blk_pread(blk, mbr[i].start_sector_abs << BDRV_SECTOR_BITS, > + data1, sizeof(data1)); > + if (ret < 0) { > error_report("error while reading: %s", strerror(-ret)); > exit(EXIT_FAILURE); > } Kevin