Received-SPF: pass (google.com: domain of linux-kernel+bounces-171064-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <a3a0c162-9226-4bab-98ec-56e2da95fff1@intel.com>
Date: Tue, 7 May 2024 12:06:03 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH net-next v5 7/7] xsk: use generic DMA sync shortcut
 instead of a custom one
To: Jakub Kicinski <kuba@kernel.org>
CC: "David S. Miller" <davem@davemloft.net>, Eric Dumazet
	<edumazet@google.com>, Paolo Abeni <pabeni@redhat.com>, Christoph Hellwig
	<hch@lst.de>, Marek Szyprowski <m.szyprowski@samsung.com>, Robin Murphy
	<robin.murphy@arm.com>, Joerg Roedel <joro@8bytes.org>, Will Deacon
	<will@kernel.org>, "Rafael J. Wysocki" <rafael@kernel.org>, Magnus Karlsson
	<magnus.karlsson@intel.com>, <nex.sw.ncis.osdt.itp.upstreaming@intel.com>,
	<bpf@vger.kernel.org>, <netdev@vger.kernel.org>, <iommu@lists.linux.dev>,
	<linux-kernel@vger.kernel.org>
References: <20240506094855.12944-1-aleksander.lobakin@intel.com>
 <20240506094855.12944-8-aleksander.lobakin@intel.com>
 <20240506112931.39614ff0@kernel.org>
