[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] dd: add punchhole feature
From: |
Bernhard Voelker |
Subject: |
Re: [PATCH] dd: add punchhole feature |
Date: |
Tue, 14 Feb 2017 07:57:08 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 |
On 02/14/2017 04:50 AM, Michael Stone wrote:
> On Mon, Feb 13, 2017 at 07:34:04PM -0800, Pádraig Brady wrote:
>> Anyway thanks for the patch.
>> I'm still slightly against merging as it's a guaranteed way
>> to lose data if you ctrl-c the command or whatever.
>> I'll let others weigh in at this point.
>
> Agreed; this makes much more sense as a special purpose tool than yet
> one more incomprensible option crammed into dd.
>From a user's point, stdin respectively the if= argument has always
been read-only and therefore safe from unwanted changes in my mind.
The proposed feature doesn't match into this paradigm.
If added to an existing tool, it would make more sense in
util-linux' fallocate(1) - or even better - a dedicated tool.
Have a nice day,
Berny