From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750928AbdE3FaA (ORCPT ); Tue, 30 May 2017 01:30:00 -0400 Received: from LGEAMRELO12.lge.com ([156.147.23.52]:52063 "EHLO lgeamrelo12.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750720AbdE3F37 (ORCPT ); Tue, 30 May 2017 01:29:59 -0400 X-Original-SENDERIP: 156.147.1.121 X-Original-MAILFROM: byungchul.park@lge.com X-Original-SENDERIP: 165.244.249.23 X-Original-MAILFROM: byungchul.park@lge.com X-Original-SENDERIP: 10.177.222.33 X-Original-MAILFROM: byungchul.park@lge.com Date: Tue, 30 May 2017 14:29:20 +0900 From: Byungchul Park To: , , , CC: , Subject: Re: [RESEND PATCH] irq_work: Don't reinvent the wheel but use existing llist API Message-ID: <20170530052920.GJ28017@X58A-UD3R> References: <1494549935-13972-1-git-send-email-byungchul.park@lge.com> MIME-Version: 1.0 In-Reply-To: <1494549935-13972-1-git-send-email-byungchul.park@lge.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-MIMETrack: Itemize by SMTP Server on LGEKRMHUB07/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2017/05/30 14:29:41, Serialize by Router on LGEKRMHUB07/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2017/05/30 14:29:41, Serialize complete at 2017/05/30 14:29:41 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 12, 2017 at 09:45:35AM +0900, Byungchul Park wrote: > Although llist provides proper APIs, they are not used. Make them used. +to peterz@infradead.org +to mingo@kernel.org +to rostedt@goodmis.org I am not sure whom should I send this patch to.. Could you check this if you are right person? > > Signed-off-by: Byungchul Park > --- > kernel/irq_work.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/kernel/irq_work.c b/kernel/irq_work.c > index bcf107c..e2ebe8c 100644 > --- a/kernel/irq_work.c > +++ b/kernel/irq_work.c > @@ -138,11 +138,7 @@ static void irq_work_run_list(struct llist_head *list) > return; > > llnode = llist_del_all(list); > - while (llnode != NULL) { > - work = llist_entry(llnode, struct irq_work, llnode); > - > - llnode = llist_next(llnode); > - > + llist_for_each_entry(work, llnode, llnode) { > /* > * Clear the PENDING bit, after this point the @work > * can be re-used. > -- > 1.9.1