From: Alexander Lobakin <aleksander.lobakin@intel.com>
Content-Language: en-US
In-Reply-To: <20240506112931.39614ff0@kernel.org>
Content-Type: text/plain; charset="UTF-8"
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?SGdOSERCbTRhRHNHWWZveDNsWXFFQnVUSXZvamxvN3pNZ2t2RkhxdktlTXFu?=
 =?utf-8?B?dHlXYU9TemZkdy9vK3N4ZnA4OWxjQlc5OW9HeGtYSi9xc0l4SDJjVjZEZWJV?=
 =?utf-8?B?U0g4aWN5YmJUSzZyMDF1YTlWRW5iakEramJUOXNCaWZGWERSQjc2Njg5T1ZR?=
 =?utf-8?B?V0JQbUZBb0R2clloSUlkR3QrckU2WXIrZ3ZDZi9VNjAwbExCdTVNRTRhQlZE?=
 =?utf-8?B?eG0zZFJzRnRmOGZzcEx2YkpTT2xhS3hIYzFiU0VHYUp6N0E0RjR5M2lMVVhw?=
 =?utf-8?B?VTZTN01rTUJ4S3JCL0pRVEh4NytmekFxd1JPc1o3Z0FwRXdweUZicmpFbnJt?=
 =?utf-8?B?ak5IdHdLNVlBdzlvVHZHTXBUNzVwbjhZdFBDMHNTTkdCT1lWY3F0eXNYU1M5?=
 =?utf-8?B?VnlEK25CV01pTU8xUGJmMlYwSnlRdnRhTXBoZER1MzYzVyt1YW8xa2d6YU1F?=
 =?utf-8?B?UlgxK1plY29LV1QxU1dRZ0UvR0lXdFh1Y1czdTVKcmFQc1BBWGxIejVNbVlt?=
 =?utf-8?B?UUYvVElSUnVkdmtHdXlLZ0VveWl6UDlpeGk3RitDWTlERVZwazlHWVJPSTdo?=
 =?utf-8?B?QzgxWnFnYjhVRUdQUkw5OGp1Mk0rODhSZTRqdElSaWhSQlZsR2JRWXl4SWJY?=
 =?utf-8?B?N2E1d0Q0b3lOQnA5Nm1FQ1NseU1UNmF5NHNwMURRak0xd1FVaS9OZ3FYZU9N?=
 =?utf-8?B?OENQb2ZqVGNuQkZzc3N1SUFTZ0dub0IxakJnbThQalpIK1UvR3pnYWpWb1gx?=
 =?utf-8?B?UjcwUlNZU3pLQlNCM3VPUXdiV1lESlVoV0tMTjBROHVpaGw1bHlSZWo5c1JY?=
 =?utf-8?B?T0VrT3RqVFlpQllzdzcyRlJzSXNjK1owNnJ0SGxzQXFGQlZiY0wySHZVbkNm?=
 =?utf-8?B?QldqY3BhTmFPMHNHbHd5YUJiNFFuVmd2SUp4bXU0b3dCMFVMU1E0dTUrbFNm?=
 =?utf-8?B?VlV0TkR3OGtGaTNNd0VCNEJMSmoyWTUzZ2VpbWk4dmpkeStBbG82NS9PSnBB?=
 =?utf-8?B?YUlDVU1veW8zbmhsTGxObUtKK0hlS2hTb3RvZFY3YUQrdFpDb1ZiYTBxWkI4?=
 =?utf-8?B?OWhWcFU1eExoUzE3ZlZKdElVdEgyT2lvTzlkbUR3bWxndzJjTmxOcmVJcHBH?=
 =?utf-8?B?a1hodStXWElhUlcxN3RDaVlYdDBna3pWZ2ZqWVgxNnRUV3lvanRoWkxpRmts?=
 =?utf-8?B?bDlNV2RhbHRHRTJKM1Zycko0c0R4ZzJpM05RSndCbUNENnE2R2t6RXM2THhH?=
 =?utf-8?B?K1A0aEo2TFovRVNmdFI0Ri94L0FvTnVab1BUM1ZhL0Fmcmc0Mnc5cmc0ajdx?=
 =?utf-8?B?K1dNRTE1VFZrSDZaNzBlc05zREErTEpCMEZwUE4vWkRIUXh3S0xPRHJkTHhF?=
 =?utf-8?B?d2U5UUZuallFazgzcHZvVEJCNVZ5WjBndVNpczBXT3pYTGxYb1lGT2k5S0hR?=
 =?utf-8?B?YkdXdVNQeTRaTytpSi9tenBjazRMbWx5QzNyTnpHVmNQaXpZYU15eE9Ma1Z6?=
 =?utf-8?B?Q2paMEptMWx2RktuN0trNU5XYjcxMGJSYmJOUzc2QnQ4K1BnK2drYVl1cG9w?=
 =?utf-8?B?MElBTzZtWTBQWlJlSTBGMEZ1aHE1TFp0U1VaU3ZkUFI4bVZiWFpHQ21FYkFX?=
 =?utf-8?B?QXlvK0pUcCtneCswL3VyVFRqbmw2aXFDVTBjNnBxQ1NzQW0zeFBIWmdWbFBP?=
 =?utf-8?B?ekI5Y09WQkFQSXJIa0tsWGE3S1RHcTA3cm1lVVZ3UmdEa1MxTFJsUyt2Qit4?=
 =?utf-8?B?ellBekNYbDd0cEtLU2JvUjRRYkgySDJRb05vaWxWZ2QrSzNPUXMzVHh1UkU1?=
 =?utf-8?B?TGhCeXpFWUJlTVlmOFR4cWkraml2WHAzaXdDbW5rUHpBQ01VMkxHcDhGUDFz?=
 =?utf-8?B?N2p1RkVhNFBndjR5VW9wRzBGN3o5c0ExY3lwbVVmRndxTFVjcVNpUEs3UHlw?=
 =?utf-8?B?TnZEYmdjbWpGZ0tkOEJEa2ZpamprYXlmRThZKzU5cjN0cW9oekdWNEVLZ1Nr?=
 =?utf-8?B?Wk1FamVMNHp6NFo0S3dJaDJ0OUZpcnJQZ3E3SFFkU3dGSlN1cEowWWJqUytz?=
 =?utf-8?B?cUhTT2NuWllDcmhlekdxWGswbGU1Vi9JeHY3Q3UrL040bnZlbE96Y0lRQy9Z?=
 =?utf-8?B?R2hjcFZ3WmlCNUtPdytYZTdZRmF3UEd0cVdTV2pjamhaYXp1bWk4bktYRU5S?=
 =?utf-8?B?WUE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: c8c9b98d-31bd-4952-69c3-08dc6e7d5f21
X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB8718.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 May 2024 10:06:32.6906
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: owftXkl9o+sWcHM9BdGm/2oD1jpoOQqAwQ4Og7ot/LGV3n7/ZmmBK2VE6zk3Z8lysL8EbcLQ656mRa+m9vBEPhG4ZKypSFfzAFwceVv8f1E=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5260
X-OriginatorOrg: intel.com

From: Jakub Kicinski <kuba@kernel.org>
Date: Mon, 6 May 2024 11:29:31 -0700

