Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2228126/?format=api
{ "id": 2228126, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2228126/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-cifs-client/patch/20260425125426.3855807-9-dhowells@redhat.com/", "project": { "id": 12, "url": "http://patchwork.ozlabs.org/api/1.1/projects/12/?format=api", "name": "Linux CIFS Client", "link_name": "linux-cifs-client", "list_id": "linux-cifs.vger.kernel.org", "list_email": "linux-cifs@vger.kernel.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20260425125426.3855807-9-dhowells@redhat.com>", "date": "2026-04-25T12:54:12", "name": "[v3,08/19] netfs: Fix write streaming disablement if fd open O_RDWR", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "5c62366e52ee035f89f32377adca37052718d8ec", "submitter": { "id": 59, "url": "http://patchwork.ozlabs.org/api/1.1/people/59/?format=api", "name": "David Howells", "email": "dhowells@redhat.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-cifs-client/patch/20260425125426.3855807-9-dhowells@redhat.com/mbox/", "series": [ { "id": 501437, "url": "http://patchwork.ozlabs.org/api/1.1/series/501437/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-cifs-client/list/?series=501437", "date": "2026-04-25T12:54:04", "name": "netfs: Miscellaneous fixes", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/501437/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2228126/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2228126/checks/", "tags": {}, "headers": { "Return-Path": "\n <linux-cifs+bounces-11117-incoming=patchwork.ozlabs.org@vger.kernel.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "linux-cifs@vger.kernel.org" ], "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256\n header.s=mimecast20190719 header.b=NWGFPU7t;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.232.135.74; helo=sto.lore.kernel.org;\n envelope-from=linux-cifs+bounces-11117-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com\n header.b=\"NWGFPU7t\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=170.10.129.124", "smtp.subspace.kernel.org;\n dmarc=pass (p=quarantine dis=none) header.from=redhat.com", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=redhat.com" ], "Received": [ "from sto.lore.kernel.org (sto.lore.kernel.org [172.232.135.74])\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 4g2qZl3bXHz1y2d\n\tfor <incoming@patchwork.ozlabs.org>; Sat, 25 Apr 2026 22:56:15 +1000 (AEST)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sto.lore.kernel.org (Postfix) with ESMTP id 93CA63012846\n\tfor <incoming@patchwork.ozlabs.org>; Sat, 25 Apr 2026 12:55:44 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 57435396D29;\n\tSat, 25 Apr 2026 12:55:19 +0000 (UTC)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.129.124])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id E1BA1395D8C\n\tfor <linux-cifs@vger.kernel.org>; Sat, 25 Apr 2026 12:55:17 +0000 (UTC)", "from mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com\n (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by\n relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3,\n cipher=TLS_AES_256_GCM_SHA384) id us-mta-543-G83z7RghPU6rZeiFNRYGow-1; Sat,\n 25 Apr 2026 08:55:15 -0400", "from mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com\n (mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.93])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby mx-prod-mc-01.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS\n id C5C11195608F;\n\tSat, 25 Apr 2026 12:55:13 +0000 (UTC)", "from warthog.procyon.org.com (unknown [10.44.48.17])\n\tby mx-prod-int-06.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP\n id C1BFF1801558;\n\tSat, 25 Apr 2026 12:55:10 +0000 (UTC)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1777121719; cv=none;\n b=YnpsfWYwRr1aHpxKN+3qSJz7bj6RXULlpM9dqbOl3BdpMAEULwdkOR6a/9b0pKLUOHvkJLEAVQzzGpYMqsuwhc7iBeB6t4F/93ei25+ta6LDDaFjCAkeCJVFGIUFKzMI54/ciY2AOhQBw8mzbrY0vFS0N2oasxrFNyfoHS0i2n4=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1777121719; c=relaxed/simple;\n\tbh=ar/1916KJK1hP9MglWUn/gBgMrjkAt6aacrmlpZbKGc=;\n\th=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n\t MIME-Version;\n b=BAxoFJ/LI8Yn+1bEf6J8bT/dyECtYd1VLAFv2akKGYHHJjrNla6kDDNjC8WurPs7/pOBxZWQuoPzJHaAZg6TGY2sql5nWujcZ+FF8PP6d/fS7cOcVx0R6ybdT+yKF211qQQVlMDkK8YTeoTDZ9dJ4XlvTAIpyUK2QJ8uPJz7E8s=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=quarantine dis=none) header.from=redhat.com;\n spf=pass smtp.mailfrom=redhat.com;\n dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com\n header.b=NWGFPU7t; arc=none smtp.client-ip=170.10.129.124", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1777121717;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\t to:to:cc:cc:mime-version:mime-version:\n\t content-transfer-encoding:content-transfer-encoding:\n\t in-reply-to:in-reply-to:references:references;\n\tbh=16oz/EvePMB/RKF9sHxIwhF0EbAmx1Zkbc6oHlQ4AMU=;\n\tb=NWGFPU7tShr8tbEJAKKLQZs/2O6m8DLWieH6RrBON7NCPjPwhwGx+ufzIM6P5TyJq220RB\n\tTXu0e/xOmkSK/vT/L3ybFsz7HFf0KYz490RSfgLBLlE5IwDPFaRr8wwkbpwUhfb6Zs9hY5\n\tYdLHwhDZr4a7XH8UeT4m6MEIfHDpY3k=", "X-MC-Unique": "G83z7RghPU6rZeiFNRYGow-1", "X-Mimecast-MFC-AGG-ID": "G83z7RghPU6rZeiFNRYGow_1777121713", "From": "David Howells <dhowells@redhat.com>", "To": "Christian Brauner <christian@brauner.io>", "Cc": "David Howells <dhowells@redhat.com>,\n\tPaulo Alcantara <pc@manguebit.org>,\n\tnetfs@lists.linux.dev,\n\tlinux-afs@lists.infradead.org,\n\tlinux-cifs@vger.kernel.org,\n\tceph-devel@vger.kernel.org,\n\tlinux-fsdevel@vger.kernel.org,\n\tlinux-kernel@vger.kernel.org,\n\tMatthew Wilcox <willy@infradead.org>", "Subject": "[PATCH v3 08/19] netfs: Fix write streaming disablement if fd open\n O_RDWR", "Date": "Sat, 25 Apr 2026 13:54:12 +0100", "Message-ID": "<20260425125426.3855807-9-dhowells@redhat.com>", "In-Reply-To": "<20260425125426.3855807-1-dhowells@redhat.com>", "References": "<20260425125426.3855807-1-dhowells@redhat.com>", "Precedence": "bulk", "X-Mailing-List": "linux-cifs@vger.kernel.org", "List-Id": "<linux-cifs.vger.kernel.org>", "List-Subscribe": "<mailto:linux-cifs+subscribe@vger.kernel.org>", "List-Unsubscribe": "<mailto:linux-cifs+unsubscribe@vger.kernel.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-Scanned-By": "MIMEDefang 3.4.1 on 10.30.177.93" }, "content": "In netfs_perform_write(), \"write streaming\" (the caching of dirty data in\ndirty but !uptodate folios) is performed to avoid the need to read data\nthat is just going to get immediately overwritten. However, this is/will\nbe disabled in three circumstances: if the fd is open O_RDWR, if fscache is\nin use (as we need to round out the blocks for DIO) or if content\nencryption is enabled (again for rounding out purposes).\n\nThe idea behind disabling it if the fd is open O_RDWR is that we'd need to\nflush the write-streaming page before we could read the data, particularly\nthrough mmap. But netfs now fills in the gaps if ->read_folio() is called\non the page, so that is unnecessary. Further, this doesn't actually work\nif a separate fd is open for reading.\n\nFix this by removing the check for O_RDWR, thereby allowing streaming\nwrites even when we might read.\n\nThis caused a number of problems with the generic/522 xfstest, but those\nare now fixed.\n\nFixes: c38f4e96e605 (\"netfs: Provide func to copy data to pagecache for buffered write\")\nSigned-off-by: David Howells <dhowells@redhat.com>\ncc: Paulo Alcantara <pc@manguebit.org>\ncc: Matthew Wilcox <willy@infradead.org>\ncc: netfs@lists.linux.dev\ncc: linux-fsdevel@vger.kernel.org\n---\n fs/netfs/buffered_write.c | 7 ++-----\n 1 file changed, 2 insertions(+), 5 deletions(-)", "diff": "diff --git a/fs/netfs/buffered_write.c b/fs/netfs/buffered_write.c\nindex 165e4a3c8b62..c7b49b38a710 100644\n--- a/fs/netfs/buffered_write.c\n+++ b/fs/netfs/buffered_write.c\n@@ -281,12 +281,9 @@ ssize_t netfs_perform_write(struct kiocb *iocb, struct iov_iter *iter,\n \n \t\t/* We don't want to do a streaming write on a file that loses\n \t\t * caching service temporarily because the backing store got\n-\t\t * culled and we don't really want to get a streaming write on\n-\t\t * a file that's open for reading as ->read_folio() then has to\n-\t\t * be able to flush it.\n+\t\t * culled.\n \t\t */\n-\t\tif ((file->f_mode & FMODE_READ) ||\n-\t\t netfs_is_cache_enabled(ctx)) {\n+\t\tif (netfs_is_cache_enabled(ctx)) {\n \t\t\tif (finfo) {\n \t\t\t\tnetfs_stat(&netfs_n_wh_wstream_conflict);\n \t\t\t\tgoto flush_content;\n", "prefixes": [ "v3", "08/19" ] }