Received-SPF: pass (google.com: domain of linux-kernel+bounces-51409-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <8c51d0d8-ae3d-418c-b6ac-5a164b40a8c5@oppo.com>
Date: Sun, 4 Feb 2024 11:35:45 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v5] f2fs: fix zoned block device information
 initialization
Content-Language: en-US
To: Wenjie Qi <qwjhust@gmail.com>, jaegeuk@kernel.org, chao@kernel.org,
 linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Cc: hustqwj@hust.edu.cn
References: <20240204031022.1189-1-qwjhust@gmail.com>
From: Yongpeng Yang <yangyongpeng1@oppo.com>
In-Reply-To: <20240204031022.1189-1-qwjhust@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?UVZOaFlVdjRDVUliOUJTNmVWTW1iYVJMb21JYlQ4MFprSFphYjJLYVhyWEJI?=
 =?utf-8?B?VTNTMHhVVW5peEJHdEE5ZjQ0b05Bc0lsTktnNVFQaXRCbndnUFUyVmt0QlJY?=
 =?utf-8?B?WkdkT2FSOGJ0eXdGeDhiMFprZzEvUExaTVIvZUJYN001WERWZzJWdWRDZjlw?=
 =?utf-8?B?OUdJc3g0VkNPZWN0VmI1QkcyTXR6aHZsL2xhY3Z0UkY0Nmh1TThXZXpsbHlr?=
 =?utf-8?B?YmVmcGRiR3M5dzJWTDM4QkhUWG1XYlB6UGphaDlnbU9kYUJaMVN4bmd1MHlm?=
 =?utf-8?B?dlhncVVYUlQ5ckt0RjBTQ0UwV2pqV04vWnUxc3U1bHdkYjNqbjk3ZGloRmlM?=
 =?utf-8?B?Z3Frd2E1WmhYcmx1d0IxSFpZSnc0bzVrSkEvWTJvRUNGbWFsWFR3VHBHUk45?=
 =?utf-8?B?clM1N01xdVlBNk9heG9kWk9SMXJueGJvZFBoejhjeE42Znpla0ZZdjk5Z09N?=
 =?utf-8?B?dEVQSURib0VOY0FabkVJWHBzZmxQcGJEN2F1M0hrcjZGR0ZoamVCcHFYelpj?=
 =?utf-8?B?dXhZNXN1OVVoMnE3MUoxRGtBZDkxYVpwWGlOWFZHc09sa0QvTHhBa0VHTGlj?=
 =?utf-8?B?UFd3WjhYOWNMWTdVZTdjSlhET01lcmJXVjdHU1pnR05ad0VKNnB3NlRON3Ix?=
 =?utf-8?B?TXpiTmoyNmV5S3JpczFQWVRVa25iQ0thSU1wSmJ6T2dqOVZOY0NRajRzZ0ZP?=
 =?utf-8?B?eXN2MzlpTW9FbU4rYURzWVI5QzhaM3ZVRXpReDQ4MlR5aDc4MS9CNGZtVU9T?=
 =?utf-8?B?WFB4c1JTdlZDaU9MeTUzelB0Z2FsekdPeFNvL1A2TlhQWmRrT2pCWG9NWndz?=
 =?utf-8?B?a2ljWGRveTNEZ0JGT2JtalNnWlh0MzNMWmhDeG9Cd3F5U3NmQVRudlNVWWJJ?=
 =?utf-8?B?d1J3am9aMGdzV0VOVGN1ZkE2d2F5Smd2ZmlTM3B0ZVR1bFpuUlkwQ1lVeE56?=
 =?utf-8?B?M1krSC9hRUQ5TGVydGw4MVAzT0czT1Z2M0RnZzE5NWRETmw5Y09zald1dFJB?=
 =?utf-8?B?WkJ0KzVCY3dIbk1VSnVYZVJMVmE0ZUloZFRWSUlabStvSVNvRUwyUW12R2ZS?=
 =?utf-8?B?WUxmbWlrdmcrSThOWXZ1TnRvcWJIVlNXYi9qQmhRZWtYZGphdmovSkk3Yzdk?=
 =?utf-8?B?UnRrbDJCRmZyL1dlLzExVHhiZDQzdmJQNXJtOWk4L2UwRVNkdGEvdnVrbG83?=
 =?utf-8?B?TFNpNmtWSlJpVUkyUU5xcG85M2hwWm9ZaUJ4V0RFSXlUeGwzZ3ZvSGxab0c0?=
 =?utf-8?B?c0l5dXdJdmtpb0J5WFkyeEFUMWtuTytiblVxK3NxMFI0ZVh1YkpMY3haNy91?=
 =?utf-8?B?VUxrSGVlUEkwUHJwZW9QYjVwZ3dxRFFsMmwwM1k3YkQvcEFnY3hsUXFDRjFu?=
 =?utf-8?B?bFJCcytRWEhjVWE5c25aVXdyMENrRndsSXdKcVU4aUY4TS81UXZLRDBCWHZE?=
 =?utf-8?B?MmM2ZHN5RkJwV0Nla0Q4Q21YRmZWZlpKQlJyMW53aHVYZ0QxNGJtYUNSR2VS?=
 =?utf-8?B?OVEzdFc5dzBUSWxPbUVNWUpodVNPNGlNY3ZuSXpzRVZYOXVUVndNODRIMXlq?=
 =?utf-8?B?V3FUYkpVdzE2dlZVT1Rta0FnRDVhKzFzcjhoaHhTNkthT1lFZjBCOHVrcUcr?=
 =?utf-8?B?VGloMUxIYkVPeTd6dkgxZklhVzA0U0U2ZS91SzF1USs5UTQxMERaK1hQQ3BN?=
 =?utf-8?B?b2k1YWxQMnBGTVBXQ1YzdFVSVld6N3YwVTlyc1h6eUwwejBwMlR3U1hKVDd0?=
 =?utf-8?B?SUdDT3QwOU01RldiOUNlQ3VBenVjVkx6WVZBczBDZVM0V20zbkozaUp1elE2?=
 =?utf-8?B?YStTNTBra2JqNjBlKy9QOUtsa0Eyek8vem41dzcxVUI5aTJoMW1GdEFpR2pJ?=
 =?utf-8?B?T1hYSW5nK3p3NTlRaVFhUlIyU3ZwTWczTHloQTlyVXJTWVZPd2JrZXQ0blV0?=
 =?utf-8?B?QlpCT3pOQ25QdllYN1JBbTdveWN4NENHclJmUDN4eWs0SVE1bXByZ3hNVEhn?=
 =?utf-8?B?L2dwSEgrQW5nRzNpZVdVOW1hQWJNd0h2bFE2bmpiZy8vU0JBdDFQSmpLZTF0?=
 =?utf-8?B?SGlNREd0aDY1Yk9Ga1dRdUIwN1I4V1FWem1KWi96eWJia2Z5Q29uZ1lIYjBG?=
 =?utf-8?B?cFJzV3QxYnd5cXduTzVoV2hhMG8zTGprQkcxK1k1NHlPUnUva3d6aEJVTXlZ?=
 =?utf-8?B?YUE9PQ==?=
X-OriginatorOrg: oppo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c84e6913-cef3-4073-9593-08dc2532607a
X-MS-Exchange-CrossTenant-AuthSource: SI2PR02MB4730.apcprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Feb 2024 03:35:47.7945
 (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: U/3N5BiKXN4Qj3BOYUaHWi7QmLQH00GlTnESrlOa1Gh96hFOj9gI8YDQkexSHkw+WARDcRq0st4ZSfS/9u/bkg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR02MB7411

Hi Wenjie,
"max_open_zones < sbi->max_open_zones" seems meaningless, this condition 
is false only if "max_open_zones == UINT_MAX"(rarely happen). Although 
it doesn't affect the correctness, I think less jmp instruction is better.

On 2/4/2024 11:10 AM, Wenjie Qi wrote:
> max_open_zones < sbi->max_open_zones