From patchwork Sat May 7 16:31:11 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: miter X-Patchwork-Id: 1627934 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: bilbo.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=outlook.com header.i=@outlook.com header.a=rsa-sha256 header.s=selector1 header.b=PS21COoW; dkim-atps=neutral Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org (client-ip=2605:bc80:3010::138; helo=smtp1.osuosl.org; envelope-from=ovs-dev-bounces@openvswitch.org; receiver=) Received: from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by bilbo.ozlabs.org (Postfix) with ESMTPS id 4KwXxk1657z9sG2 for ; Sun, 8 May 2022 02:31:22 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 5B52282951; Sat, 7 May 2022 16:31:20 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id oj2S5NZayGuL; Sat, 7 May 2022 16:31:19 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [IPv6:2605:bc80:3010:104::8cd3:938]) by smtp1.osuosl.org (Postfix) with ESMTPS id 8A085828DA; Sat, 7 May 2022 16:31:18 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 6D7BFC007A; Sat, 7 May 2022 16:31:18 +0000 (UTC) X-Original-To: dev@openvswitch.org Delivered-To: ovs-dev@lists.linuxfoundation.org Received: from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133]) by lists.linuxfoundation.org (Postfix) with ESMTP id 87D6AC002D for ; Sat, 7 May 2022 16:31:17 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 76A5540603 for ; Sat, 7 May 2022 16:31:17 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Authentication-Results: smtp2.osuosl.org (amavisd-new); dkim=pass (2048-bit key) header.d=outlook.com Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id K2k3orkJ0cfh for ; Sat, 7 May 2022 16:31:16 +0000 (UTC) X-Greylist: whitelisted by SQLgrey-1.8.0 Received: from AUS01-SY4-obe.outbound.protection.outlook.com (mail-sy4aus01olkn20826.outbound.protection.outlook.com [IPv6:2a01:111:f403:7005::826]) by smtp2.osuosl.org (Postfix) with ESMTPS id 555FC405F4 for ; Sat, 7 May 2022 16:31:16 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Npkp2h1ArbgD7RljDZfuUbB547AlBGH0N4kQ+IVGtXmIAcJel00xswLF2DbQs04xUD4F8+OqN4A1tzp70F5iFhvAgpHTu1GiQ66jWFSDf6vkFrUxRlGh9CajRCnqKq8i25Ojm+7eoAwvVZpY4HzqoQhwdkENFoolGrTg0G5W4OB1yQHfKyZJsvEhd0ZrhhVr9oy258Md1/dcP2tD6hjP0SxiPz8BmGb5Jio2aMI4SG9kE051hQ0AWv4imDt2o69WswOePYMofU9HNA7c6TePkzcyo3gGZ/z+i5abvspCi4WqQPLtLd9sCkXO6sXVyJj64O/t8aZ7g4kTUYth6Mbu9w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QAm3CCcbCa/AyO5gnfO4Ba9c5D5oS4LGNOycqQu8MZU=; b=bfdBLpG/ljCVtbuv8+g5zniyHzMnxRdKX5wXnT58Vne491jOt4E9hPXtzMVZeYcmBdThr44Xurkd0sjmFdA5t7M4Pl0JyJFn1ajXdsoGX9vg0Yk742/4SXcWXZjvvYxscCHUmTPnmT4WhzBz6JsCcEUWR7+8I9fiRuT+jGtOle4wJyy2OT4EziPHZPLGO5NZzySlLZIkDegwBw6rTBZaIGCNlwVmdLzjd8S2UOiAt1kLHnID3dI0aK7UqnEPpPmeBfF6GWfgaJldya0yCwki8GEjorvp9VuFn9fdBf6r26KDyQhC+0n+3iqpJJiE3lfGxpPdQgghNYQL5CCWwUkrJw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QAm3CCcbCa/AyO5gnfO4Ba9c5D5oS4LGNOycqQu8MZU=; b=PS21COoWOuzMSZIR22ck+DvrW+awmZAqDYGB6GVlF9nsNKLE/K1KYBJL4faeMu7GYmjjNaDh7KNXxqYQJSh8jl4bhdheqAmEFUhscp0O8JBYDZzrKLjlRx566VMp4xbHuamjgAiGFeEVL74QmiY7TRtSqh0YOkUhxheef0rqfTQfopYxLo4jnTNs40e3socIMJKbdFVjccaf/kTWf/bYIbrnENCuRUPITmRInNEdFdoVwHYQgVc+ZDsdGCXmPF1rNqA5Jjr2Z31eSfULwvT1xo+Gd3V4GXbd5XrUwusdl9ZgFY9DcZLlo7aAAtq2vd7HoNRr+2tHGxgkeolh8kHhkg== Received: from MEYP282MB3302.AUSP282.PROD.OUTLOOK.COM (2603:10c6:220:15b::24) by SY4P282MB3584.AUSP282.PROD.OUTLOOK.COM (2603:10c6:10:17f::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5227.20; Sat, 7 May 2022 16:31:11 +0000 Received: from MEYP282MB3302.AUSP282.PROD.OUTLOOK.COM ([fe80::f1b0:b062:c467:b9a]) by MEYP282MB3302.AUSP282.PROD.OUTLOOK.COM ([fe80::f1b0:b062:c467:b9a%3]) with mapi id 15.20.5227.022; Sat, 7 May 2022 16:31:11 +0000 From: lin huang To: "dev@openvswitch.org" Thread-Topic: [PATCH 2/2] dpif-netdev : Fix ALB 'rebalance_intvl' max hard limit. Thread-Index: AdhiL6QS/QziVyZOQfuUxIKvojdIjA== Date: Sat, 7 May 2022 16:31:11 +0000 Message-ID: Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [SPXnGyLBAJRVfI2MboefakqUS/b3C6NO] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 5d18a6cb-2412-4231-57f2-08da3046ff2b x-ms-traffictypediagnostic: SY4P282MB3584:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 40idoyLt9P0UzJnpeic4NwjU7zCnmGAqWwwIE3cDqD9X+KP7pK6RrW7ln/msUpDlTfcD1ywGdY5gmZbqaLTWYj19vNA0RoDkPEkQ5f7iRi04C4WY/uzAY+nFrPWgkYE0M9kh2W1NNWKkQBU7PuUK52ahIlWZ+CWpDWVKigpQLmc8JdIsiJXhaSjQOfx7Mj2puTHIjupUXqg5yd2z+gbeTACg7GuyPcsId8EAjTn1BOqWx/kSdirPJZIKnnwgxq2x+R6Fom2Y5NVSKRZFn31TauAJE1lfI5JsHHPAaL+KIuWWeqpSTsnVTTLh5ogk58SSY8BvpWwHtseWprMpTmtqrDu7Nsphv5MRisbSvUUjXQnLnNWkX9EdlevgMTJgzb7vdx2Nfp7VF6VoBgNFZn7596haW/XIqnkoDFHI52WsgrDVxj5TJcXPhaWPdfMfv57rTfBz7gAnCb8zL8v9GZPsW4CGbx4HkJSw+d7y0hdiWvLP2oU7n3gjndnHwyxpxFMiBKQnRZNk2OvNtKsrhIWHhmLG+3J2yfUPCqhQ73iGrBsbjVr7+WDA6thZCDbocKFM5Xp4CqbU4sdadDRlX/0Ix4bYKkmpwAH2Oo4DzEdfkuhNrJ2rB5FVub/S1tPb42Xd x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?eucgb2312_cn?b?MnFDaS9La1BPZjErVXRBSzB6?= =?eucgb2312_cn?b?UkYyZnNXb0MzOStaSWFxL3E4aDFmWlFNd09qdXk1VnBJam1GMTdIRGlZSjVLYm5B?= =?eucgb2312_cn?b?OW1qODhieWFXaU9YbGk1SEZpVU1qVXFvMHFTaUhKR09LQXllQXlzUGs5Smh2SXRO?= =?eucgb2312_cn?b?dFRaREhKcVFXbEJ6T0phbHJOdDVOejZmNzR4MnR0aEZjaE5pL0ZVMjRGSXE2bGFy?= =?eucgb2312_cn?b?TSs4VllYRWZoZXlIUHhoRVJteFZTbFIrMSttRC9QdW5WQkVwaC9rU2lqZWdNQ0pw?= =?eucgb2312_cn?b?Z3NIeUVyUVVJS0FaaC81WnFDRW1EOU9DN0llNzlxR3JVOFdUVEJaSEQ2TWljSVZk?= =?eucgb2312_cn?b?MWR2SUN6Vjd0S3hHMThTRmgzUkRPajJVVnJvYXNNRllXdExJVnBJRW8rOTZDNjVP?= =?eucgb2312_cn?b?NUVVVmRMRkZ1T29GNkFZK3h4MktGaEFRZUpyUGo0YnM5S1psZm9lMm9vSWc5aDEv?= =?eucgb2312_cn?b?dlBrK3NORW9JUDQwYnRJWlFmcnR5aHZReHdCZS9UaFlFdlUzU1BBNy8zS2JJS21v?= =?eucgb2312_cn?b?NDFIYkZCMEVScWhRNllJM2tmeFhjdHFGRkhPS0RGUGZMMWZkc3dVdE1aN1BxaGQ2?= =?eucgb2312_cn?b?RVI5NUtEY1FCMldLc0grWFhCb0IvamZiVXhWUkpIKys2OVpwZmlrTW1lWUZzZFJS?= =?eucgb2312_cn?b?TmlqV2Y0M3lseGJSUTdTeTVvMlNBNy9kTzMxZUJWemFRMlRWNVRDclYxSWhGbVlh?= =?eucgb2312_cn?b?Z2dBUG1jWTEwVW5XeVVNVHUzS2hDTDlySisyMWRTWWZBaGcxMGNhRUhFc2haQ3hC?= =?eucgb2312_cn?b?bWRoZnBKN3NpZW0rMWE4bENvRndVSm5EZ2ZVUG1yUEZ2THN4dWJ2OFNuSFNxaTQv?= =?eucgb2312_cn?b?RmVkeG93TjFmTFN5RXViVHRWN1lZbkNvNG1TdHFuSnpSeER5cnE2bGNCM1NMOXdG?= =?eucgb2312_cn?b?NHFJSVpSdUJuUFBDeUlhbks1c2d0MEhaSG1DTmE0THVQbmdjaDZZejFQbTlBUUd2?= =?eucgb2312_cn?b?akl4MFhGc2RLejZxVFY0Q3NlY3BsSXNySll6VVN0OTcyZWsxcXVTNmZTc0EwWjdp?= =?eucgb2312_cn?b?Vjhvb0xrdFY1M2x0TnYwN0FoamxxWUIzb1NFTm5nWlMrVHVpaFR3MVVJVWFva0tp?= =?eucgb2312_cn?b?UU9KN2lMTDZNOGQvdkk2VnNDV2RnZlJ3ZDZlQWFDSHVod013Sm1JRHZsSUh3OHgr?= =?eucgb2312_cn?b?R3Z0QThET3VITmlUeG5yeDA5aTB1bllmdUV5ZFJTOGk0UFlWU2xQbmU5SWprV2pR?= =?eucgb2312_cn?b?ellZbmVjbitZdWlqSjBSYUFHZTlJSHVrQjM1cFZEazZ6QUNTdnhIeUY0VjRNM21R?= =?eucgb2312_cn?b?Q0RvMmlsaHErWjBYWUVWQVNLWDhFOFVxWmZYUzllWGcxakVVME5oZVVWN280bUQw?= =?eucgb2312_cn?b?QWJqL25FTEhOaVluRmN3YTd5c0c1cWxIbktTQ3VtR0o0WEszbUlDbDlPQTBXeTYr?= =?eucgb2312_cn?b?aW5FTEsya2taQjc1eW5heXFWeDRrUnZIT0pGM0ZlR0tnQUhJRHlHa2VGdGFPWlRK?= =?eucgb2312_cn?b?ZlFVbGxZempnSzFkd08wNXB1NXhDSFB4SVR5dzRrUEZSWU8zekpDR0NIUGlOZVM1?= =?eucgb2312_cn?b?SlhmT0d6eWJjNlpCTjZabmpVS2NLNGhYbXJKWVE3Qmo0NnozdGh0dGdjT0YvZ3FJ?= =?eucgb2312_cn?b?aVloN00xZDBlSDdlb3JyVDVlbVVNVklZaVpKOVVQZlpPRmQ2M1dGZGdwZ0tDVnQy?= =?eucgb2312_cn?b?b1ZiaFB5b1YrMTh6cFU0QjJvWjhUYy9kcnV1VU9zaXppc2Z0d3NlWmxER1RldkJF?= =?eucgb2312_cn?b?STFQaGxnaTB1aFZRK29LMUlXcmxmOGFEN2hWT1puL1g5ZXBJQW8wdTRYTTRuRzN4?= =?eucgb2312_cn?b?QmQxOEtXTWZGOElMQlF0bVF2NHlnR09LZ2NIWGloeW5qeDFsSVNqa2doRFN0NVBM?= =?eucgb2312_cn?b?SWg5Y0t1RmVDZXhjcGZWOEZiSmc9PQ==?= MIME-Version: 1.0 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MEYP282MB3302.AUSP282.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 5d18a6cb-2412-4231-57f2-08da3046ff2b X-MS-Exchange-CrossTenant-originalarrivaltime: 07 May 2022 16:31:11.2752 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SY4P282MB3584 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: [ovs-dev] [PATCH 2/2] dpif-netdev : Fix ALB 'rebalance_intvl' max hard limit. X-BeenThere: ovs-dev@openvswitch.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ovs-dev-bounces@openvswitch.org Sender: "dev" Currently the pmd-auto-lb-rebal-interval's value was not been checked properly. It maybe a negative, or too big value (>2 weeks between rebalances), which will be lead to a big unsigned value. So reset it to default if the value exceeds the max permitted as described in vswitchd.xml. Fixes: 5bf84282482a ("Adding support for PMD auto load balancing") Signed-off-by: Lin Huang linhuang@ruijie.com.cn --- tests/alb.at | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/tests/alb.at b/tests/alb.at index 2bef06f39..17bb754ae 100644 --- a/tests/alb.at +++ b/tests/alb.at @@ -197,7 +197,25 @@ get_log_next_line_num AT_CHECK([ovs-vsctl set open_vswitch . other_config:pmd-auto-lb-rebal-interval="0"]) CHECK_ALB_PARAM([interval], [1 mins], [+$LINENUM]) -# No check for above max as it is only a documented max value and not a hard limit +# Set new value +get_log_next_line_num +AT_CHECK([ovs-vsctl set open_vswitch . other_config:pmd-auto-lb-rebal-interval="100"]) +CHECK_ALB_PARAM([interval], [100 mins], [+$LINENUM]) + +# Set above max value +get_log_next_line_num +AT_CHECK([ovs-vsctl set open_vswitch . other_config:pmd-auto-lb-rebal-interval="50000"]) +CHECK_ALB_PARAM([interval], [1 mins], [+$LINENUM]) + +# Set new value +get_log_next_line_num +AT_CHECK([ovs-vsctl set open_vswitch . other_config:pmd-auto-lb-rebal-interval="1000"]) +CHECK_ALB_PARAM([interval], [1000 mins], [+$LINENUM]) + +# Set Negative value +get_log_next_line_num +AT_CHECK([ovs-vsctl set open_vswitch . other_config:pmd-auto-lb-rebal-interval="-1"]) +CHECK_ALB_PARAM([interval], [1 mins], [+$LINENUM]) OVS_VSWITCHD_STOP AT_CLEANUP -- 2.27.0