From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from invmail4.hynix.com (exvmail4.hynix.com [166.125.252.92]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 92A805695; Thu, 16 Oct 2025 00:46:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=166.125.252.92 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760575612; cv=none; b=KSMEV0T9biTZjbm/2GcEEW90hDMLqWj6U5UD97j+s2AhBNfHqPg9371fmaW/hV2F7EX4xaJxpjndr0B2Y0PslA9eBHqZ2iyaETSWTX0QIx6ggCpOLgnBpJqQ4F0nMXrgsqxtNHemZJHv69Dv+OBKqXfQ2BOFLSF0tgUPARBx+ts= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760575612; c=relaxed/simple; bh=bOKJ6TkZJV7OT6XeerMfxJtiPz1ZRcgBwgHMoF0YAxo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=TmUsu6IiC2zDlZv4qTZzhRi0tuylifAifWBwu+1IE358LoT4A5cKj1iSFeX8TUq2QqeOwIiHw6mozmDgLXu0G6zaucShEOaT9Z59CY0C/BJ78Ng0tmcKt5JuN7iygrNNMGxrE9O2zoA/dbEyQrD5BHMe2L2+4Ts72t5qCqPbaK8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sk.com; spf=pass smtp.mailfrom=sk.com; arc=none smtp.client-ip=166.125.252.92 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sk.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sk.com X-AuditID: a67dfc5b-c45ff70000001609-3b-68f04075064b Date: Thu, 16 Oct 2025 09:46:40 +0900 From: Byungchul Park To: Arnd Bergmann Cc: Mathieu Desnoyers , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, kernel_team@skhynix.com, Linus Torvalds , Damien Le Moal , linux-ide@vger.kernel.org, Andreas Dilger , linux-ext4@vger.kernel.org, Ingo Molnar , Peter Zijlstra , Will Deacon , Thomas Gleixner , Steven Rostedt , Joel Fernandes , Sasha Levin , Daniel Vetter , duyuyang@gmail.com, Johannes Berg , Tejun Heo , Theodore Ts'o , Matthew Wilcox , Dave Chinner , Amir Goldstein , kernel-team@lge.com, linux-mm@kvack.org, Andrew Morton , Michal Hocko , Minchan Kim , Johannes Weiner , vdavydov.dev@gmail.com, SeongJae Park , jglisse@redhat.com, Dennis Zhou , Christoph Lameter , Pekka Enberg , David Rientjes , Vlastimil Babka , ngupta@vflare.org, linux-block@vger.kernel.org, Josef Bacik , linux-fsdevel@vger.kernel.org, Jan Kara , Jeff Layton , Dan Williams , Christoph Hellwig , "Darrick J. Wong" , dri-devel@lists.freedesktop.org, rodrigosiqueiramelo@gmail.com, melissa.srw@gmail.com, hamohammed.sa@gmail.com, harry.yoo@oracle.com, chris.p.wilson@intel.com, Gwan-gyeong Mun , max.byungchul.park@gmail.com, Boqun Feng , Waiman Long , yunseong.kim@ericsson.com, ysk@kzalloc.com, Yeoreum Yun , Netdev , Matthew Brost , her0gyugyu@gmail.com, Jonathan Corbet , Catalin Marinas , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Andy Lutomirski , Sumit Semwal , gustavo@padovan.org, Christian =?iso-8859-1?Q?K=F6nig?= , Andi Shyti , Lorenzo Stoakes , "Liam R. Howlett" , Mike Rapoport , Suren Baghdasaryan , Luis Chamberlain , Petr Pavlu , da.gomez@kernel.org, Sami Tolvanen , "Paul E. McKenney" , Frederic Weisbecker , neeraj.upadhyay@kernel.org, joelagnelf@nvidia.com, Josh Triplett , "Uladzislau Rezki (Sony)" , Lai Jiangshan , qiang.zhang@linux.dev, Juri Lelli , Vincent Guittot , Dietmar Eggemann , Benjamin Segall , Mel Gorman , Valentin Schneider , Chuck Lever , neil@brown.name, okorniev@redhat.com, Dai Ngo , Tom Talpey , trondmy@kernel.org, Anna Schumaker , Kees Cook , Sebastian Andrzej Siewior , Clark Williams , Mark Rutland , ada.coupriediaz@arm.com, kristina.martsenko@arm.com, Kefeng Wang , Mark Brown , Kevin Brodsky , David Woodhouse , Shakeel Butt , Alexei Starovoitov , Zi Yan , Yu Zhao , Baolin Wang , usamaarif642@gmail.com, joel.granados@kernel.org, Wei Yang , Geert Uytterhoeven , tim.c.chen@linux.intel.com, linux , Alexander Shishkin , lillian@star-ark.net, Huacai Chen , francesco@valla.it, guoweikang.kernel@gmail.com, link@vivo.com, Josh Poimboeuf , Masahiro Yamada , Christian Brauner , Thomas =?iso-8859-1?Q?Wei=DFschuh?= , Oleg Nesterov , Mateusz Guzik , Andrii Nakryiko , wangfushuai@baidu.com, linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org, linux-i2c@vger.kernel.org, Linux-Arch , linux-modules@vger.kernel.org, rcu , linux-nfs@vger.kernel.org, linux-rt-devel@lists.linux.dev Subject: Re: [PATCH v17 01/47] llist: move llist_{head,node} definition to types.h Message-ID: <20251016004640.GB2948@system.software.com> References: <20251002081247.51255-1-byungchul@sk.com> <20251002081247.51255-2-byungchul@sk.com> <2025100230-grafted-alias-22a2@gregkh> <63034035-03e4-4184-afce-7e1a897a90e9@efficios.com> <3bbe14af-ccdc-4c78-a7ca-d4ed39fa6b5d@app.fastmail.com> Precedence: bulk X-Mailing-List: rcu@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: <3bbe14af-ccdc-4c78-a7ca-d4ed39fa6b5d@app.fastmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-Brightmail-Tracker: H4sIAAAAAAAAA03Sf0zMYRwHcM/395277euEh8zmMK1NYTafjcwW8zUmpvkDw9F37qaudqeI NSxHbrQkWZfsdCtXXckZw5JkuVVDJqfFdaGdbt1d5IqSy13N+OfZ6/m8n+e954+HIxWd9DxO oz0q6rSqVCUjpaR+2c1lWeu/qpe3NsjAeaaJAvdwHgLDwwkKgqPvWfheH2LgmvcMBd6WTXDz o4uEstJCBLmW2wwUl9kpeD0wToC70kNB8Z05UHotlwgv/QSMVlaz8MlqYmHCnA51/lc0tPY4 aRjwFDLgdpyjocc2QUNHU3s4MFVR8OpRLQ21gXIGrgQ8CCqHB1nwvD1HQKFniIV7TwwIHJea CKgIBkh4UfqShsa83vDW6qfBbnMy0Ob6zsKY6wENbWNtBHzK97MwVBaioeY1BfXfbjHw6+d1 BgIFQXr9cuGHIZ8SbDdsSAhW5JKCoSCsxyNmSrj8Ypnw0ORihbON3axgtmcKd62xgqXBSwjd AwmCvfoCI9iHClnB6O8khA/OBmZ79G7p2hQxVZMl6uLXHZCqzQO/iYw6/vjztlrmNPLJjEjC YX4Vtjieob++68ojI6b4JfhN8AcVMcMvxV1do5PzKH4hvtrvmTTJe2JwUYc+4pl8MnbWVE+e l/OrcbmxJWwpp+DHEK7z/aKnghm4taSPmroci7tCXsKIuLCj8a0QFxlL+A3Y9/Y8E/EsfhFu uu8gIj2Y75FgR/kHZuqhc/FTaxdVgHjTf7Wm/2pN/2rNiKxGCo02K02lSV0Vp87Wao7HHUpP s6PwX6vMGd/zAA117GxGPIeUMnmvdVCtoFVZ+uy0ZoQ5UhklX33Sp1bIU1TZJ0Rd+n5dZqqo b0bRHKWcI185cixFwR9WHRWPiGKGqPubEpxk3mkkRy6F5x69efrPfbufBciiZO+2pAP+vPHB x4eLh6MXZa4sYIp8MduSUnrXxGydaHQ3hhJr2se2BL/s3ZEwPK11z+f27JmXtS0ZVaNV+YsN 6+r73N7O/He2nNl9svl9V4uZl3UlcQcvli5ITDK28yWWU92JGy03+iWxybsyp4/E++8oKb1a tSKW1OlVfwA4H8q/ZwMAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA02Se0xTdxTH87uP37001txVjDeSLKEb6EzwtRlPlCnJJF5NYPsDddlitJGb tQLFtIqgceMZCW4TO9uGVl0HsRIKBQGRihiCo4s6ItipnVqxpvKQMhZpIVBK12LM/Ofkc76P 5PxxWFIWpleyKvVRUaNW5MqxhJJkbi1LOZb2r3J958gn8Kikh4JgoJKCC82NGCpba2gYsNsQ DAUrEcyEzCRUOCIUhHVOBgKzTxmIdDsRGAZ1JDS2lxAw1bKAYfz2GwR6rw+DcayEgknrjwhM w2YGxvp2wsRQFw0RzwgBj6f9CKy+BQJ8PacRhA058GttG4ZQ/30SjPoBBL95PSSMtkTNdudz BN31pRheVV8jweVbCn8FJzHc0Z/BMDF4gYB/WjBYSrtpuGjWISira8ZguNhKgePFDQYGx+cJ eGbQEWBrzYAh6zAF96prieh90dTVFWA2lhHRMUqAvqmLgFlrAwN/1j2jwFqcBOZ+Fw0v600M zHs3QMSSD07bCAOes3oK7BP36TQ9EmYqfqaEhrYOQqh4EMZC46VGJITmdEgIXC4jhYrq6Hrb P0kK5W3Hhcv3/FiYCz7EQve0hRLu1vLCuf4UwWHyMEL5rSfMV1u+kaRmi7mqAlGzbttBidIy HiaO2LlC590mXIz8S6pQHMtzn/FtnkoyxhSXxLsCM1SMMbeKd7tnF/V4LpHXjw4vMskNr+bP D2hjvIzL4h/ZGhbzUm4zX1vVF2UJK+PmEG/3h+i3xgf8nRof9ba8hncvjBFViI1yAn9lgY3J cdwO3v/wNI7xcu4jvqfjD6IaSU3vtU3vtU3/ty2IbEDxKnVBnkKVu2mtNkdZpFYVrj2Un9eK ok9pPTV/rhMFXDt7Ecci+RLpi/pJpYxWFGiL8noRz5LyeOnmk36lTJqtKDohavIPaI7litpe lMBS8hXS3fvEgzLuO8VRMUcUj4iady7Bxq0sRvG7iY+nBgO3nExhnnq9p+VA35Wp1Kb6oo2v p9+k7/82/dOUr8O+mj1fRpy/nEwJ9m5JXv75aM7Zpemp7r3N0sc/7BO/KPcmlzrcrw/HaQe8 icmO8/aqzKEe+3xId8a1/SfjhyP6XRldaXRC/6q9x/GrrN9P/W30J9pu3PxescuQVHJdTmmV ig1rSI1W8R8ELch0kAMAAA== X-CFilter-Loop: Reflected On Fri, Oct 03, 2025 at 01:19:33AM +0200, Arnd Bergmann wrote: > On Thu, Oct 2, 2025, at 15:53, Mathieu Desnoyers wrote: > > On 2025-10-02 04:24, Greg KH wrote: > >> On Thu, Oct 02, 2025 at 05:12:01PM +0900, Byungchul Park wrote: > >>> llist_head and llist_node can be used by some other header files. For > >>> example, dept for tracking dependencies uses llist in its header. To > >>> avoid header dependency, move them to types.h. > >> > >> If you need llist in your code, then include llist.h. Don't force all > >> types.h users to do so as there is not a dependency in types.h for > >> llist.h. > >> > >> This patch shouldn't be needed as you are hiding "header dependency" for > >> other files. > > > > I agree that moving this into a catch-all types.h is not what we should > > aim for. > > > > However, it's a good practice to move the type declarations to a > > separate header file, so code that only cares about type and not > > implementation of static inline functions can include just that. > > > > Perhaps we can move struct llist_head and struct llist_node to a new > > include/linux/llist_types.h instead ? > > We have around a dozen types of linked lists, and the most common > two of them are currently defined in linux/types.h, while the > rest of them are each defined in the same header as the inteface > definition. > > Duplicating each of those headers by splitting out the trivial > type definition doesn't quite seem right either, as we'd end > up with even more headers that have to be included indirectly > in each compilation unit. > > Maybe a shared linux/list_types.h would work, to specifically I found a way to resolve my issue, but I thought it's good idea regardless of my issue and took a quick look. However, it seems like there's an overwhelming amount of work since it might require to replace all the existing include for use of list things with the new one :-). Byungchul > contain all the list_head variants that are meant to be included > in larger structures? > > Arnd