Date:   Wed, 8 Feb 2023 12:37:31 +0100
From:   Simon Horman <simon.horman@corigine.com>
To:     Paolo Abeni <pabeni@redhat.com>
Cc:     netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        Jonathan Corbet <corbet@lwn.net>, Shuah Khan <shuah@kernel.org>
Subject: Re: [PATCH v4 net-next 1/4] net-sysctl: factor out cpumask parsing
 helper
Message-ID: <Y+OJeyIey9DvsKcy@corigine.com>
References: <cover.1675789134.git.pabeni@redhat.com>
 <8e69455b0e3bc339ca6c00f04fe86660e2aad58b.1675789134.git.pabeni@redhat.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <8e69455b0e3bc339ca6c00f04fe86660e2aad58b.1675789134.git.pabeni@redhat.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?UQc8Ramidt0OUBADmh6gtW+86lfcimiE9SzQZoFvhigst2oGnVfwiyzX+EFZ?=
 =?us-ascii?Q?ek73dutxFU+UsvxTWqewQTcijWrg+5mwOXzR+E03js7wfBBVVbYYUIIWfRMW?=
 =?us-ascii?Q?SFEDq5DTstwTJSfmRZM+lhTFmzQk7sNVIRRmHAPviFks5MRYltkUq0Um9XDp?=
 =?us-ascii?Q?T2ap5BSzOSY4t14+s19Eeyoz3LS4m8ZlS+iIFEEmjcVzHQofO6pHUsPmjDau?=
 =?us-ascii?Q?2+umY6vj3oC+nB2eO/8zSjMLnisFEd3wBg6PpOjqsaPbCzVlxWUv90OuDW6Q?=
 =?us-ascii?Q?AWsKxIlKNxHg9afotn84sNi4ZnkPOPYhbUiiQgqLMdP6x4IQ0MbpmAA0EO/2?=
 =?us-ascii?Q?U6iuWW8eNEgfzDWWHQ6sps3q8bllOADt0TIsgbNq04s0QSlWo+zJP4VY/W08?=
 =?us-ascii?Q?VOfq1ZFY/yurw61d7ujCpnK98DWuBqLrfOuykJf5ypB5NqwMc7hasvXxfN9P?=
 =?us-ascii?Q?0kCPFPCQW7+PtIdmlZ9FP1btVldWG6LFByZerGj9uhgkpWUIIIyd53/mn4df?=
 =?us-ascii?Q?KgtA1ZCKyhrDNmfIh5+lnUW8kXBVqP93YLRZDBEnom+z3nNBG58N9fshdF0T?=
 =?us-ascii?Q?VyJGt2vSiuS69fRa5+unPFa0gtU1Yw0xFlz4Q8SMrpwDGzVsTA6OtnMFV2Eg?=
 =?us-ascii?Q?VLaQcX8YshY0pv5qnfUe7Y8Hi2SOxkXxYvN8MsJ2VapirxZ0lwZ4r3MGYgg7?=
 =?us-ascii?Q?God7N5/SrdMlRXT39kFI6PfIpgld/s64x/g11DHieS2GINBiESTwlgRHyu2S?=
 =?us-ascii?Q?l7ahqtKHAX8oAfvxizx8oEo7JpGNbLrmyxjs26r/mdBH7eDqyhI8B1v95asA?=
 =?us-ascii?Q?AmHQQ5WqWAgySQzhN4H8rNLn5Z44oUw0Ien92Qlk4+t/fi/zw1+FFaKZoqq1?=
 =?us-ascii?Q?Xbi6l3kMh3cg6/mseLt5vPwibPf6KouyWOYDPYru1HbrXYJrXNyCm/YV3d+u?=
 =?us-ascii?Q?UmGmpLFpjtOnPn2QVDkRu3kAqp6F6NhnHWoMbeQqMo7EO5HqV+r8ePgf4rcg?=
 =?us-ascii?Q?MzdelzCSUfivSY/jyl06OUPnyBP8tyOyhsjA1sxFhYAb9URX3DI/2pT+QgWJ?=
 =?us-ascii?Q?TTn8jQDMNZnQDE/Y4qvIruHIRUYODd2QFaQNuP0IQymqeeyDss/r+Bh9f0i2?=
 =?us-ascii?Q?djSAsa5G0xKnOTQ3DJmGJoSvNHhKLtO+WfbuMs0J6zVHR4DaUeW/zAVGBCfQ?=
 =?us-ascii?Q?rMIKcp1mwf2QQpHLeRRICcRcDS7dbSwrigpYZdsrymBTPD3F78TvS2M3xSXL?=
 =?us-ascii?Q?DFwNIFCSkcVJQtfnrijNpmsR9+CVRwLls27OMtUt9b5A5M2++nk9yuL1hKT/?=
 =?us-ascii?Q?9lI83JXybKMmSGVrisaZXps02EE3OsHUtPeqBSVfeJishp/t+VGMo5JlCehC?=
 =?us-ascii?Q?wiB/HD0isD2FDIn2nUS6xnFvLqaMrYdNFwAXoEvAUi+FsP8ATN1cxaDfsapx?=
 =?us-ascii?Q?siak8IEnI25V7N1WlsuwlWxtxFyljBvtPbQ1KdkVWDVoV1+Mf7GY1sbB7AQ2?=
 =?us-ascii?Q?o71B6H7Msl0NERh0FNsc9Fui5w3RtglWKPweq/I1fESzs2BPg7CTNiu606yY?=
 =?us-ascii?Q?xfjVlFNu/SwwA2E4mfzpzfS88lvuZ0YfMxuRCJlMA6/WlOadcSZEAHvmbB2b?=
 =?us-ascii?Q?IdQHipjCNQklel402ffFQGjCkTPbXz5zqfWTfCrGEXrs3nLZqDaZK/X+paDi?=
 =?us-ascii?Q?CKIhbw=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0aefaf24-7715-4495-0e79-08db09c8e10e
X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Feb 2023 11:37:37.8971
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: QT/TRrtViggA7O/Q+92D5XeNur7UeUAWhNZIkoRCHtr3DgemVpyF/NHtnlJktZmcXEDtqPFpNMvsu259Y/SxxdeM51lmjWNb5A509bp2+mE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR13MB6157
Precedence: bulk
List-ID: <netdev.vger.kernel.org>
X-Mailing-List: netdev@vger.kernel.org

On Tue, Feb 07, 2023 at 07:44:55PM +0100, Paolo Abeni wrote:
> Will be used by the following patch to avoid code
> duplication. No functional changes intended.
> 
> The only difference is that now flow_limit_cpu_sysctl() will
> always compute the flow limit mask on each read operation,
> even when read() will not return any byte to user-space.
> 
> Note that the new helper is placed under a new #ifdef at
> the file start to better fit the usage in the later patch
> 
> Signed-off-by: Paolo Abeni <pabeni@redhat.com>

Reviewed-by: Simon Horman <simon.horman@corigine.com>