From: Heinz Mauelshagen <heinzm@redhat.com>
To: Brassow Jonathan <jbrassow@redhat.com>,
device-mapper development <dm-devel@redhat.com>
Subject: Re: [PATCH v2 3/4] Documentation: add version history entries for raid0 support _and_ forgotten devices_handle_discards_safely module paramater
Date: Fri, 08 May 2015 19:14:21 +0200 [thread overview]
Message-ID: <554CEEED.3010208@redhat.com> (raw)
In-Reply-To: <B561FA32-A0F7-4986-BE99-E85A4D307A56@redhat.com>
On 05/08/2015 06:03 PM, Brassow Jonathan wrote:
>> On Apr 29, 2015, at 7:03 AM, heinzm@redhat.com wrote:
>>
>> From: Heinz Mauelshagen <heinzm@redhat.com>
>>
>> This patch adds version history for md raid0 to dm-raid.txt
>> _and_
>> adds forgotten doc for the 1.6.0 version change as well
>>
>> Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com>
>>
>> ---
>> Documentation/device-mapper/dm-raid.txt | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/Documentation/device-mapper/dm-raid.txt b/Documentation/device-mapper/dm-raid.txt
>> index ef8ba9f..bceb865 100644
>> --- a/Documentation/device-mapper/dm-raid.txt
>> +++ b/Documentation/device-mapper/dm-raid.txt
>> @@ -224,3 +224,6 @@ Version History
>> New status (STATUSTYPE_INFO) fields: sync_action and mismatch_cnt.
>> 1.5.1 Add ability to restore transiently failed devices on resume.
>> 1.5.2 'mismatch_cnt' is zero unless [last_]sync_action is "check".
>> +1.6.0 Add devices_handle_discards_safely module parameter to
>> + allow discards on disks announcing uncertain discard_zeroes_data
>> +1.7.0 Add support for MD RAID0 mappings
>> --
>> 2.1.0
> I was hoping for a new section(e.g. “DISCARDS”) to be added to the doc file for the ‘devices_handle_discards_safely’.
>
> Wasn’t 1.6.0 where discard support was added? If so, 1.6.0 should just be “Add discard support”. The new section should explain all the short-comings and issues with discards + parity RAID.
>
> brassow
Yes, 1.6.0 added discard support for raid1/10 and disabled it for raid4/5/6
with commit upstream 75b8e04bbf01bdd5c42a1d8ac54abf757196ce49
Let's add more docuemtnation with a followup patch then
Heinz
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
next prev parent reply other threads:[~2015-05-08 17:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-29 12:03 [PATCH v2 3/4] Documentation: add version history entries for raid0 support _and_ forgotten devices_handle_discards_safely module paramater heinzm
2015-05-08 16:03 ` Brassow Jonathan
2015-05-08 17:14 ` Heinz Mauelshagen [this message]
-- strict thread matches above, loose matches on Subject: below --
2015-04-29 12:03 heinzm
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=554CEEED.3010208@redhat.com \
--to=heinzm@redhat.com \
--cc=dm-devel@redhat.com \
--cc=jbrassow@redhat.com \
/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.