From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fout-b2-smtp.messagingengine.com (fout-b2-smtp.messagingengine.com [202.12.124.145]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2494D1DC988 for ; Mon, 25 Aug 2025 00:28:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=202.12.124.145 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756081692; cv=none; b=SmZ/JnVBcwk/MoZaCLzeuD8dLOX0rW/arxdUrZ8nDl9xOOKW80BAZllv/DkOj13A4wNIpTllC8DkrTAy93NbTbQ86vxm/6WFkbs/Go1vpvL35+1U+YyuXQ7pMsjDMoUlyk/4QRQO6GnKc8TPjZFuDYnoM324HzqsA49/OWZPMyU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756081692; c=relaxed/simple; bh=zm5o8sMY36TrtvuSWjSFZhTqpAqhBSkx+CMcvcQjklY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=QMF8D1kdw1oZyMUXVRRB7cbL9B8bReikiqssK30QIJAjIwoF5ltHihe5+J6VEW5VYxRRzC6K2nJAcD+Vtb2GtYv3PaaF88+d7JAQ1teipaQiRLmRzW9RiVew7T2lJzutCvcFEVrYESicOZaxqgBchIj4Qb3shTnQ73lUZiFhxwY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=sakamocchi.jp; spf=pass smtp.mailfrom=sakamocchi.jp; dkim=pass (2048-bit key) header.d=sakamocchi.jp header.i=@sakamocchi.jp header.b=n/fPLQRV; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b=G0JY6ni1; arc=none smtp.client-ip=202.12.124.145 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=sakamocchi.jp Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sakamocchi.jp Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=sakamocchi.jp header.i=@sakamocchi.jp header.b="n/fPLQRV"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="G0JY6ni1" Received: from phl-compute-04.internal (phl-compute-04.internal [10.202.2.44]) by mailfout.stl.internal (Postfix) with ESMTP id B47731D00045; Sun, 24 Aug 2025 20:28:07 -0400 (EDT) Received: from phl-mailfrontend-01 ([10.202.2.162]) by phl-compute-04.internal (MEProxy); Sun, 24 Aug 2025 20:28:07 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sakamocchi.jp; h=cc:cc:content-type:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:subject:subject:to:to; s=fm3; t=1756081687; x= 1756168087; bh=7assxD8rK9CczJ5BGjPadwI9gaAFx3RtST678EbjCo0=; b=n /fPLQRVTWHbw6qx+X/pUqQ1OKLtzmfGAugnv/2aHGa0QE6bGKzkmuQDvNy4ZQJEn XwnYLxKq+GBdE8TS1APt25O6CMQ1mw8XhugK4PuYEAM275+l7VCkwX6r7O75z2+5 lJeWGWaiO/n+mkR0uxIlJBe5+55wRt6a19yIvntl2ilg9e/7dHqccixFy+7n5/Rg hsdcyJjPGO7TcwOyVj/Io2yeibQIAEaScBNUkBsaQjsueX7nK2NvRFTXrC4QJhsH n/IvzE1WvthWgsGKLm9jxJHPuLx/IQuU0hAGzUyibuakOXxUZfZu1q/gWNOmKrcc W2Kiy5PP8IE0KYF9VNaHA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:subject:subject:to :to:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t= 1756081687; x=1756168087; bh=7assxD8rK9CczJ5BGjPadwI9gaAFx3RtST6 78EbjCo0=; b=G0JY6ni1d1D4Y1fkLFOvSq8r1ed4zyAU1kOv7hjvrriWz7/Ni7o kUJcX+MBb4vEovrnSHtM4MF4ga0HU3nsqOG6zWADuWZ3WyRLpSsNMtExLFJ/vtFF eo86PKlOFNA9SbC1ligiJ6Rqy2XwInU5HVYfiwWAhj3z5o3Gj5x2ocKSfrvz2yLk W8eB7VdXaEJLMPEbNUyZXcJ7uLuqtb8UyDrFJ9R6mT2/ZSi5jNyWdDY1yxrgxg3P hvGH7uYJvx0KYAsPW4iZkTl/LoAJiv439DT+VFL43nEu73/fEWTFqDqWlvs6HHd8 pT+ZP/tq5zpgNp8/Y5hmBHifU/JRFxHo5Bg== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeeffedrtdefgddujedtleehucetufdoteggodetrf dotffvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfurfetoffkrfgpnffqhgenuceu rghilhhouhhtmecufedttdenucenucfjughrpeffhffvvefukfhfgggtuggjsehttdertd dttddvnecuhfhrohhmpefvrghkrghshhhiucfurghkrghmohhtohcuoehoqdhtrghkrghs hhhisehsrghkrghmohgttghhihdrjhhpqeenucggtffrrghtthgvrhhnpeehhffhteetgf ekvdeiueffveevueeftdelhfejieeitedvleeftdfgfeeuudekueenucevlhhushhtvghr ufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehoqdhtrghkrghshhhisehsrg hkrghmohgttghhihdrjhhppdhnsggprhgtphhtthhopedvpdhmohguvgepshhmthhpohhu thdprhgtphhtthhopehlihhnuhigudefleegqdguvghvvghlsehlihhsthhsrdhsohhurh gtvghfohhrghgvrdhnvghtpdhrtghpthhtoheplhhinhhugidqkhgvrhhnvghlsehvghgv rhdrkhgvrhhnvghlrdhorhhg X-ME-Proxy: Feedback-ID: ie8e14432:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sun, 24 Aug 2025 20:28:06 -0400 (EDT) Date: Mon, 25 Aug 2025 09:28:03 +0900 From: Takashi Sakamoto To: linux1394-devel@lists.sourceforge.net Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/3] firewire: ohci: switch to threaded IRQ handler for SelfIDComplete event Message-ID: <20250825002803.GA283774@workstation.local> Mail-Followup-To: linux1394-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org References: <20250823030954.268412-1-o-takashi@sakamocchi.jp> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250823030954.268412-1-o-takashi@sakamocchi.jp> On Sat, Aug 23, 2025 at 12:09:51PM +0900, Takashi Sakamoto wrote: > Hi, > > This patchset replaces the module-local workqueue with a threaded IRQ > handler for handling the SelfIDComplete event in the 1394 OHCI PCI driver. > > The SelfIDComplete event is the first step in maintaining bus topology. > It occurs after a bus reset or when the topology changes, and must be > processed outside the hard IRQ context due to the latency involved in > enumerating the SelfID sequence. Historically, this was handled by a > module-local workqueue with the WQ_MEM_RECLAIM flag. A threaded IRQ > handler offers a cleaner and more reliable solution, leveraging the > kernel's common infrastructure and eliminating the need for maintaining > a custom workqueue. > > Takashi Sakamoto (3): > firewire: ohci: move self_id_complete tracepoint after validating > register > firewire: ohci: use threaded IRQ handler to handle SelfIDComplete > event > firewire: ohci: remove module-local workqueue > > drivers/firewire/ohci.c | 61 +++++++++++++++++------------------------ > 1 file changed, 25 insertions(+), 36 deletions(-) Applied to for-next branch. Regards Takashi Sakamoto