From: Christoph Hellwig <hch@lst.de>
To: Rusty Russell <rusty@rustcorp.com.au>
Cc: virtualization@lists.linux-foundation.org,
Avi Kivity <avi@redhat.com>,
kvm@vger.kernel.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH v2] virtio-blk physical block size
Date: Mon, 4 Jan 2010 09:30:35 +0100 [thread overview]
Message-ID: <20100104083035.GA7461@lst.de> (raw)
In-Reply-To: <201001041338.52621.rusty@rustcorp.com.au>
On Mon, Jan 04, 2010 at 01:38:51PM +1030, Rusty Russell wrote:
> I thought this was what I was doing, but I have shown over and over that
> I have no idea about block devices.
>
> Our current driver treats BLK_SIZE as the logical and physical size (see
> blk_queue_logical_block_size).
>
> I have no idea what "logical" vs. "physical" actually means. Anyone? Most
> importantly, is it some Linux-internal difference or a real I/O-visible
> distinction?
Those should be the same for any sane interface. They are for classical
disk devices with larger block sizes (MO, s390 dasd) and also for the
now appearing 4k sector scsi disks. But in the ide world people are
concerned about dos/window legacy compatiblity so they came up with a
nasty hack:
- there is a physical block size as used by the disk internally
(4k initially)
- all the interfaces to the operating system still happen in the
traditional 512 byte blocks to not break any existing assumptions
- to make sure modern operating systems can optimize for the larger
physical sectors the disks expose this size, too.
- even worse disks can also have alignment hacks for the traditional
DOS partitions tables, so that the 512 byte block zero might even
have an offset into the first larger physical block. This is also
exposed in the ATA identify information.
All in all I don't think this mess is a good idea to replicate in
virtio. Virtio by defintion requires virtualization aware guests, so we
should just follow the SCSI way of larger real block sizes here.
>
> Rusty.
>
---end quoted text---
next prev parent reply other threads:[~2010-01-04 8:30 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1262018363-15871-1-git-send-email-avi@redhat.com>
2010-01-04 3:08 ` [Qemu-devel] Re: [PATCH v2] virtio-blk physical block size Rusty Russell
2010-01-04 7:02 ` Avi Kivity
2010-01-05 20:18 ` Jamie Lokier
2010-01-04 8:30 ` Christoph Hellwig [this message]
2010-01-05 12:56 ` Rusty Russell
2010-01-05 12:58 ` Avi Kivity
2010-01-05 20:16 ` Jamie Lokier
2010-01-08 15:40 ` Christoph Hellwig
2010-01-10 12:35 ` Avi Kivity
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100104083035.GA7461@lst.de \
--to=hch@lst.de \
--cc=avi@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=qemu-devel@nongnu.org \
--cc=rusty@rustcorp.com.au \
--cc=virtualization@lists.linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).