Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Message-ID: <d1d46df7-e761-5086-571f-d35bba2f8e69@oppo.com>
Date:   Tue, 8 Feb 2022 11:03:24 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.4.1
Subject: Re: [f2fs-dev] [PATCH] f2fs-tools: add atomic write related options
 to f2fs_io write command
Content-Language: en-US
To:     Daeho Jeong <daeho43@gmail.com>, linux-kernel@vger.kernel.org,
        linux-f2fs-devel@lists.sourceforge.net, kernel-team@android.com
Cc:     Daeho Jeong <daehojeong@google.com>
References: <20220202201727.3410310-1-daeho43@gmail.com>
From:   Chao Yu <chao.yu@oppo.com>
In-Reply-To: <20220202201727.3410310-1-daeho43@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bDJHNEhqcFhNdmJra2syUGhNbEprQ1hlYm1VR3dCNGR5NE13ZUdKZ0ZyQXlR?=
 =?utf-8?B?QzU5T3NwR0RoclQxMkdmeU1ESUswVm9JSzVVWlpMTmEvRnRPTnFEZ1NGUUhl?=
 =?utf-8?B?V29aNW90TytpQmhDaDlvMWpKS29pK2tJYm1uL2h6dURHNVFRbWdOQkI2TkJJ?=
 =?utf-8?B?TkUwY1BuNFE2WEs3WndZd0cyMWdpM1ErNVdUM05qNjZ6WFBkd0l3WE5EQ2pL?=
 =?utf-8?B?WUxTNE5RNmV0Szk5blRYU0pSRVc5NFVVd3A4bVh2enF6STBLV3dIVFFVZllJ?=
 =?utf-8?B?eStsTGNiS0RESzlMdWQ4bHJmb3ZKTWY1Rm50RXh1TGdTZnNWVGVzY1d2VXFO?=
 =?utf-8?B?MWcwR1ZKbHoyTVFYVG1BelFmbTg5dGxzZ1FUcGh1aFRuYWwwa05XTzJuR2k3?=
 =?utf-8?B?TEtjaXBIaHpjUVNvWU1zNFVyT091Njg5YVZFV2JiWXFYN21xNDhtN3hTVStU?=
 =?utf-8?B?cHZRSnhTV3JPQ1c2OHBqbUdEbU5yeVBNM25uQUgyOC9NeUw1V0tNWVhxT3FN?=
 =?utf-8?B?LzNRTkg1cXpWL3RIaW1Qa3B1ZnZlelZXc1hmWStSVEV4TTlUdU14L0xSRFVt?=
 =?utf-8?B?b01IQUxGZVp0YXVsdmhRRGRSa0pDNzJOS0tmZHFkRE11am5nQnI2YWVEa0tX?=
 =?utf-8?B?eU5QTGh2TlU2b3JLOHIyVDErT20vZWVpcWpFelU0RFFXNlJKamFaS0hQNGFp?=
 =?utf-8?B?Zmd5MGJvYk9ORVYrK2Z2K3lQdStCSTAyU2JrVUNZbTBrZlNxdTYzWVN0cWFl?=
 =?utf-8?B?RmJoMk01ME5pcUhxRHVSbE4wZkxSai8wRHhZdGNrajV1VUU3SFdDUkdOeFVY?=
 =?utf-8?B?aUR6b29Pa3RKaEJEaUtDTnc5Y210djZnakRrdzNRem9tTlJKdUoyUmpUc0Rz?=
 =?utf-8?B?Smh2Ylk2dHl6UzIzcE51anFqNlZucjFPcmU3MjRxTWxBN0xzbW5Sb0FQTkRw?=
 =?utf-8?B?Ykp6a21MVUhWb0VoVGU5Tzhwbk40VDZoRUdORUpnUG8zZHdqV2JaRzk4RUxk?=
 =?utf-8?B?L08vVG1tc1pMQTVUSGh5ZEh1SDJicnE2cFRmamg2R3E1ejJLdEtpTUh0dndK?=
 =?utf-8?B?REZDSEJCQWVOOEFqbWRLOWVwMktUaTBsVkJzSXRQYzBIbVhKZDRxZFE1S1E2?=
 =?utf-8?B?em5MZ3ArS0MyS2Z1SVk4Tm84Um1jSGlFY0lsckZCZ0Y4cmJ0VkFPUHVram80?=
 =?utf-8?B?SUVEbGVEQTR6SUFnRkxGT1EwbnJqWkRLRmE2K3BqNFRpNnY0aFB5c09XcUY5?=
 =?utf-8?B?Z1pPdEUzVTNDdkZsMXdUcjgreGFjVlFPRE84bmh3eUk0azYwS1o1czZmNy91?=
 =?utf-8?B?bUdZdUNkcURCRUhBNzM1WlRIMndmMUVBL2lFYWJHaTVIUnBtWGQwa3V2b216?=
 =?utf-8?B?VzRGczA0QisraFdVakdoR3RFNmV2ZzlNMTZ4Q2o2MDBzYW16Q01IVVdBZDJz?=
 =?utf-8?B?OW5nWG1QY3RCWSs0Zk5JUmNTNmc0a3RVN3N3WGFoVDB4Nm5WdG93MUh4UUxr?=
 =?utf-8?B?OXcrV1hUeFk2aFJXaXpxV2RPbHZ6SDM4eGZnQVRGRHlRY1kzQVhnaWNHY3px?=
 =?utf-8?B?T1B5RFU2dnM2cjFyWmdXTW90d2FQWU5hYVdyZjVHMGxpVnhFd2Q5M3kwZ2hL?=
 =?utf-8?B?K0dGWDRBQk9YaGpvVlo3YzRWUzJFN3g1ZnNreUsvVDJiTlo2ZFA4NDliRWd1?=
 =?utf-8?B?dThpeXJENnJ2V0JML2k4VUsyR0xKU2JqWjFOaXVqN05ZZ1NpWnBhczZFZDY2?=
 =?utf-8?B?NDRnUTYrQzhnMXpyQlJGTmtrQ044aUtnVkRDSWgzaVBIQmZkelNGSUg4UitT?=
 =?utf-8?B?ODBjaTdxb003WnBTZVA3bTUxSzVHUEE3SXUzZGczeVZKU0JWR2RhVHFZNWFo?=
 =?utf-8?B?S3dPLzYwOW81Z3hKanJndjY3RnN6QWczVnBDS1BTNEE4UGlWWnJJMHB1RERx?=
 =?utf-8?B?VW51V0lNbmdTSVhHdExsM3pRNGFBR1V3ZXZlUnpTakdsWVNVaU1oT1pOUFFP?=
 =?utf-8?B?bDVqaGZIeWJESzQvSmFTeEpLQXBtSis2TDIwNHNTTnNhdk5DS0JGYVV4aDNB?=
 =?utf-8?B?c0grOC9pcEtVSnRXSTRaRTFVMmk1UEs2MVpZdVdQVHdHYlVyNnBTRTVMVmlz?=
 =?utf-8?B?bVBVeGwxY3dUbnlRQXl1V0tBWEh1NVR4MkF6bnZXbE45N09IaUkyL3E0Yjdu?=
 =?utf-8?Q?wjbP/B/MEYx3mxgiMfBBtQs=3D?=
X-OriginatorOrg: oppo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 26156fab-6888-45de-4c28-08d9eaaf944e
X-MS-Exchange-CrossTenant-AuthSource: TY2PR02MB3165.apcprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Feb 2022 03:03:28.2190
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: JvQ2fZxvjDrSx8+pJ/y8TahKQoAftTy+ptMs8K1NroZYtAYdnimwbqjObQ5aQgVx45k1fXBP+Eu2cYYKGFS2QQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR02MB5339
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2022/2/3 4:17, Daeho Jeong wrote:
> From: Daeho Jeong <daehojeong@google.com>
> 
> Added two option to f2fs_io "write" command.
>   - atomic_commit : atomic write & commit
>   - atomic_abort  : atomic write & abort
> Also, added an optional option {delay}, which is in ms unit and optional
> only for atomic_commit and atomic_abort.
> 
> Signed-off-by: Daeho Jeong <daehojeong@google.com>

Reviewed-by: Chao Yu <chao@kernel.org>

Thanks,