linux-kernel - Re: [Drbd-dev] [PATCH 05/30] drbd: Introduce new disk config option rs-discard-granularity

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2101862.7IMGIqiMZK@phil-dell-xps.local>
Date:	Mon, 25 Apr 2016 11:21:17 -0500
From:	Philipp Reisner <philipp.reisner@...bit.com>
To:	Bart Van Assche <bart.vanassche@...disk.com>
Cc:	Jens Axboe <axboe@...com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"drbd-dev@...ts.linbit.com" <drbd-dev@...ts.linbit.com>
Subject: Re: [Drbd-dev] [PATCH 05/30] drbd: Introduce new disk config option rs-discard-granularity

Am Montag, 25. April 2016, 08:35:26 schrieb Bart Van Assche:
> On 04/25/2016 05:10 AM, Philipp Reisner wrote:
> > As long as the value is 0 the feature is disabled. With setting
> > it to a positive value, DRBD limits and aligns its resync requests
> > to the rs-discard-granularity setting. If the sync source detects
> > all zeros in such a block, the resync target discards the range
> > on disk.
> 
> Hello Phil,
> 
> Can you explain why rs-discard-granularity is configurable instead of
> e.g. setting it to the least common multiple of the discard
> granularities of the underlying block devices at both sides?
> 
> Thanks,
> 

Hi Bart,

we had this idea as well. It seems that real world devices like larger
discards better than smaller discards. The other motivation was that
a device mapper logical volume might change it on the fly...
So we think it is best to delegate the decision on the discard chunk
size to user space.

best regards,
 Phil