From patchwork Thu Feb 13 11:03:59 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alice Ryhl X-Patchwork-Id: 13973101 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 07653C0219D for ; Thu, 13 Feb 2025 11:04:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 825406B0083; Thu, 13 Feb 2025 06:04:41 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 7D3546B0088; Thu, 13 Feb 2025 06:04:41 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 673796B0089; Thu, 13 Feb 2025 06:04:41 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 4C2476B0083 for ; Thu, 13 Feb 2025 06:04:41 -0500 (EST) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id C7E69ACF2F for ; Thu, 13 Feb 2025 11:04:40 +0000 (UTC) X-FDA: 83114638320.03.C441A92 Received: from mail-wm1-f74.google.com (mail-wm1-f74.google.com [209.85.128.74]) by imf04.hostedemail.com (Postfix) with ESMTP id B159340009 for ; Thu, 13 Feb 2025 11:04:38 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=google.com header.s=20230601 header.b=t7VDWZRw; spf=pass (imf04.hostedemail.com: domain of 3xdGtZwkKCGQCNKEGTaJNIQQING.EQONKPWZ-OOMXCEM.QTI@flex--aliceryhl.bounces.google.com designates 209.85.128.74 as permitted sender) smtp.mailfrom=3xdGtZwkKCGQCNKEGTaJNIQQING.EQONKPWZ-OOMXCEM.QTI@flex--aliceryhl.bounces.google.com; dmarc=pass (policy=reject) header.from=google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1739444678; 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: references:dkim-signature; bh=8sIraLMTcSHYFNbgRkGQfIKI9Dkm/hcGG5yDyhtczYI=; b=0BW0GXRdHyOq7MJYADn/xrq8W4cpZmZf11AFd7Ot7jXEluMgw3X9opD4wgCFFgNka0CZFp pgagxDrmqg3s4ntKoATmQt0tmLmuVBhLrthrkiv+a8SVL8Kp2R30SUhLLFU6PnvPVPmL2z jAP0uAzHcJsJLykwy9hheBCHj9Y1Oi4= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=google.com header.s=20230601 header.b=t7VDWZRw; spf=pass (imf04.hostedemail.com: domain of 3xdGtZwkKCGQCNKEGTaJNIQQING.EQONKPWZ-OOMXCEM.QTI@flex--aliceryhl.bounces.google.com designates 209.85.128.74 as permitted sender) smtp.mailfrom=3xdGtZwkKCGQCNKEGTaJNIQQING.EQONKPWZ-OOMXCEM.QTI@flex--aliceryhl.bounces.google.com; dmarc=pass (policy=reject) header.from=google.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1739444678; a=rsa-sha256; cv=none; b=P2rusUc4HW8Qh8tDsOt2q18gxDn9eqHxHHiNuqTZSyoespPyKmeVHVv1GwkS8Xf2T7Q/7P SgXvu83C9bvXp6DUR+TMUN6PIkMGzPA39uhMA0os9ve7O4s/P8K+9KxIJXJOYTcRsvut9G J+VsRa++0kJilNW1GyW7S/yUabCI2lQ= Received: by mail-wm1-f74.google.com with SMTP id 5b1f17b1804b1-4393e873962so3541915e9.3 for ; Thu, 13 Feb 2025 03:04:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1739444677; x=1740049477; darn=kvack.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=8sIraLMTcSHYFNbgRkGQfIKI9Dkm/hcGG5yDyhtczYI=; b=t7VDWZRw8CM+JnbCDchntJphORibss8aBwAOqh0nQk08VvFSSb7aOAjbn0fWAoArNb YdD9DIPRRgzQTy5u/uPFUlsPSP/0hs0s/hBMIB4zwaL3MX8S15+jefJgI6J4la/z2Clw KGX5ZR6pnl9hQBeHb+Muz0sY5JQonKE2nm2vwBJRcw+mvln/6MxH6yE03OLORYhdv8hU S58/LB6DeZzkXvbwSdse1JZnjTKkhM/rNnHxfNZij4N4QxQgrZAL/0Xie40aKx41SAdW FQvATGFw1AgBoxdMADknwCzI3ZUxSr2BSU9S7Wz2sW6JDah8o0d2+QycRYC0k/U2QH8W KIyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1739444677; x=1740049477; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=8sIraLMTcSHYFNbgRkGQfIKI9Dkm/hcGG5yDyhtczYI=; b=rVF2qZyx+2aIpM15H42bKfes7oaS3zf3hBD86Olce0EInn1mry3Gd2vG/0R8kR7k5j GrudpWbNe9EzGz2YfxGSuW1rpH8Ypoe/7iHrcG4M8WT6O9ANqSl5y/XaQcJMpMVv/CXC wKbqb2iEfxiW8pSD+dE3jeILNc8uilEKp1rDaM6InSLaIhLCkBJsklRbX8gpxvZ9GWDp brNQ0sGF9XnniG5noyMSGsToXR+cTVkP2FEfidkiGJJ74+CSWLGO/1D9Ipe9cgu9r4OX 5ENJk9fDPJfxpQAmt6IePxGrS4ca3cbKm63yNFTo8dlrax8voxNLBI+dsnHIiKHSYMry CkPg== X-Forwarded-Encrypted: i=1; AJvYcCUqdkl+d1forqcLQKgqyI2bylz5D3pHN3JrbWcJvmpwS37nA5MEEc0XIY2DLY11VTTMFOTYFvrZqQ==@kvack.org X-Gm-Message-State: AOJu0Yxd3zNgiZFFjMp4byNTJcgud41VRCv5kchNPaN+bU1ZMTCSUpCF yG+X0ddS/AmvmO0ARqPBf4SEy/r79e7UVOr3dh/dGSYjt5w1tnw1zuQzHimNPxzTf7gaPCjdrqE nGRqWwanI5lOwgA== X-Google-Smtp-Source: AGHT+IFeJbxmgNkUd67pTqHr9xLxt49v4hGS4InUlnB3frDcfO94vjYdcWPTcm+KqK3hfJavANacJ88dKIoI9O8= X-Received: from wmbjh8.prod.google.com ([2002:a05:600c:a088:b0:439:4036:8da1]) (user=aliceryhl job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:3b9d:b0:434:f5c0:328d with SMTP id 5b1f17b1804b1-439601a11bbmr30019455e9.23.1739444677115; Thu, 13 Feb 2025 03:04:37 -0800 (PST) Date: Thu, 13 Feb 2025 11:03:59 +0000 Mime-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAJ/RrWcC/2XSzWrDMAwH8FcpOc9Dkr932nuMHezYbgNrU5IRN krffWqTNSk+BZv8ZCH9L82Yhy6Pzdvu0gx56sauP/EB1cuuaQ/htM+iS3zREJACS1JMxyCKA0R ffIlSN/znecil+7mX+fjk86Ebv/vh9151wtvts59QoJAUkg5cyOX8vu/7/Vd+bftjcysw0RbZG RGjEGXWnmxqZaiQXBE3OCPJKEXTYguhpOArpLaIZqQYgcfgLGgXlKyQ3iIzI31DCtAV0ClTrJB 5IASEGRkBIjkP0kdTtK1fslukZmQZAWZJxRneg66QWxHS8pJjlKNERToFY+r2/BYtg/CMLJKNp U1Jq3pPCFvll+1ye0KlUngSMfC2araGAgmXVfGXmTFGgSJX2noY+IiF5tzohREzaTV4HzKgrXO BcmUE/xGUzCh678i4IL15Ytfr9Q+M5pzMHAMAAA== X-Developer-Key: i=aliceryhl@google.com; a=openpgp; fpr=49F6C1FAA74960F43A5B86A1EE7A392FDE96209F X-Developer-Signature: v=1; a=openpgp-sha256; l=5737; i=aliceryhl@google.com; h=from:subject:message-id; bh=hLnQzTlqJUpLYZOPmxfZzN82Dj0rBNMo1kF2NSwDm/Q=; b=owEBbQKS/ZANAwAKAQRYvu5YxjlGAcsmYgBnrdGzeBJ5RqLDVa6MkQemTYrqLC6LzHhkD8CUZ gLFANXNBGCJAjMEAAEKAB0WIQSDkqKUTWQHCvFIvbIEWL7uWMY5RgUCZ63RswAKCRAEWL7uWMY5 RqjDD/0UFwc/XoVZ+C/UCMbJ7j/HtNooMQ9WHeYZBZZXSb/xEZNGh8MDTSFh3ZlLWOwEQMqvMnA CrQSTftCKT6xxeeD0l5qkS/6mszI3n69eC8fWVwvZ9vsLa3Liy68gs0qiUCTKin1IqJiViZn56z cB1lD0ajFMnxoLrMRrrKJcWFXjPZiT4AeADNrxo6q868eFRbdvEPYWo+P8iGP0jkEh2OvUWzYcU 3GK77d3ywFxLOid38Gm7+ZuLoFYLk2u7BT74tPwBajLyOxUevvrQ+ans6D+Bhu5US9CvAYXQQ+c FuKUTOZ09eTpEOG+/4tPsbPUqVsIup1inwxIFsnq+3mXIa+0GKuUq/ieF7cf2z1oa6k/cOLn+ZA Ww5GVwoH7vPHTQW4KVFvLYxB8Jaz/1beliQWvM+ltdz8f2sd+lrNwXf3w8Pz0mbOnJYPMpdai4t V62uyT6aFhLOelhf7nykj8cxiU63MOBcmp1tFvYGVU6dVJrYuqEEfkVkpsI0UzSsg87+cPVUoA0 hfJueWHYegEhmLrmHi62claUThcInw7XoGKBqR1xJU1Vtdxkw7KicWO2R3BOcpnwOi63BmwX1Tz 3tdf55IfK1uQB4ucP2zfMAef6HizLBMyuqXNaT9RcZWr5Uaghs5o/3IZtSTMY8HNXKuhtKiHnSQ fUIBKx5I1KnjLTQ== X-Mailer: b4 0.13.0 Message-ID: <20250213-vma-v14-0-b29c47ab21f5@google.com> Subject: [PATCH v14 0/8] Rust support for mm_struct, vm_area_struct, and mmap From: Alice Ryhl To: Miguel Ojeda , Matthew Wilcox , Lorenzo Stoakes , Vlastimil Babka , John Hubbard , "Liam R. Howlett" , Andrew Morton , Greg Kroah-Hartman , Arnd Bergmann , Jann Horn , Suren Baghdasaryan Cc: Alex Gaynor , Boqun Feng , Gary Guo , " =?utf-8?q?Bj=C3=B6rn_Roy_Baron?= " , Benno Lossin , Andreas Hindborg , Trevor Gross , linux-kernel@vger.kernel.org, linux-mm@kvack.org, rust-for-linux@vger.kernel.org, Alice Ryhl , Balbir Singh X-Rspam-User: X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: B159340009 X-Stat-Signature: jho3yjns8kubno8rww6b3u7cnxphs1dc X-HE-Tag: 1739444678-734217 X-HE-Meta: U2FsdGVkX1+i+uPQIK3tdwbl/4E25VSY19WtO0nMYUa09eTJ9J6653TrKxO8TCGVSNKlaWnwImaN24mOBtstM9KCIQpqAZvjfUEmXYgsOiIF8uiHgGg/lW5GBuo37IJMjYNya5H22kp1d9fOBwaE5QKs3bwa9Yuz4XZ85Zh5eltABuldlHtMi3T5M/84S3jVbRScNjyE/zBjbinT6rCm3xJ9pPli29slpW02f9FzFJke/f0CXPTFJ89PXja4itoX3k84NIOCHLmTiUgm1hR43vh8nGWyUMytCQGEfWpzzc8czWn+gJaGzbJwLYM7JOxzv7V42SgQPGm/A9tyn8WsV+Agii2K8ktGoqkWej79dM3+IP/ZF0oZXhIdiNMRr3ZVMmFk/Scpq0QKVo4LGmCgslml5k5XewuxPiJ1OmJteRuhRItO6Y0LnyBtU5yA4OM54XiOuqBP/kXQf67VHQah7hwagQtxqtv+KqxPCYx3uVf6W4ZHjxB9/NCh6tKlPjtEwTs2xgdtlfIeF+4uAgopednupieB7w7UBxN4qGMtfrrPQetbw/Cl9re3aPZ0AYPj+AUW91gG0VELx/GoZGELNQouuLxGhgJKXcB5cOjRiUZ3YbD0bQLnkZVMUmaxO+1Ck3rzn7ovNHNO4EIdgh+AvDCkNpLAitMVKOHXI/0C8+rtKAv1v8g0tWz3WKEQzSSQpm1bp9m8TuNEhaeJmZIhqWJnCYMn9MuvzzwQe5kdv0olPuy+K1LdFC9H7iXMhC3sFPXTE8kUrirR0UHcPLCnerBJnJPJrGRBayas1E52az9mcdytZxTw+iacHCONApvjO+cBNv/rAB61g/oPl70dSbV4gmGB7MnZuCPGnXbU/T3Ko0x3CpOUvgzSJu3Lehi/nkpEwBYkjBW7EoY5wuLIkgeeRFSrxMHDcvk9+247D1oWfzJywB8nYbo/5SG4NkkPO6nNWO6/5i6oM2WmWUm 7GV/KR/y /SFqsSd1Ps2UtKK6lEoSQ6gqif06LFLDgwCldoiYc/0vJ7TYeWtpBm67WvP4Wujxda6OTYdgTbRMsHjk8sdW+ZM/+HArsbFyJDIw+Yzbbh6wXGdshIqSTRT9W2mdNo7y6bTa5AhKV8bZzgnEDErRMDObcq3dOXTnjd9e5vr+chMw1H1qEvOCJWO8AEXyQDo4nIhlbwey3JcJrP3nAn/kY4uchzKBqtC1kfPByP0eDLaG+CRAYQk1tjSr5BwuLHnf/cf7bNtarrzTR/nw+5ZykqtMtxWRXhHpYPqYuECZSwSWECHiI0OrUSJsxigzRlUC9EQSB7zKV/6A9ttqZzXssdxgLlMnTev8p8PBTk/HCcF/DLD2GCHAdU7fb/2IC0mptVJRAnsWyBbJ8VOmIwFVxgL8oqnb4agHfoXFiI2NY+GZ4kwCesg6Jg9xu7OxE5meNy4bLBPxOv43kif/+tFepLZ+pPHKR+2QQgj7MDbgkTHpoxB8ekJJA40Qx80BfuftfyXB+i0XtpOmMpSAQcKAY/l7GKeIYC3RoJPZwDUryz5NDguC0qPIGsgAUr8/LWv26ozUv5otnPsDATTLoconb3HI1kh3NzhYYv70PbhDtpCD6Ttr6W8BcMoh6eizRUc1Im6/FY7rwGK+/mQUbTDfX/4GgUuIRSYdq28kCsARRUiD5thTiHJsiCEMlU8RKDnh68JzPQnr7Sb0+pXU= X-Bogosity: Ham, tests=bogofilter, spamicity=0.244716, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: This updates the vm_area_struct support to use the approach we discussed at LPC where there are several different Rust wrappers for vm_area_struct depending on the kind of access you have to the vma. Each case allows a different set of operations on the vma. MM folks, what do you prefer I do for the MAINTAINERS file? Would you prefer that I add these files under MEMORY MANAGEMENT, or would you like a separate entry similar to the BLOCK LAYER DEVICE DRIVER API [RUST] entry? Or something else? Signed-off-by: Alice Ryhl Acked-by: Liam R. Howlett --- Changes in v14: - Rename VmArea* to Vma* as suggested by Liam. - Update a few comments in patch 02. - Link to v13: https://lore.kernel.org/r/20250203-vma-v13-0-2b998268a396@google.com Changes in v13: - Rebase on v6.14-rc1. - Remove casts that are now unnecessary due to improved Rust/C integer type mappings. - Link to v12: https://lore.kernel.org/r/20250115-vma-v12-0-375099ae017a@google.com Changes in v12: - Add additional documentation to modules at the top of mm.rs and virt.rs. - Explain why the name Mm is used in commit message of patch 1. - Expand zap_page_range_single with docs suggested by Lorenzo. - Update safety comment in vm_insert_page - Mention that VmAreaMixedMap is identical to VmAreaRef except for VM_MIXEDMAP. - Update docs for as_mixedmap_vma. - Add additional docs for VmAreaNew struct. - Rename `get_read` -> `readable` and equivalent for write/exec. - Use mut pointers for `from_raw` for VMAs. - Update safety comment for fops_mmap. - Add additional docs for MiscDevice::mmap. - Don't introduce and immediately delete mmgrab_current. - Reduce active_pid_ns comment at Andreas's suggestion and link to get_pid_ns. - Fix documentation test in rust/kernel/task.rs. - Fix warning about unused variables in lock_vma_under_rcu when CONFIG_PER_VMA_LOCK=n. - Fix minor typos. - Link to v11: https://lore.kernel.org/r/20241211-vma-v11-0-466640428fc3@google.com Changes in v11: - Add accessor for the vm_mm field of vm_area_struct. - Pass the file to MiscDevice::mmap for consistency with https://lore.kernel.org/r/20241210-miscdevice-file-param-v3-1-b2a79b666dc5@google.com - Link to v10: https://lore.kernel.org/r/20241129-vma-v10-0-4dfff05ba927@google.com Changes in v10: - Update docs for `set_io`. - Check address in `zap_page_range_single`. - Completely redo the last patch. - Link to v9: https://lore.kernel.org/r/20241122-vma-v9-0-7127bfcdd54e@google.com Changes in v9: - Be more explicit about VmAreaNew being used with f_ops->mmap(). - Point out that clearing VM_MAYWRITE is irreversible. - Use __vm_flags to set the flags. - Use as_ and into_ prefixes for conversions. - Update lock_vma_under_rcu docs and commit msg - Mention that VmAreaRef::end is exclusive. - Reword docs for zap_page_range_single. - Minor fixes to flag docs. - Add way to access current->mm without a refcount increment. - Link to v8: https://lore.kernel.org/r/20241120-vma-v8-0-eb31425da66b@google.com Changes in v8: - Split series into more commits to ease review. - Improve read locks based on Lorenzo's doc: either the mmap or vma lock can be used. - Get rid of mmap write lock because it's possible to avoid the need for it. - Do not allow invalid flag combinations on VmAreaNew. - Link to v7: https://lore.kernel.org/r/20241014-vma-v7-0-01e32f861195@google.com Changes in v7: - Make the mmap read/write lock guards respect strict owner semantics. - Link to v6: https://lore.kernel.org/r/20241010-vma-v6-0-d89039b6f573@google.com Changes in v6: - Introduce VmArea{Ref,Mut,New} distinction. - Add a second patchset for miscdevice. - Rebase on char-misc-next (currently on v6.12-rc2). - Link to v5: https://lore.kernel.org/r/20240806-vma-v5-1-04018f05de2b@google.com Changes in v5: - Rename VmArea::from_raw_vma to from_raw. - Use Pin for mutable VmArea references. - Go through `ARef::from` in `mmgrab_current`. - Link to v4: https://lore.kernel.org/r/20240802-vma-v4-1-091a87058a43@google.com Changes in v4: - Pull out ARef::into_raw into a separate patch. - Update invariants and struct documentation. - Rename from_raw_mm to from_raw. - Link to v3: https://lore.kernel.org/r/20240801-vma-v3-1-db6c1c0afda9@google.com Changes in v3: - Reorder entries in mm.rs. - Use ARef for mmput_async helper. - Clarify that VmArea requires you to hold the mmap read or write lock. - Link to v2: https://lore.kernel.org/r/20240727-vma-v2-1-ab3e5927dc3a@google.com Changes in v2: - mm.rs is redesigned from scratch making use of AsRef - Add notes about whether destructors may sleep - Rename Area to VmArea - Link to v1: https://lore.kernel.org/r/20240723-vma-v1-1-32ad5a0118ee@google.com --- Alice Ryhl (8): mm: rust: add abstraction for struct mm_struct mm: rust: add vm_area_struct methods that require read access mm: rust: add vm_insert_page mm: rust: add lock_vma_under_rcu mm: rust: add mmput_async support mm: rust: add VmaNew for f_ops->mmap() rust: miscdevice: add mmap support task: rust: rework how current is accessed rust/helpers/helpers.c | 1 + rust/helpers/mm.c | 50 +++++ rust/kernel/lib.rs | 1 + rust/kernel/miscdevice.rs | 44 +++++ rust/kernel/mm.rs | 341 +++++++++++++++++++++++++++++++++ rust/kernel/mm/virt.rs | 471 ++++++++++++++++++++++++++++++++++++++++++++++ rust/kernel/task.rs | 247 ++++++++++++------------ 7 files changed, 1037 insertions(+), 118 deletions(-) --- base-commit: a64dcfb451e254085a7daee5fe51bf22959d52d3 change-id: 20240723-vma-f80119f9fb35 Best regards,