Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d156c532-3531-c1ca-7d9f-fd77df47242a@oppo.com>
Date:   Thu, 1 Jun 2023 22:37:24 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.0
Subject: Re: [f2fs-dev] [PATCH] f2fs: add f2fs_ioc_[get|set]_extra_attr
Content-Language: en-US
To:     Eric Biggers <ebiggers@kernel.org>
Cc:     jaegeuk@kernel.org, chao@kernel.org, linux-kernel@vger.kernel.org,
        linux-f2fs-devel@lists.sourceforge.net
References: <20230529013502.2230810-1-shengyong@oppo.com>
 <20230601031658.GA728@quark.localdomain>
From:   Sheng Yong <shengyong@oppo.com>
In-Reply-To: <20230601031658.GA728@quark.localdomain>
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?SkV2Tjg0NXVrT253Vy8vam54bXcrOXcvVVZhWW5pSjFUL0ZuZTRNU0JULyt3?=
 =?utf-8?B?TDR2TzJybHlIbHcwUXYwOGZOSDhXWlNkQUo5Wjh3K1V6OUdaZnVoTDdTUXpq?=
 =?utf-8?B?QUtPMWd0RXYzc0s1d0dFRVl4ZkpCUlR0ZmdHSG1EQ0RFZ2FrVWh0amovYk8x?=
 =?utf-8?B?Z3dCbk8weEdjeUtDYU9Sb09vbTZjcFY1UkNnTk9YUVFMU2habXpTd3huaExN?=
 =?utf-8?B?Rk9aNHlFZGI2bE1jY2pucHJRWmRyRXhpcjNlYUJOeUxoNWpRbGllOWI2NStJ?=
 =?utf-8?B?UmcweWJ2MkFGeGNzNW41K1pmZk5lTzVnNFpLV0NJM1lsbS9pRFBKM0VLUm4w?=
 =?utf-8?B?S0JUYXloRmpaSC9CWVdFV1R0VERQOGl3TXNzVmh0T2I0am9YT0hFT1FSZmt3?=
 =?utf-8?B?Mm9Ub29ZYU43TlIyaXRZZFM4cG13ckJETGVuTU44UnhDekxsTE4rS1pia1Bn?=
 =?utf-8?B?ak4vMkdyRmduS2dQTm9kVmJLQTlCTGg3T0JrUFNHcDZ6M1ZsMUZoakpRdHpZ?=
 =?utf-8?B?QW5hbEhGbVRZcEFGYUx5SmJnYklod3RjdmhwSlVLbWZmTU9RWW92ZjVtT1h2?=
 =?utf-8?B?UGdNekJwTE9UU1hndHlJdDdBV1YzSFg2dEJETldYS3BrTUNRWnRoRGxtRTZp?=
 =?utf-8?B?d2JDRXFnaUxmZmw5L25kVWppdHhFMS9udi9oTTNOMmdhSXgrcFA3WWR4Nm5p?=
 =?utf-8?B?NGF2WXdIM2hURWtoaVZVM2NoNUFVV1MvemJuWFRaeHJ1Vm0rRjgxVWxFcUJ4?=
 =?utf-8?B?RlMraGhZTGQ1aU1HcTk0d3E0RVI3NlBNcDlhVjBPOTRjOVo3dGhKV0NCQmFT?=
 =?utf-8?B?bytJZVNMOXd5cUhzQ2Q3NElUdGljUW16TkJ3M243K1dVaXFUQ3BoWjVSWE9F?=
 =?utf-8?B?THNsZHdXa1M2YVIxY0xpV1YvbUxVMXNsekRtZmwvQzBOdlVVc3YrSThBdzJJ?=
 =?utf-8?B?OU5WMkwxQklqQ2RDSjIxeWs0T1Vpbk9Nb2RzTllmRDRZSXFQbzdGekVCTmNV?=
 =?utf-8?B?TTllSm1wRjkwT2pNV2E5MVNsR3ZYVFVWVU9FSmRTazArU3FQd1A1Z1NEZm5R?=
 =?utf-8?B?WFNUZHhORXVzMDE3SUFVNHZ6aGNUR0V3LzZvczNBVlZaV1hQRllnOXNESUdx?=
 =?utf-8?B?UHNjUTFtSVdOZldEVlpxN2JaZ0lnQ3VCeE1JYzQvb3h2OFdQQUNEQ2w3TGFr?=
 =?utf-8?B?R2t1YnFSRENvSkd3bStYSXNuaEpObERjUmZlcnY4M3RtcGpEcVNLMDhBZitJ?=
 =?utf-8?B?OEpWZXJQSHY3OHhoUU12S3JvMTFtTHpwQUt1cVpjZGZ3Tmp1UVdBSVc3aUM5?=
 =?utf-8?B?eEk1SmFRRHZLY1R4aGxnYUtIV3RZNFpyNDg0MmJpRnhoUnRjMnVBbndOQWRK?=
 =?utf-8?B?THZEaGJmbXFYUEZwTWVQcDJJcDZoRnBLNXRYODRIL084VkRlVzZyMVlGK3l4?=
 =?utf-8?B?c0pTWGpoRjZEOExBd24rcWZmNmNTYU9GUjlwb1NhbXoxWVQxWk5FTzlrb3RZ?=
 =?utf-8?B?bHhvUURrVFdPM3dFYnNuM21LY2FJTkdEUnZobW5EQWJnYXR4S1Z4UDlEWXAy?=
 =?utf-8?B?cWUrZDkwaTg5di9TQmM4akRaUlZvK2V4eXNlT0RnUkVvdE9ycFJzbUR5c29r?=
 =?utf-8?B?WFlyem9rblZjMEM0MGIrQWJZR0xjb2VJM1B4ZDliSlVVSk1KYkNXczhCUEpz?=
 =?utf-8?B?T0VVSitsQmU2RzZlL0JUbFJFak5DUFluL2xQV1dzTTZZNHdKVVdrUUdQeFNL?=
 =?utf-8?B?SCt2azBDTHFzb01DMjRoMHdhSjVOU2p0SmpSM1Y2RzJ1WVMvZWdwNDhXd3J1?=
 =?utf-8?B?V2RXalNzMFVQMTdZNkhneG1GZzZoR3JNbmpNQ2JmQUY0OTFqSjFxM241Tzdi?=
 =?utf-8?B?WTFPaG9zeEErR1M0ZWlJUzI2MHJvMXU3QkU3bisxclp0TFAxZmJUN2srK3dz?=
 =?utf-8?B?bE5VQlB6T3ZjMVVLcy9pUlA1VGdzNjBnVnI0MjM5SDZKckZCb0krSUxFWTBJ?=
 =?utf-8?B?THZ5RkZVWGRidGVyeFNSQnBhM3RqRDZqbHNpc011S3hRSk5sTmQzLytPOGpF?=
 =?utf-8?B?a05EUlVLdWZoV1ltczZNTzhwMFdjTlFwZ3NlT1p0WjdKZUZYMm9rZER1OFJy?=
 =?utf-8?Q?vSFA8kWdb0ajBdQHddBvww0dJ?=
