All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Murphy <lists@colorremedies.com>
To: Btrfs BTRFS <linux-btrfs@vger.kernel.org>
Subject: Re: btrfs on whole disk (no partitions)
Date: Sun, 22 Jun 2014 12:56:42 -0600	[thread overview]
Message-ID: <77EEF3DC-14BF-483A-BEB3-6BA4C17C2FA7@colorremedies.com> (raw)
In-Reply-To: <53A6EC60.3030606@chinilu.com>


On Jun 22, 2014, at 8:46 AM, George Mitchell <george@chinilu.com> wrote:
> 
> http://johannes-bauer.com/linux/wdc/?menuid=3

OK well a post full of hyperbole from an misogynistic jackass doesn't really convince me there's a real problem here. Telling Linux/fdisk/parted that a 4096 byte physical sector drive is 512 byte physical sector doesn't cause any change in behavior regarding alignment. Alignment is done by starting each partition at an LBA that's 8 sector aligned. This has been done for quite a long time on Linux by starting partition 1 at LBA 2048 which is not merely 8 sector aligned but is 1MB aligned. And further by specifying partition sizes in whole MB is both easy and assures alignment for any subsequent partitions. It makes no difference if the drive is 512/512 or 512/4096.

What might matter is if the Linux SCSI layer is optimizing writes to 512/512 drives, where a 4096 byte file system block doesn't need to be completely rewritten, rather only fewer than all 8 sectors. So if the Linux SCSI layer does that on a 4096 byte physical sector drive, it will cause the drive to do RMW. So either someone familiar with the code maybe will speak up how this works, or how to find out indirectly by intercepting commands to the drive.

Chris Murphy


  reply	other threads:[~2014-06-22 18:56 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-18 19:29 btrfs on whole disk (no partitions) Daniel Cegiełka
2014-06-18 20:10 ` Chris Murphy
2014-06-19 11:15   ` Austin S Hemmelgarn
2014-06-18 21:19 ` Imran Geriskovan
2014-06-19  0:07 ` Russell Coker
2014-06-19  8:58   ` Imran Geriskovan
2014-06-19  9:11     ` Imran Geriskovan
2014-06-21 19:19       ` Daniel Cegiełka
2014-06-22  1:36         ` Chris Murphy
2014-06-21 19:12   ` Daniel Cegiełka
2014-06-22  1:34     ` Chris Murphy
2014-06-22  7:49       ` Imran Geriskovan
2014-06-22 13:44         ` George Mitchell
2014-06-22 14:11           ` Roman Mamedov
2014-06-22 14:41             ` George Mitchell
2014-06-22 14:46             ` George Mitchell
2014-06-22 18:56               ` Chris Murphy [this message]
2014-06-22 18:47           ` Chris Murphy
2014-06-23  2:10             ` Duncan
2014-06-23 12:24               ` Martin K. Petersen
2014-06-24  5:37                 ` Duncan
2014-06-25 13:01                 ` Imran Geriskovan
2014-06-25 16:01                   ` Duncan
2014-06-26 18:26                     ` Imran Geriskovan
2014-06-26 18:41                   ` Chris Murphy
2014-06-26 20:46                     ` Imran Geriskovan
2014-06-22 18:31         ` Chris Murphy
2014-06-23 11:34           ` Martin K. Petersen
2014-06-19  1:01 ` George Mitchell
2014-06-19  4:52   ` Russell Coker

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=77EEF3DC-14BF-483A-BEB3-6BA4C17C2FA7@colorremedies.com \
    --to=lists@colorremedies.com \
    --cc=linux-btrfs@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.