Re: [PATCH v2 00/14] list: Prepare entry iterators to cache cursor state
Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/[email protected]> Wed, 10 Jun 2026 18:02:32 +0300
| Newsgroups | gmane.linux.kernel.drbd.devel,gmane.comp.video.dri.devel,gmane.linux.kernel,gmane.comp.freedesktop.xorg.drivers.intel,gmane.linux.block,gmane.linux.ports.tegra,gmane.linux.kernel.firewire.devel,gmane.linux.ports.arm.kernel,gmane.linux.sound,gmane.linux.kernel.spi.devel |
|---|---|
| Organization | Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo |
| Message-ID | <[email protected]> |
On Wed, Jun 10, 2026 at 11:11:34AM +0200, Christian König wrote: > On 6/10/26 10:18, Kaitao Cheng wrote: > > 在 2026/6/10 16:07, Christian König 写道: ... > > Should we revert to v1, or keep list_for_each_entry() and > > list_for_each_entry_safe() as they are, close this thread, and make no > > changes? > > > > Link to v1: > > https://lore.kernel.org/all/20260529082149.76764-1-kaitao.cheng-fxUVXftIFDnyG1zEObXtfA@public.gmane.org/ > > > > Or do you have any better suggestions? > > v1 looks perfectly reasonable to me. But why not just hiding that once for all (in case they don't use the temporary iterator)? Easy to automate, robust — everyone is happy? > You should just include some patches in the same patch set to actually use > the new macros. > > If you modify the files under drivers/dma-buf or drivers/gpu/drm/amd to use > the new macro I'm happy to review that. -- With Best Regards, Andy Shevchenko