Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2216331/?format=api
{ "id": 2216331, "url": "http://patchwork.ozlabs.org/api/patches/2216331/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-ext4/patch/20260326095354.16340-75-jack@suse.cz/", "project": { "id": 8, "url": "http://patchwork.ozlabs.org/api/projects/8/?format=api", "name": "Linux ext4 filesystem development", "link_name": "linux-ext4", "list_id": "linux-ext4.vger.kernel.org", "list_email": "linux-ext4@vger.kernel.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260326095354.16340-75-jack@suse.cz>", "list_archive_url": null, "date": "2026-03-26T09:54:27", "name": "[33/42] fs: Provide functions for handling mapping_metadata_bhs directly", "commit_ref": null, "pull_url": null, "state": "not-applicable", "archived": false, "hash": "44fb1917e6701e8684d8cfd47aec05cd6b510b33", "submitter": { "id": 363, "url": "http://patchwork.ozlabs.org/api/people/363/?format=api", "name": "Jan Kara", "email": "jack@suse.cz" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-ext4/patch/20260326095354.16340-75-jack@suse.cz/mbox/", "series": [ { "id": 497561, "url": "http://patchwork.ozlabs.org/api/series/497561/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-ext4/list/?series=497561", "date": "2026-03-26T09:53:55", "name": "fs: Move metadata bh tracking from address_space", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/497561/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2216331/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2216331/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "\n <SRS0=UF9K=B2=vger.kernel.org=linux-ext4+bounces-15429-patchwork-incoming=ozlabs.org@ozlabs.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "linux-ext4@vger.kernel.org" ], "Delivered-To": [ "patchwork-incoming@legolas.ozlabs.org", "patchwork-incoming@ozlabs.org" ], "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=suse.cz header.i=@suse.cz header.a=rsa-sha256\n header.s=susede2_rsa header.b=N96OunMJ;\n\tdkim=pass header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT;\n\tdkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.a=rsa-sha256 header.s=susede2_rsa header.b=N96OunMJ;\n\tdkim=neutral header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=ozlabs.org\n (client-ip=2404:9400:2221:ea00::3; helo=mail.ozlabs.org;\n envelope-from=srs0=uf9k=b2=vger.kernel.org=linux-ext4+bounces-15429-patchwork-incoming=ozlabs.org@ozlabs.org;\n receiver=patchwork.ozlabs.org)", "gandalf.ozlabs.org;\n arc=pass smtp.remote-ip=172.234.253.10 arc.chain=subspace.kernel.org", "gandalf.ozlabs.org;\n dmarc=none (p=none dis=none) header.from=suse.cz", "gandalf.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=suse.cz header.i=@suse.cz header.a=rsa-sha256\n header.s=susede2_rsa header.b=N96OunMJ;\n\tdkim=pass header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT;\n\tdkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.a=rsa-sha256 header.s=susede2_rsa header.b=N96OunMJ;\n\tdkim=neutral header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT;\n\tdkim-atps=neutral", "gandalf.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.234.253.10; helo=sea.lore.kernel.org;\n envelope-from=linux-ext4+bounces-15429-patchwork-incoming=ozlabs.org@vger.kernel.org;\n receiver=ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=\"N96OunMJ\";\n\tdkim=permerror (0-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=\"UA96zSZT\";\n\tdkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=\"N96OunMJ\";\n\tdkim=permerror (0-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=\"UA96zSZT\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=195.135.223.131", "smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=suse.cz", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=suse.cz", "smtp-out2.suse.de;\n\tdkim=pass header.d=suse.cz header.s=susede2_rsa header.b=N96OunMJ;\n\tdkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=UA96zSZT" ], "Received": [ "from mail.ozlabs.org (mail.ozlabs.org [IPv6:2404:9400:2221:ea00::3])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fhKLZ70rPz1y1G\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 26 Mar 2026 21:11:34 +1100 (AEDT)", "from mail.ozlabs.org (mail.ozlabs.org [IPv6:2404:9400:2221:ea00::3])\n\tby gandalf.ozlabs.org (Postfix) with ESMTP id 4fhKLZ6XGxz4wBB\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 26 Mar 2026 21:11:34 +1100 (AEDT)", "by gandalf.ozlabs.org (Postfix)\n\tid 4fhKLZ6SQ5z4wB7; Thu, 26 Mar 2026 21:11:34 +1100 (AEDT)", "from sea.lore.kernel.org (sea.lore.kernel.org [172.234.253.10])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby gandalf.ozlabs.org (Postfix) with ESMTPS id 4fhKLW2GcLz4wBB\n\tfor <patchwork-incoming@ozlabs.org>; Thu, 26 Mar 2026 21:11:31 +1100 (AEDT)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 508AB3182640\n\tfor <patchwork-incoming@ozlabs.org>; Thu, 26 Mar 2026 10:00:15 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id D1D333BFE31;\n\tThu, 26 Mar 2026 09:57:23 +0000 (UTC)", "from smtp-out2.suse.de (smtp-out2.suse.de [195.135.223.131])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 942E73CCFDE\n\tfor <linux-ext4@vger.kernel.org>; Thu, 26 Mar 2026 09:57:19 +0000 (UTC)", "from imap1.dmz-prg2.suse.org (imap1.dmz-prg2.suse.org\n [IPv6:2a07:de40:b281:104:10:150:64:97])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby smtp-out2.suse.de (Postfix) with ESMTPS id 8F2025BDC1;\n\tThu, 26 Mar 2026 09:55:01 +0000 (UTC)", "from imap1.dmz-prg2.suse.org (localhost [127.0.0.1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 7B79A4A0A7;\n\tThu, 26 Mar 2026 09:55:01 +0000 (UTC)", "from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167])\n\tby imap1.dmz-prg2.suse.org with ESMTPSA\n\tid fFMjHnUCxWnyYAAAD6G6ig\n\t(envelope-from <jack@suse.cz>); Thu, 26 Mar 2026 09:55:01 +0000", "by quack3.suse.cz (Postfix, from userid 1000)\n\tid A0AF4A0C77; Thu, 26 Mar 2026 10:54:49 +0100 (CET)" ], "ARC-Seal": [ "i=2; a=rsa-sha256; d=ozlabs.org; s=201707; t=1774519894; cv=pass;\n\tb=aqwR6Rzdo8a7NUsr1QfQ0UdeVUpEs+yO/mLRhVKZBkmLU0YvtiyUKfVOJ70HmTrOMfdXtCt2vG9R+pa6Extlw1jERqElSdU2EBNKqm8jKN1ut5fE9+sUDNYRFeBijfrkm3knu+0wUBUtVKTvj5fuTxW4dSPk156whm/RirDUQ+/ZLsu2+BLyYYBCR2182auE/lLycXZrB6oPWspe9/+RVwf3fp3A5UaKP9tC14pcPoYr1ejmB9yBOILTbqSQV3m1Gqa69qwT3auCAUjazqjVajdzLxMOfaLbUn/y2ef0M97D56juDFKHgDVkXk48axLUydpxvIPeuITgfhm8Lzl7Zw==", "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1774519043; cv=none;\n b=KjWPQoleUMyUBDCUyC2J2IFradyhffl5Z41dhVsQ0yPe+5vPoEoyll0kIOjhoMa2Lnsjm4o5DCeKE6zfAc1cGOaIOYcOI60Z11e36ZoOmqgxXcG/+xp+w9vNBNtjxXB9EcWR32bohug0GljJg/RRTdEMrwjTcnf1K8O4jhB3giQ=" ], "ARC-Message-Signature": [ "i=2; a=rsa-sha256; d=ozlabs.org; s=201707;\n\tt=1774519894; c=relaxed/relaxed;\n\tbh=4dA7bzxeJg32q4HGmMXYWKaVsPPNa/KgHdhJnJEenLY=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=KLHc38qOuMPZpDNE4mRFdbcey9a0MaI/d9aGY8LXTsSveNgDypj0eL+Fs1XtNw5/oHkv4rHUbbQe+k7QoxMoWwij4/d8i6AADqTvNOwdTM3xwj3fcJjrHPuYwbJpJoh0VxpxoE4Eej0sJU4CcdkvFU/yvpdGjMEgzCKE/vKFmN1AfH7/N4ZfgZ/Y3h8ajb1FnVIFWHcd4Wg5RrLUxAj48KondJtpOxEjCs46Bkz9gGi1/hS5luhFV5jdspTtRns2aVzRFToe0Dlf0sdgVLLLCkUAbwpkplRyx6fYivKs7uzxUzD3BxApIXMnthwiJiOW6b8qhAXfAJxIGQ/YPLfA2A==", "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1774519043; c=relaxed/simple;\n\tbh=sXhxY5oIVAvA3ZZOll63xAi+RxkOdCbAxoLzbD6vFBo=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=kXJukEeZ1X+nzuI/R6ywvZn8mL5I+1ZNgF0lKG3rZCnmMk5nycAKb0aChVCqgAAOme8mnJI8LvV1tnjg27YInWViGpMdVyMtTf+1FBXnBZRvr7RvgFGp6A/yzltH+/myE/DNYjnFmGeGO9ME+R86MiD+Amk/Wp8nuDaOy2+z0F4=" ], "ARC-Authentication-Results": [ "i=2; gandalf.ozlabs.org;\n dmarc=none (p=none dis=none) header.from=suse.cz; dkim=pass (1024-bit key;\n unprotected) header.d=suse.cz header.i=@suse.cz header.a=rsa-sha256\n header.s=susede2_rsa header.b=N96OunMJ;\n dkim=pass header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT;\n dkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.a=rsa-sha256 header.s=susede2_rsa header.b=N96OunMJ;\n dkim=neutral header.d=suse.cz header.i=@suse.cz header.a=ed25519-sha256\n header.s=susede2_ed25519 header.b=UA96zSZT; dkim-atps=neutral;\n spf=pass (client-ip=172.234.253.10; helo=sea.lore.kernel.org;\n envelope-from=linux-ext4+bounces-15429-patchwork-incoming=ozlabs.org@vger.kernel.org;\n receiver=ozlabs.org) smtp.mailfrom=vger.kernel.org", "i=1; smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=suse.cz;\n spf=pass smtp.mailfrom=suse.cz;\n dkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=N96OunMJ;\n dkim=permerror (0-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=UA96zSZT;\n dkim=pass (1024-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=N96OunMJ;\n dkim=permerror (0-bit key) header.d=suse.cz header.i=@suse.cz\n header.b=UA96zSZT; arc=none smtp.client-ip=195.135.223.131" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz;\n s=susede2_rsa;\n\tt=1774518901;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n\t mime-version:mime-version:\n\t content-transfer-encoding:content-transfer-encoding:\n\t in-reply-to:in-reply-to:references:references;\n\tbh=4dA7bzxeJg32q4HGmMXYWKaVsPPNa/KgHdhJnJEenLY=;\n\tb=N96OunMJKejP0kYY3O/3COuZ88wamClfurT3YbNFIVmzBY4NQ4SEU8E/xDB8QRuqIKcnhM\n\t3+LQgCFKQ9bCEsuaVkmwN/Pz7+ggAbPTnVknbuZHDzzbYvmqJg4oLVnIRY/lyJ0J2MVHpr\n\tnCxicR9mz5KvMDfJU67h/6JVSmAJMSU=", "v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz;\n\ts=susede2_ed25519; t=1774518901;\n\th=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n\t mime-version:mime-version:\n\t content-transfer-encoding:content-transfer-encoding:\n\t in-reply-to:in-reply-to:references:references;\n\tbh=4dA7bzxeJg32q4HGmMXYWKaVsPPNa/KgHdhJnJEenLY=;\n\tb=UA96zSZThOauNjseqcwNekij7l0cvJwO+cnqGB1r7HGOUrCSvqYjJkeyWY/+WIeqB0LmvR\n\tyWyV9eGPNxb2AzDw==", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz;\n s=susede2_rsa;\n\tt=1774518901;\n h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n\t mime-version:mime-version:\n\t content-transfer-encoding:content-transfer-encoding:\n\t in-reply-to:in-reply-to:references:references;\n\tbh=4dA7bzxeJg32q4HGmMXYWKaVsPPNa/KgHdhJnJEenLY=;\n\tb=N96OunMJKejP0kYY3O/3COuZ88wamClfurT3YbNFIVmzBY4NQ4SEU8E/xDB8QRuqIKcnhM\n\t3+LQgCFKQ9bCEsuaVkmwN/Pz7+ggAbPTnVknbuZHDzzbYvmqJg4oLVnIRY/lyJ0J2MVHpr\n\tnCxicR9mz5KvMDfJU67h/6JVSmAJMSU=", "v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz;\n\ts=susede2_ed25519; t=1774518901;\n\th=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc:\n\t mime-version:mime-version:\n\t content-transfer-encoding:content-transfer-encoding:\n\t in-reply-to:in-reply-to:references:references;\n\tbh=4dA7bzxeJg32q4HGmMXYWKaVsPPNa/KgHdhJnJEenLY=;\n\tb=UA96zSZThOauNjseqcwNekij7l0cvJwO+cnqGB1r7HGOUrCSvqYjJkeyWY/+WIeqB0LmvR\n\tyWyV9eGPNxb2AzDw==" ], "From": "Jan Kara <jack@suse.cz>", "To": "<linux-fsdevel@vger.kernel.org>", "Cc": "<linux-block@vger.kernel.org>,\n\tChristian Brauner <brauner@kernel.org>,\n\tAl Viro <viro@ZenIV.linux.org.uk>,\n\t<linux-ext4@vger.kernel.org>,\n\tTed Tso <tytso@mit.edu>,\n\t\"Tigran A. Aivazian\" <aivazian.tigran@gmail.com>,\n\tDavid Sterba <dsterba@suse.com>,\n\tOGAWA Hirofumi <hirofumi@mail.parknet.co.jp>,\n\tMuchun Song <muchun.song@linux.dev>,\n\tOscar Salvador <osalvador@suse.de>,\n\tDavid Hildenbrand <david@kernel.org>,\n\tlinux-mm@kvack.org,\n\tlinux-aio@kvack.org,\n\tBenjamin LaHaise <bcrl@kvack.org>,\n\tJan Kara <jack@suse.cz>", "Subject": "[PATCH 33/42] fs: Provide functions for handling mapping_metadata_bhs\n directly", "Date": "Thu, 26 Mar 2026 10:54:27 +0100", "Message-ID": "<20260326095354.16340-75-jack@suse.cz>", "X-Mailer": "git-send-email 2.51.0", "In-Reply-To": "<20260326082428.31660-1-jack@suse.cz>", "References": "<20260326082428.31660-1-jack@suse.cz>", "Precedence": "bulk", "X-Mailing-List": "linux-ext4@vger.kernel.org", "List-Id": "<linux-ext4.vger.kernel.org>", "List-Subscribe": "<mailto:linux-ext4+subscribe@vger.kernel.org>", "List-Unsubscribe": "<mailto:linux-ext4+unsubscribe@vger.kernel.org>", "MIME-Version": "1.0", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=12578; i=jack@suse.cz;\n h=from:subject; bh=sXhxY5oIVAvA3ZZOll63xAi+RxkOdCbAxoLzbD6vFBo=;\n b=owGbwMvMwME4Z+4qdvsUh5uMp9WSGDKPMvkVKLOJyIhbnjBb/TDx+edE52KrmsMaYTfW6i536\n f7XpSPdyWjMwsDIwSArpsiyOvKi9rV5Rl1bQzVkYAaxMoFMYeDiFICJ/P/N/k/nW16liHRy48pk\n sVfOaq89G91lNjzLOqfE9LIq/mGO5GOJXJ0jFz2k0xRLOO9ltwu7Mj9WXbJHIiTr5WP+LXuTnnq\n bp6/JcGiO/Tsp8Ai7kNCrlJmxaiLbRb0U2h/sSSspN7GVzvfMCmvMaL6muuAIe7z66dQ9F5dMKp\n fPlj5damykKyMvFjzrSNyJM8V9mfsiHC9kLpmgH/yP/8oqe442Bcamyd1ef8409LE/tnuRvMR3l\n +EXn6ta0wOfTlbb6Ze4bBV75pVN3QonpHZNid+zU/jpluUf3m3k5LjJKPz1U3nmZqPcI3On+1XI\n J7U8lyrj4HXkY/ng4Mkwe9tra7lrgRd8p6plLtwoEyUAAA==", "X-Developer-Key": "i=jack@suse.cz; a=openpgp;\n fpr=93C6099A142276A28BBE35D815BC833443038D8C", "Content-Transfer-Encoding": "8bit", "X-Rspamd-Action": "no action", "X-Rspamd-Server": "rspamd2.dmz-prg2.suse.org", "X-Spamd-Result": "default: False [-1.51 / 50.00];\n\tBAYES_HAM(-3.00)[100.00%];\n\tSUSPICIOUS_RECIPS(1.50)[];\n\tMID_CONTAINS_FROM(1.00)[];\n\tNEURAL_HAM_LONG(-1.00)[-1.000];\n\tR_MISSING_CHARSET(0.50)[];\n\tNEURAL_HAM_SHORT(-0.20)[-1.000];\n\tR_DKIM_ALLOW(-0.20)[suse.cz:s=susede2_rsa,suse.cz:s=susede2_ed25519];\n\tMIME_GOOD(-0.10)[text/plain];\n\tMX_GOOD(-0.01)[];\n\tDBL_BLOCKED_OPENRESOLVER(0.00)[imap1.dmz-prg2.suse.org:helo,imap1.dmz-prg2.suse.org:rdns];\n\tRCVD_COUNT_THREE(0.00)[3];\n\tDKIM_SIGNED(0.00)[suse.cz:s=susede2_rsa,suse.cz:s=susede2_ed25519];\n\tRBL_SPAMHAUS_BLOCKED_OPENRESOLVER(0.00)[2a07:de40:b281:104:10:150:64:97:from];\n\tFUZZY_RATELIMITED(0.00)[rspamd.com];\n\tMIME_TRACE(0.00)[0:+];\n\tARC_NA(0.00)[];\n\tRCPT_COUNT_TWELVE(0.00)[16];\n\tRCVD_TLS_LAST(0.00)[];\n\tTO_MATCH_ENVRCPT_ALL(0.00)[];\n\tFROM_HAS_DN(0.00)[];\n\tRCVD_VIA_SMTP_AUTH(0.00)[];\n\tTAGGED_RCPT(0.00)[];\n\tRECEIVED_SPAMHAUS_BLOCKED_OPENRESOLVER(0.00)[2a07:de40:b281:106:10:150:64:167:received];\n\tSPAMHAUS_XBL(0.00)[2a07:de40:b281:104:10:150:64:97:from];\n\tR_RATELIMIT(0.00)[to_ip_from(RLhafujjw6m7bafrsz8p45s31g),to(RLpnp1psq57oqao6ocwdpbxen9)];\n\tFREEMAIL_CC(0.00)[vger.kernel.org,kernel.org,ZenIV.linux.org.uk,mit.edu,gmail.com,suse.com,mail.parknet.co.jp,linux.dev,suse.de,kvack.org,suse.cz];\n\tDKIM_TRACE(0.00)[suse.cz:+];\n\tFROM_EQ_ENVFROM(0.00)[];\n\tTO_DN_SOME(0.00)[];\n\tFREEMAIL_ENVRCPT(0.00)[gmail.com]", "X-Rspamd-Queue-Id": "8F2025BDC1", "X-Spam-Score": "-1.51", "X-Spam-Level": "", "X-Spam-Status": "No, score=-1.2 required=5.0 tests=ARC_SIGNED,ARC_VALID,\n\tDKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DMARC_MISSING,\n\tHEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,\n\tSPF_PASS autolearn=disabled version=4.0.1", "X-Spam-Checker-Version": "SpamAssassin 4.0.1 (2024-03-25) on gandalf.ozlabs.org" }, "content": "As part of transition toward moving mapping_metadata_bhs to fs-private\npart of the inode, provide functions for operations on this list\ndirectly instead of going through the inode / mapping.\n\nSigned-off-by: Jan Kara <jack@suse.cz>\n---\n fs/buffer.c | 110 +++++++++++++++++-------------------\n include/linux/buffer_head.h | 44 ++++++++++++---\n 2 files changed, 87 insertions(+), 67 deletions(-)", "diff": "diff --git a/fs/buffer.c b/fs/buffer.c\nindex b0436481d0f1..cbed175f418b 100644\n--- a/fs/buffer.c\n+++ b/fs/buffer.c\n@@ -467,31 +467,25 @@ EXPORT_SYMBOL(mark_buffer_async_write);\n * a successful fsync(). For example, ext2 indirect blocks need to be\n * written back and waited upon before fsync() returns.\n *\n- * The functions mark_buffer_dirty_inode(), fsync_inode_buffers(),\n- * mmb_has_buffers() and invalidate_inode_buffers() are provided for the\n- * management of a list of dependent buffers in mapping_metadata_bhs struct.\n+ * The functions mmb_mark_buffer_dirty(), mmb_sync(), mmb_has_buffers()\n+ * and mmb_invalidate() are provided for the management of a list of dependent\n+ * buffers in mapping_metadata_bhs struct.\n *\n * The locking is a little subtle: The list of buffer heads is protected by\n * the lock in mapping_metadata_bhs so functions coming from bdev mapping\n * (such as try_to_free_buffers()) need to safely get to mapping_metadata_bhs\n * using RCU, grab the lock, verify we didn't race with somebody detaching the\n * bh / moving it to different inode and only then proceeding.\n- *\n- * FIXME: mark_buffer_dirty_inode() is a data-plane operation. It should\n- * take an address_space, not an inode. And it should be called\n- * mark_buffer_dirty_fsync() to clearly define why those buffers are being\n- * queued up.\n- *\n- * FIXME: mark_buffer_dirty_inode() doesn't need to add the buffer to the\n- * list if it is already on a list. Because if the buffer is on a list,\n- * it *must* already be on the right one. If not, the filesystem is being\n- * silly. This will save a ton of locking. But first we have to ensure\n- * that buffers are taken *off* the old inode's list when they are freed\n- * (presumably in truncate). That requires careful auditing of all\n- * filesystems (do it inside bforget()). It could also be done by bringing\n- * b_inode back.\n */\n \n+void mmb_init(struct mapping_metadata_bhs *mmb, struct address_space *mapping)\n+{\n+\tspin_lock_init(&mmb->lock);\n+\tINIT_LIST_HEAD(&mmb->list);\n+\tmmb->mapping = mapping;\n+}\n+EXPORT_SYMBOL(mmb_init);\n+\n static void __remove_assoc_queue(struct mapping_metadata_bhs *mmb,\n \t\t\t struct buffer_head *bh)\n {\n@@ -533,12 +527,12 @@ bool mmb_has_buffers(struct mapping_metadata_bhs *mmb)\n EXPORT_SYMBOL_GPL(mmb_has_buffers);\n \n /**\n- * sync_mapping_buffers - write out & wait upon a mapping's \"associated\" buffers\n- * @mapping: the mapping which wants those buffers written\n+ * mmb_sync - write out & wait upon all buffers in a list\n+ * @mmb: the list of buffers to write\n *\n- * Starts I/O against the buffers at mapping->i_metadata_bhs and waits upon\n- * that I/O. Basically, this is a convenience function for fsync(). @mapping\n- * is a file or directory which needs those buffers to be written for a\n+ * Starts I/O against the buffers in the given list and waits upon\n+ * that I/O. Basically, this is a convenience function for fsync(). @mmb is\n+ * for a file or directory which needs those buffers to be written for a\n * successful fsync().\n *\n * We have conflicting pressures: we want to make sure that all\n@@ -553,9 +547,8 @@ EXPORT_SYMBOL_GPL(mmb_has_buffers);\n * buffer stays on our list until IO completes (at which point it can be\n * reaped).\n */\n-int sync_mapping_buffers(struct address_space *mapping)\n+int mmb_sync(struct mapping_metadata_bhs *mmb)\n {\n-\tstruct mapping_metadata_bhs *mmb = &mapping->i_metadata_bhs;\n \tstruct buffer_head *bh;\n \tint err = 0;\n \tstruct blk_plug plug;\n@@ -626,33 +619,35 @@ int sync_mapping_buffers(struct address_space *mapping)\n \tspin_unlock(&mmb->lock);\n \treturn err;\n }\n-EXPORT_SYMBOL(sync_mapping_buffers);\n+EXPORT_SYMBOL(mmb_sync);\n \n /**\n- * generic_buffers_fsync_noflush - generic buffer fsync implementation\n- * for simple filesystems with no inode lock\n+ * mmb_fsync_noflush - fsync implementation for simple filesystems with\n+ * \t\t metadata buffers list\n *\n * @file:\tfile to synchronize\n+ * @mmb:\tlist of metadata bhs to flush\n * @start:\tstart offset in bytes\n * @end:\tend offset in bytes (inclusive)\n * @datasync:\tonly synchronize essential metadata if true\n *\n- * This is a generic implementation of the fsync method for simple\n- * filesystems which track all non-inode metadata in the buffers list\n- * hanging off the address_space structure.\n+ * This is an implementation of the fsync method for simple filesystems which\n+ * track all non-inode metadata in the buffers list hanging off the @mmb\n+ * structure.\n */\n-int generic_buffers_fsync_noflush(struct file *file, loff_t start, loff_t end,\n-\t\t\t\t bool datasync)\n+int mmb_fsync_noflush(struct file *file, struct mapping_metadata_bhs *mmb,\n+\t\t loff_t start, loff_t end, bool datasync)\n {\n \tstruct inode *inode = file->f_mapping->host;\n \tint err;\n-\tint ret;\n+\tint ret = 0;\n \n \terr = file_write_and_wait_range(file, start, end);\n \tif (err)\n \t\treturn err;\n \n-\tret = sync_mapping_buffers(inode->i_mapping);\n+\tif (mmb)\n+\t\tret = mmb_sync(mmb);\n \tif (!(inode_state_read_once(inode) & I_DIRTY_ALL))\n \t\tgoto out;\n \tif (datasync && !(inode_state_read_once(inode) & I_DIRTY_DATASYNC))\n@@ -669,34 +664,35 @@ int generic_buffers_fsync_noflush(struct file *file, loff_t start, loff_t end,\n \t\tret = err;\n \treturn ret;\n }\n-EXPORT_SYMBOL(generic_buffers_fsync_noflush);\n+EXPORT_SYMBOL(mmb_fsync_noflush);\n \n /**\n- * generic_buffers_fsync - generic buffer fsync implementation\n- * for simple filesystems with no inode lock\n+ * mmb_fsync - fsync implementation for simple filesystems with metadata\n+ * \t buffers list\n *\n * @file:\tfile to synchronize\n+ * @mmb:\tlist of metadata bhs to flush\n * @start:\tstart offset in bytes\n * @end:\tend offset in bytes (inclusive)\n * @datasync:\tonly synchronize essential metadata if true\n *\n- * This is a generic implementation of the fsync method for simple\n- * filesystems which track all non-inode metadata in the buffers list\n- * hanging off the address_space structure. This also makes sure that\n- * a device cache flush operation is called at the end.\n+ * This is an implementation of the fsync method for simple filesystems which\n+ * track all non-inode metadata in the buffers list hanging off the @mmb\n+ * structure. This also makes sure that a device cache flush operation is\n+ * called at the end.\n */\n-int generic_buffers_fsync(struct file *file, loff_t start, loff_t end,\n-\t\t\t bool datasync)\n+int mmb_fsync(struct file *file, struct mapping_metadata_bhs *mmb,\n+\t loff_t start, loff_t end, bool datasync)\n {\n \tstruct inode *inode = file->f_mapping->host;\n \tint ret;\n \n-\tret = generic_buffers_fsync_noflush(file, start, end, datasync);\n+\tret = mmb_fsync_noflush(file, mmb, start, end, datasync);\n \tif (!ret)\n \t\tret = blkdev_issue_flush(inode->i_sb->s_bdev);\n \treturn ret;\n }\n-EXPORT_SYMBOL(generic_buffers_fsync);\n+EXPORT_SYMBOL(mmb_fsync);\n \n /*\n * Called when we've recently written block `bblock', and it is known that\n@@ -717,20 +713,18 @@ void write_boundary_block(struct block_device *bdev,\n \t}\n }\n \n-void mark_buffer_dirty_inode(struct buffer_head *bh, struct inode *inode)\n+void mmb_mark_buffer_dirty(struct buffer_head *bh,\n+\t\t\t struct mapping_metadata_bhs *mmb)\n {\n-\tstruct address_space *mapping = inode->i_mapping;\n-\n \tmark_buffer_dirty(bh);\n \tif (!bh->b_mmb) {\n-\t\tspin_lock(&mapping->i_metadata_bhs.lock);\n-\t\tlist_move_tail(&bh->b_assoc_buffers,\n-\t\t\t\t&mapping->i_metadata_bhs.list);\n-\t\tbh->b_mmb = &mapping->i_metadata_bhs;\n-\t\tspin_unlock(&mapping->i_metadata_bhs.lock);\n+\t\tspin_lock(&mmb->lock);\n+\t\tlist_move_tail(&bh->b_assoc_buffers, &mmb->list);\n+\t\tbh->b_mmb = mmb;\n+\t\tspin_unlock(&mmb->lock);\n \t}\n }\n-EXPORT_SYMBOL(mark_buffer_dirty_inode);\n+EXPORT_SYMBOL(mmb_mark_buffer_dirty);\n \n /**\n * block_dirty_folio - Mark a folio as dirty.\n@@ -797,14 +791,12 @@ bool block_dirty_folio(struct address_space *mapping, struct folio *folio)\n EXPORT_SYMBOL(block_dirty_folio);\n \n /*\n- * Invalidate any and all dirty buffers on a given inode. We are\n+ * Invalidate any and all dirty buffers on a given buffers list. We are\n * probably unmounting the fs, but that doesn't mean we have already\n * done a sync(). Just drop the buffers from the inode list.\n */\n-void invalidate_inode_buffers(struct inode *inode)\n+void mmb_invalidate(struct mapping_metadata_bhs *mmb)\n {\n-\tstruct mapping_metadata_bhs *mmb = &inode->i_data.i_metadata_bhs;\n-\n \tif (mmb_has_buffers(mmb)) {\n \t\tspin_lock(&mmb->lock);\n \t\twhile (!list_empty(&mmb->list))\n@@ -812,7 +804,7 @@ void invalidate_inode_buffers(struct inode *inode)\n \t\tspin_unlock(&mmb->lock);\n \t}\n }\n-EXPORT_SYMBOL(invalidate_inode_buffers);\n+EXPORT_SYMBOL(mmb_invalidate);\n \n /*\n * Create the appropriate buffers when given a folio for data area and\ndiff --git a/include/linux/buffer_head.h b/include/linux/buffer_head.h\nindex 44094fd476f5..e207dcca7a25 100644\n--- a/include/linux/buffer_head.h\n+++ b/include/linux/buffer_head.h\n@@ -205,12 +205,30 @@ struct buffer_head *create_empty_buffers(struct folio *folio,\n void end_buffer_read_sync(struct buffer_head *bh, int uptodate);\n void end_buffer_write_sync(struct buffer_head *bh, int uptodate);\n \n-/* Things to do with buffers at mapping->private_list */\n-void mark_buffer_dirty_inode(struct buffer_head *bh, struct inode *inode);\n-int generic_buffers_fsync_noflush(struct file *file, loff_t start, loff_t end,\n-\t\t\t\t bool datasync);\n-int generic_buffers_fsync(struct file *file, loff_t start, loff_t end,\n-\t\t\t bool datasync);\n+/* Things to do with metadata buffers list */\n+void mmb_mark_buffer_dirty(struct buffer_head *bh, struct mapping_metadata_bhs *mmb);\n+static inline void mark_buffer_dirty_inode(struct buffer_head *bh,\n+\t\t\t\t\t struct inode *inode)\n+{\n+\tmmb_mark_buffer_dirty(bh, &inode->i_data.i_metadata_bhs);\n+}\n+int mmb_fsync_noflush(struct file *file, struct mapping_metadata_bhs *mmb,\n+\t\t loff_t start, loff_t end, bool datasync);\n+static inline int generic_buffers_fsync_noflush(struct file *file,\n+\t\t\t\t\t\tloff_t start, loff_t end,\n+\t\t\t\t\t\tbool datasync)\n+{\n+\treturn mmb_fsync_noflush(file, &file->f_mapping->i_metadata_bhs,\n+\t\t\t\t start, end, datasync);\n+}\n+int mmb_fsync(struct file *file, struct mapping_metadata_bhs *mmb,\n+\t loff_t start, loff_t end, bool datasync);\n+static inline int generic_buffers_fsync(struct file *file,\n+\t\t\t\t\tloff_t start, loff_t end, bool datasync)\n+{\n+\treturn mmb_fsync(file, &file->f_mapping->i_metadata_bhs,\n+\t\t\t start, end, datasync);\n+}\n void clean_bdev_aliases(struct block_device *bdev, sector_t block,\n \t\t\tsector_t len);\n static inline void clean_bdev_bh_alias(struct buffer_head *bh)\n@@ -515,9 +533,18 @@ bool block_dirty_folio(struct address_space *mapping, struct folio *folio);\n \n void buffer_init(void);\n bool try_to_free_buffers(struct folio *folio);\n+void mmb_init(struct mapping_metadata_bhs *mmb, struct address_space *mapping);\n bool mmb_has_buffers(struct mapping_metadata_bhs *mmb);\n-void invalidate_inode_buffers(struct inode *inode);\n-int sync_mapping_buffers(struct address_space *mapping);\n+void mmb_invalidate(struct mapping_metadata_bhs *mmb);\n+int mmb_sync(struct mapping_metadata_bhs *mmb);\n+static inline void invalidate_inode_buffers(struct inode *inode)\n+{\n+\tmmb_invalidate(&inode->i_data.i_metadata_bhs);\n+}\n+static inline int sync_mapping_buffers(struct address_space *mapping)\n+{\n+\treturn mmb_sync(&mapping->i_metadata_bhs);\n+}\n void invalidate_bh_lrus(void);\n void invalidate_bh_lrus_cpu(void);\n bool has_bh_in_lru(int cpu, void *dummy);\n@@ -527,6 +554,7 @@ extern int buffer_heads_over_limit;\n \n static inline void buffer_init(void) {}\n static inline bool try_to_free_buffers(struct folio *folio) { return true; }\n+static inline int mmb_sync(struct mapping_metadata_bhs *mmb) { return 0; }\n static inline void invalidate_inode_buffers(struct inode *inode) {}\n static inline int sync_mapping_buffers(struct address_space *mapping) { return 0; }\n static inline void invalidate_bh_lrus(void) {}\n", "prefixes": [ "33/42" ] }