From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2DFD6EB64D9 for ; Sat, 17 Jun 2023 13:52:19 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 46D016B0072; Sat, 17 Jun 2023 09:52:19 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 41D196B0075; Sat, 17 Jun 2023 09:52:19 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2BDBB8E0001; Sat, 17 Jun 2023 09:52:19 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 1D25B6B0072 for ; Sat, 17 Jun 2023 09:52:19 -0400 (EDT) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id CDC13B04A9 for ; Sat, 17 Jun 2023 13:52:18 +0000 (UTC) X-FDA: 80912379156.30.B5714DE Received: from NAM02-BN1-obe.outbound.protection.outlook.com (mail-bn1nam02on2129.outbound.protection.outlook.com [40.107.212.129]) by imf13.hostedemail.com (Postfix) with ESMTP id 092122001E for ; Sat, 17 Jun 2023 13:52:15 +0000 (UTC) Authentication-Results: imf13.hostedemail.com; dkim=pass header.d=corigine.onmicrosoft.com header.s=selector2-corigine-onmicrosoft-com header.b=aJrVoKQp; spf=pass (imf13.hostedemail.com: domain of simon.horman@corigine.com designates 40.107.212.129 as permitted sender) smtp.mailfrom=simon.horman@corigine.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); dmarc=pass (policy=none) header.from=corigine.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1687009936; a=rsa-sha256; cv=pass; b=ATbWvFC3ktcWQ97iiTNp6R0hdGfeD8OcHkQs40/lIGL0D1U8Q9BuTDBq9KJVIEo/9OqwY6 lgNhWDfhzbs9UOFu7+IXe5xa0M2TEvUmM8xVyrJqq2SWxYO5SMPO/RWlxK8eNCs/Wgay7l MxrUknIMxVAG1DDjuLiskKvtHdIh4pM= ARC-Authentication-Results: i=2; imf13.hostedemail.com; dkim=pass header.d=corigine.onmicrosoft.com header.s=selector2-corigine-onmicrosoft-com header.b=aJrVoKQp; spf=pass (imf13.hostedemail.com: domain of simon.horman@corigine.com designates 40.107.212.129 as permitted sender) smtp.mailfrom=simon.horman@corigine.com; arc=pass ("microsoft.com:s=arcselector9901:i=1"); dmarc=pass (policy=none) header.from=corigine.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1687009936; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=j3MyQXE+WTyrTmJiiIsKyRKjMCACTEohIO4bjRx76IY=; b=Kvkom1iglNGzYXf6B5wrm5N5YZ1BsEyo4t4bgYI1iAypCbrB90kQ2KuBQtbSD4JoEgomyV 3OvzKrRC7XoRVct7r9GDUkSwQ3fkvkKEZKqtmAKaf4X05GjSYYan9zWjtiuexcNyXt8Vzq yfEnOvMcbKXCLdCl6HOb/eTB1/SrtBM= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lhXwocKAUS0PQ0T7UXqroZBn8v+jdZQPNba4YxGAz3dG4F6kKTnL62z0E+P9Dqa45ZtrivIOg26sqrL/q0z2akVxuxqULEWA/CN7GABFVUD7+XUMnmZNPXVXMouGHcE1jLTqXVD58Z/CqaKNdZFwka2yO7TCbRV+AO9sp95anNZD69/t7hZHEnVNsSR7Wu+H6eKrMA5W97zrBTzu1dJHxU8L2Xlfso/ncWy55OUI6PmFiAdEQBNOgcfuzdQNdWk78oaaJL+FOhqZ49oPDli3KVDefx3M5wC5tMRxNNx8/yh2yHaZR/YQWjnAqkd85ypEJu5ETC1mKdz1yxbtLtD1XQ== 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=j3MyQXE+WTyrTmJiiIsKyRKjMCACTEohIO4bjRx76IY=; b=BCHZHmXNV/svIPCtPqXOdNLG5wOJPXJEsasgTdJza7qeysdaPudke3kNiKqh7vVnAJLMVp8P26TqZnC/8tlza1VVRpfRPWCbSNvBLpVyA6zlZtT1ziwmAOnI7F84LYgtW3QjpGkwFQqPwHNWpXRJGnatjs/FptMcsidYtww5+L+PNTFa3pqjVs+bKMeZ81q/v1r8Voq2ZX5Se7doECndF5Xp1bmUhTA/epuG0fmfWY/oYk4LJeRCutjgvjSaI/tb9kO6/bH5QoRyFlsOJQvw/txVSVcYzxgK7TjC64XMGSDRNQOBvEaMNHfBWgT6bVYC2G6rRD05baihEZsTFfIsmg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=j3MyQXE+WTyrTmJiiIsKyRKjMCACTEohIO4bjRx76IY=; b=aJrVoKQpcKRzRVMlKFzHLWtllPwdtbMx4jgZVIUvvzpo5/OMNWEk9h8Deb9BdnNeuRjdT7Pt/HGuZTxaOs1/JOX885pKw5WJHnny2Vhuphe+RzaW8xovwYKRwHflmSKKQ/qlAvkaVuIvY7dYlciDP1sPaLaUE1teznsKnaNJ8DE= Received: from PH0PR13MB4842.namprd13.prod.outlook.com (2603:10b6:510:78::6) by SN4PR13MB5757.namprd13.prod.outlook.com (2603:10b6:806:214::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6455.32; Sat, 17 Jun 2023 13:52:12 +0000 Received: from PH0PR13MB4842.namprd13.prod.outlook.com ([fe80::eb8f:e482:76e0:fe6e]) by PH0PR13MB4842.namprd13.prod.outlook.com ([fe80::eb8f:e482:76e0:fe6e%5]) with mapi id 15.20.6500.031; Sat, 17 Jun 2023 13:52:12 +0000 Date: Sat, 17 Jun 2023 15:52:04 +0200 From: Simon Horman To: David Howells Cc: netdev@vger.kernel.org, Alexander Duyck , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Willem de Bruijn , David Ahern , Matthew Wilcox , Jens Axboe , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Menglong Dong Subject: Re: [PATCH net-next 01/17] net: Copy slab data for sendmsg(MSG_SPLICE_PAGES) Message-ID: References: <20230616161301.622169-1-dhowells@redhat.com> <20230616161301.622169-2-dhowells@redhat.com> <708510.1686984195@warthog.procyon.org.uk> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <708510.1686984195@warthog.procyon.org.uk> X-ClientProxiedBy: AM8P191CA0002.EURP191.PROD.OUTLOOK.COM (2603:10a6:20b:21a::7) To PH0PR13MB4842.namprd13.prod.outlook.com (2603:10b6:510:78::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH0PR13MB4842:EE_|SN4PR13MB5757:EE_ X-MS-Office365-Filtering-Correlation-Id: 8d849ec3-157d-48f2-793a-08db6f3a0cee X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: iBrRhrBaweHwjFzrsHXEyeDMB4ntSM0gwO2xJtEWfogja8+yAWNhFEhTnymgfWlmIVtiX+dPlBAJZkBu54YZfu9Sclpvq+K3TsPp1z6sF4LWurf5vfRkiZ7ZaR1cxEimmVzP7OpqzKziuPY/+4LSuBwCqTfnEVB4LxZ2D50iiJodpude0H65gR+ViyTA5RRdU2FBRQefsIBHQNNgDoBRADywf4dqbx9kdJA+/RT12qJHUAwS3K4LKt1UTShlX8wjXmS67bbMBf+OaD3t8IHq5zSmBP3rKlj18PrhOMKB1X0XIiZxK1w9Mfg7/iBtHvRKTDdzu/DYPE20xfhv9ZMhb98Uqn6pg4trGorEi/jB8dEn3wwyX6Scaty5sHnXc31sX5KT4J8o3r7qSgKF1RUVjdIfND78s6rSS5mE3irgQz6yToXR7zJc4uyr6vH2GcNPdnI69UjsUlZrsvLVSOfk2CPIC0f6yd0lAQREl7BCDYy6/7AlUZkcMWv66Vz78yCYaSGpJzHgbfYMN2qnZ8yhjaqg2/uGXUfR73+3RUTEPtSF7JjXAxq5Pm4lOechqHAm X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR13MB4842.namprd13.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(136003)(39830400003)(396003)(376002)(366004)(346002)(451199021)(7416002)(41300700001)(8676002)(66556008)(44832011)(4326008)(6916009)(316002)(66476007)(5660300002)(66946007)(8936002)(54906003)(4744005)(2906002)(478600001)(2616005)(6512007)(86362001)(38100700002)(6506007)(6486002)(186003)(36756003)(6666004);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?mtPZohzH0O+JI8bXeCNy6gtpd5KI1MrCMe+kxfNtcv+DHZ+DKWE6PvrADmA1?= =?us-ascii?Q?iUPG2b4cFttv9GU/v/ducOn3dxl1r+H7BK29CXOa4514iqQ1mTxQZiUaNqyj?= =?us-ascii?Q?XSnpng2vvi3aj9LvTbjogySe4ST9OCFOu2YYS6bTF9Hx3G3G5lJZSGfGOduA?= =?us-ascii?Q?DQLnE3LCWrwzlsKv+qjDkSp1ehTtpcsDbQPVpbDoDPWNfjJ8d+HsPTZoeJrg?= =?us-ascii?Q?j4rZZFZsn98kOunhscixa1MPiTYuuEfZ/gXPzlvKC7uN240QH7PgyTRdzfu9?= =?us-ascii?Q?96qSOgb3iJgeNAeSZe8XqhRurIhLhYqOxZIP1yV7+SptoWWRD6jT21uygQAs?= =?us-ascii?Q?VWwDIJhsyLePujDBjTrecExhviBJgyofmRFANdP7eQuRAmYa5YakGe16Jdbk?= =?us-ascii?Q?PJ1OedZGP76PRIBPr2jtlEatg6vJi6SLgdbSPNZHPXDumogywg6bNfSX+ZGo?= =?us-ascii?Q?IrwosCKvEDIFf5ojL1L7HaoVLpQ4rj6vbBq4LrA03m728EcQJrVfRXhueHzu?= =?us-ascii?Q?6IE2mJXB/2ODHu5/EFvZozMFnCSRiIP6B29LEpdq97Wy3/ItClf3QFNtU8VG?= =?us-ascii?Q?uEx3WZXedIAWN7hIYdvIJQs3mRT94MqNR/j4Qyrk6KAL8P12FBg5oSeZ7sjE?= =?us-ascii?Q?wBFpLmrnBh/m9YvETv2wtn3KWuQFDOohvmSd7uafdUQ81RoL2kHpfPtc217h?= =?us-ascii?Q?wDR/OGGIHz6W+D78YRykeY1PUymgTShzDFtqjlIZ5qOPO6cFOrP2/a6jeNPm?= =?us-ascii?Q?LONbdX1lLASRiApIZw48U5qnaCEpjUI3bzbgQSaXbV7Ljn9gUNqgcSKkEp6I?= =?us-ascii?Q?At++0XUVTxEu2KD6J6MlNzMcwbHmHJkBowEgUN3KFJVnM7If22Ql6gCbeu1l?= =?us-ascii?Q?O0kTMmelcrnOROC5Ca0AeqBb0qX7KQO1aRgteIIZt6ZTuFyUm4Q/TK6pDOcZ?= =?us-ascii?Q?xlVAW7fow7BQmIA8ejjiLHIfxi/9nGAlji4fcM2OzxoqzHsPF97uZVEU/GsN?= =?us-ascii?Q?Oqw8J0Zt8xrH/T3yQ1xCxQp2CUX14PNj1SX4jkTpIOKJYv7pPtDKa2frYQJS?= =?us-ascii?Q?0QQ/8Lv6t0Lfr/6LFPjARC2+hwA3YxgymJposUNuqUZ8K3PL9SVFdz3tzcCv?= =?us-ascii?Q?qGGs+FLzmkz6iThamTs/gSPro6UbueJD/VWLBx8c863+Y0emW259MLQtKzKT?= =?us-ascii?Q?7OCWKzst4gZ9dTDLPGDGDO9yo0xTSsSUgswBzeh0et/UOtO08zEoD7DF42iS?= =?us-ascii?Q?sG0X5RXRUI5k91Gs9uk9ba/y6F6LHiolp9BWNiH53ThLt0hm5In3V/YP4v5J?= =?us-ascii?Q?q5IT1dbDtdKSlIdnRSDT+YX3eyKKeRBlK5hHO3RPASg8CJ6p6Wjk4/AqeG+6?= =?us-ascii?Q?MTpYjiM05wGBF8lAPHZbcGBpVMUnmiSSgv3/47M5p2QnAYSzbWGhzevzaW/M?= =?us-ascii?Q?JCSwIm4kYLPyuC7P0aFTJBeFToSGs1TW2/z3ezjXbWXDZca/F0xkhzbn5uu0?= =?us-ascii?Q?kh52M71rIVqjj+9z3QRz4LZ3CrLIUMneFGyQx5KsCVdWnqPlmwWq2gM8tETd?= =?us-ascii?Q?p9BbA1FbSB8IQs37D0VbFWWx+ykVesLkkvvuGcYiFXphapueWnwPzIRW4yom?= =?us-ascii?Q?jrja4demc0pj2Bl/JNLH8CHRKPJstuJjlynp4Cm2qBKK0IY9oXCUfErMMRba?= =?us-ascii?Q?XlBjSQ=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8d849ec3-157d-48f2-793a-08db6f3a0cee X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Jun 2023 13:52:12.1201 (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: RvbFXz52/IleX5apN+ltwsJjPJxre2Use8mmnB96xDLXiocunECEGwaVMGBZQ3P0gJH8SAvuv0YHyKEPnquoopQpqluUNVSfzqd0qnFOZiM= X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN4PR13MB5757 X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: 092122001E X-Stat-Signature: fcuxoff7jmj3xmn8x4gphym3cdxicsem X-Rspam-User: X-HE-Tag: 1687009935-841673 X-HE-Meta: U2FsdGVkX186+kJCQKbjjaA7LWFMslaZFoMySyJGi/tZDvYfdc3EnSxhNJhIjNYt6ikA9Gkprtnwg7itqj9iKITk/FfR6baTD/kUCzQ5jQLsW6J2ChRGnSrtW5fhb8s1bJyw+Kgqq83JPaIZMzexnM5c0fWs2xgRHEaT4K8QFsIoYJOeLiMM8CjXrD31kijZvrI6LNQAFuTm0lzp/zptNWmigtKA/5dkXBF0NkETJZDBRxKM6/gbr1zlkz+YZ8Q3A36lvM7a0r3Kr2TykaY0Yn4kwQfmiIe8oPq6TIynkez+XrXKm/2gLZU8U/MayG7xXeKI8oclcC7OM/jeQGJQC3VSbU96Q+1KANd2j3XQKF9UAat5L+z7Y6juNuT1RK3FvHLbGQwvK4SOgB7IgPaspfKcj757dHK9463XtJJho7i9fjIkWhV3XHMaLgp0wSo3b0oMS+cOrVW/hVYySt5n3yMFSTK0HQPYh0/gfsAFVIpfSyHgPtc0wqiVFSoPhyP8g/HsE/onqJ+5C3q7qpde1R8MMDW8XE04LUivwxa6U+D7j3qmQ4WYI51pyO5TjNwDiGMLVXtmREO0KTm0aZFTAZvqOg5+cHtQImmbvNT0pSn2dFVuGkkteW2p8r1fhJvp3emUezhtMFyNMlIl2ooe+5w4jWHRJU9WB3BQ9XDyqJpQMsWcb2PUrB54qgtaS6/x/YLmUQqm90YT34zqRzpU7hqebPngemSa8g89fMi98IRo4O7TbLMQltjN4Xjl6F97pEI75pvWP6iYZs+u+ulARG7EbYKXoQ2sAaAAHz5lCZrzvmvY9GT8CWuqdXm0IZkBOCjY8cRgnMmmPAn6KZ0+j+aoZQYAv/pqrLIr/3FZ6odvjFSeGbnTJBKBnBtM0avyNPY/O3wws6wtt+XmMzinsOM5CQX+BiDTnk3YyXUbRy1BrxgaC6aMj3hLn8OciwENoVTT1hFk1brCv3gBjRK h4pOVsVS 210GehlU3ovk9ykiNQ9y/nWs27hy+EM7krh5cDC4EHAlAXWEtKUjgyNmYhVSkCjTioNodMXqCmu78+/djzQCv3SlI0c4i5UVfUuxYhg4wmJ1jKbiIFC0nLSWVweHDjDbaQ+SZ2HeAeK8mo877XwD/BUu4osMxcLny0GhwQ94pdZFKsLfc6W67a2hPnNzbYm3AvJzdT4mlGiu6lAKXK28Hm4uxNBfJNEISMr0LDGYQ1dCxAwcmaOI3Ry+eYB/ASnGju5B6CqOkX74TqrizTcg1p3qpDvXwnzEMTt3ChE4BpCPaMXOfEseVNhdeA6aASArklu6jSrgdn4Sm8ptG1IT7GieuNShDEQNk5Y+DH9LsTrqc/UbsJWkRAeOiDve6sxZZ/vAFr0Wneh8Tk0BQQ1i6oTiegv2v8b5xrU9GXTUNjNMA/R9NbwXadq16V7WhosLCuwzdngAvGpzVvQY= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Sat, Jun 17, 2023 at 07:43:15AM +0100, David Howells wrote: > Simon Horman wrote: > > > > + cache = get_cpu_ptr(&skb_splice_frag_cache); > ... > > > + put_cpu_ptr(skb_splice_frag_cache); > > > > Hi David, > > > > I don't think it makes any difference at run-time. > > But to keep Sparse happy, perhaps this ought to be put_cpu_var() > > Actually, the problem is a missing "&". I think I should use put_cpu_ptr() to > match get_cpu_ptr(). It doesn't crash because the argument is ignored. Thanks David, I agree that is a better idea.