X-OriginatorOrg: oppo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f03563c0-ea3a-4d08-7f30-08db62adb8cc
X-MS-Exchange-CrossTenant-AuthSource: SI2PR02MB5148.apcprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Jun 2023 14:37:27.3301
 (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: +HcCrOxbCvnGvHMUJUGJlIeD5oDKvVGZU62yo/Nu4n7zUNGEY2g2Ev6tyDsRydepRI2YuBPKyfm47GeAGIl4/g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR02MB5988
X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham 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 2023/6/1 11:16, Eric Biggers wrote:
> On Mon, May 29, 2023 at 09:35:00AM +0800, Sheng Yong via Linux-f2fs-devel wrote:
>> This patch introduces two ioctls:
>>    * f2fs_ioc_get_extra_attr
>>    * f2fs_ioc_set_extra_attr
>> to get or modify values in extra attribute area.
>>
>> The argument of these two ioctls is `struct f2fs_extra_attr', which has
>> three members:
>>    * field: indicates which field in extra attribute area is handled
>>    * attr: value or userspace pointer
>>    * attr_size: size of `attr'
>>
>> The `field' member could help extend functionality of these two ioctls
>> without modify or add new interfaces, if more fields are added into
>> extra attributes ares in the feture.
>>
>> Signed-off-by: Sheng Yong <shengyong@oppo.com>
> 
> Aren't there enough things called extra or extended attributes already?  Besides
> the standard "extended attributes" retrievable with the getxattr() system call,
> there is already the FS_IOC_FSGETXATTR ioctl too.
Hi, Eric,

The name extra_attr is a bit confusing :-(
But f2fs usually extends new features through extra_attr, like compression, inode
chksum. Since this area is easy to extend, new features could also be added here
in the future. To avoid adding new ioctls for new features, these two could help
integrate with all extra_attr related ioctls. And it seems not appropriate to add
new fields or get/set f2fs-only attributes by FS_IOC_FSGETXATTR/FS_IOC_FSGETXATTR.
xattrs could not be accessed through these two, only inline xattr size is allowed
to be modified for an empty file.

thanks,
shengyong

> 
> - Eric