> On Mon,  6 May 2024 11:48:55 +0200 Alexander Lobakin wrote:
>> XSk infra's been using its own DMA sync shortcut to try avoiding
>> redundant function calls. Now that there is a generic one, remove
>> the custom implementation and rely on the generic helpers.
>> xsk_buff_dma_sync_for_cpu() doesn't need the second argument anymore,
>> remove it.
> 
> I think this is crashing xsk tests:
> 
>   [   91.048963] BUG: kernel NULL pointer dereference, address: 0000000000000464
>   [   91.049412] #PF: supervisor read access in kernel mode
>   [   91.049739] #PF: error_code(0x0000) - not-present page
>   [   91.050057] PGD 0 P4D 0
>   [   91.050221] Oops: 0000 [#1] PREEMPT SMP NOPTI
>   [   91.050500] CPU: 1 PID: 114 Comm: new_name Tainted: G           OE      6.9.0-rc6-gad3c108348fd-dirty #372
>   [   91.051088] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.13.0-1ubuntu1.1 04/01/2014
>   [   91.051649] RIP: 0010:xp_alloc+0x76/0x240

Ah okay. Didn't account generic (non-ZC) XSk. Will fix in v6.
Thanks for the report!

>   [   91.051903] Code: 48 89 0a 48 89 00 48 89 40 08 41 c7 44 24 34 00 00 00 00 49 8b 44 24 18 48 05 00 01 00 00 49 89 04 24 49 89 44 24 10 48 8b 3b <f6> 87 64 04 00 00 20 0f 85 16 01 00 00 48 8b 44 24 08 65 48 33 04
>   [   91.053055] RSP: 0018:ffff99e7c00f0b00 EFLAGS: 00010286
>   [   91.053400] RAX: ffff99e7c0c9d100 RBX: ffff89a400901c00 RCX: 0000000000010000
>   [   91.053838] RDX: 0000000000000000 RSI: 0000000000000010 RDI: 0000000000000000
>   [   91.054277] RBP: ffff89a4026e30e0 R08: 0000000000000001 R09: 0000000000009000
>   [   91.054716] R10: 779660ad50f0d4e6 R11: 79b5ce88640fb4f7 R12: ffff89a40c31d870
>   [   91.055156] R13: 0000000000000020 R14: 0000000000000000 R15: ffff89a4068c6000
>   [   91.055596] FS:  00007f87685bef80(0000) GS:ffff89a43bd00000(0000) knlGS:0000000000000000
>   [   91.056090] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>   [   91.056458] CR2: 0000000000000464 CR3: 000000010229c001 CR4: 0000000000770ef0
>   [   91.056904] PKRU: 55555554
>   [   91.057079] Call Trace:
>   [   91.057237]  <IRQ>
>   [   91.057371]  ? __die_body+0x1f/0x70
>   [   91.057595]  ? page_fault_oops+0x15a/0x460
>   [   91.057852]  ? find_held_lock+0x2b/0x80
>   [   91.058093]  ? __skb_flow_dissect+0x30f/0x1f10
>   [   91.058374]  ? lock_release+0xbd/0x280
>   [   91.058610]  ? exc_page_fault+0x67/0x1e0
>   [   91.058859]  ? asm_exc_page_fault+0x26/0x30
>   [   91.059126]  ? xp_alloc+0x76/0x240
>   [   91.059341]  __xsk_rcv+0x1f0/0x360
>   [   91.059558]  ? __skb_get_hash+0x5b/0x1f0
>   [   91.059804]  ? __skb_get_hash+0x5b/0x1f0
>   [   91.060050]  __xsk_map_redirect+0x7c/0x2c0
>   [   91.060315]  ? rcu_read_lock_held_common+0x2e/0x50
>   [   91.060622]  xdp_do_redirect+0x28f/0x4b0
>   [   91.060871]  veth_xdp_rcv_skb+0x29e/0x930
>   [   91.061126]  veth_xdp_rcv+0x184/0x290
>   [   91.061358]  ? update_load_avg+0x8c/0x8c0
>   [   91.061609]  ? select_task_rq_fair+0x1ff/0x15a0
>   [   91.061894]  ? place_entity+0x19/0x100
>   [   91.062131]  veth_poll+0x6c/0x2f0
>   [   91.062343]  ? _raw_spin_unlock_irqrestore+0x27/0x50
>   [   91.062653]  ? try_to_wake_up+0x261/0x8d0
>   [   91.062905]  ? find_held_lock+0x2b/0x80
>   [   91.063147]  __napi_poll+0x27/0x200
>   [   91.063376]  net_rx_action+0x172/0x320
>   [   91.063617]  __do_softirq+0xb6/0x3a3
>   [   91.063843]  ? __dev_direct_xmit+0x167/0x1b0
>   [   91.064114]  do_softirq.part.0+0x3b/0x70
>   [   91.064373]  </IRQ>
>   [   91.064511]  <TASK>
>   [   91.064650]  __local_bh_enable_ip+0xbd/0xe0
>   [   91.064913]  __dev_direct_xmit+0x16c/0x1b0
>   [   91.065171]  xsk_generic_xmit+0x703/0xb10
>   [   91.065425]  xsk_sendmsg+0x21f/0x2f0

Olek