Received-SPF: pass (google.com: domain of linux-kernel+bounces-79386-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Fri, 23 Feb 2024 17:08:47 -0800
Subject: [PATCHSET 5/6] thread_with_file: cleanups and fixes
From: "Darrick J. Wong" <djwong@kernel.org>
To: akpm@linux-foundation.org, daniel@gluo.nz, kent.overstreet@linux.dev,
 djwong@kernel.org
Cc: linux-xfs@vger.kernel.org, linux-bcachefs@vger.kernel.org,
 linux-kernel@vger.kernel.org
Message-ID: <170873669390.1861696.14195299906744951144.stgit@frogsfrogsfrogs>
In-Reply-To: <20240224010017.GM6226@frogsfrogsfrogs>
References: <20240224010017.GM6226@frogsfrogsfrogs>
User-Agent: StGit/0.19
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Hi all,

This series fixes some problems with the thread_with_file code -- namely
that blocking stdout writes attempt a non-atomic memory allocation while
holding a spinlock.  It also cleans up the ops handling so that we can
support ioctls on the thread_with_file itself.

If you're going to start using this code, I strongly recommend pulling
from my git trees, which are linked below.

This has been running on the djcloud for months with no problems.  Enjoy!
Comments and questions are, as always, welcome.

--D

kernel git tree:
https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=twf-cleanups
---
Commits in this patchset:
 * thread_with_file: allow creation of readonly files
 * thread_with_file: fix various printf problems
 * thread_with_file: create ops structure for thread_with_stdio
 * thread_with_file: allow ioctls against these files
 * thread_with_file: Fix missing va_end()
---
 fs/bcachefs/chardev.c            |   18 ++++--
 include/linux/thread_with_file.h |   20 +++++--
 lib/thread_with_file.c           |  113 ++++++++++++++++++++++++++++++--------
 3 files changed, 115 insertions(+), 36 deletions(-)