Return-path: Received: from userp1040.oracle.com ([156.151.31.81]:20443 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751095AbbEYJR4 (ORCPT ); Mon, 25 May 2015 05:17:56 -0400 Date: Mon, 25 May 2015 12:17:37 +0300 From: Dan Carpenter To: Larry Finger Cc: Haggai Eran , devel@driverdev.osuosl.org, Sudip Mukherjee , gregkh@linuxfoundation.org, linux-wireless@vger.kernel.org, Stable , netdev@vger.kernel.org, Arek Rusniak Subject: Re: [PATCH] staging: r8712u: Fix kernel warning for improper call of del_timer_sync() Message-ID: <20150525091737.GQ4150@mwanda> (sfid-20150525_111800_114985_A0135B6C) References: <1432415812-1285-1-git-send-email-Larry.Finger@lwfinger.net> <556268BC.5050807@lwfinger.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <556268BC.5050807@lwfinger.net> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sun, May 24, 2015 at 07:11:40PM -0500, Larry Finger wrote: > On 05/24/2015 02:03 PM, Haggai Eran wrote: > >On 24 May 2015 at 00:16, Larry Finger wrote: > >>The driver is reporting a warning at kernel/time/timer.c:1096 due to calling > >>del_timer_sync() while in interrupt mode. Such warnings are fixed by calling > >>del_timer() instead. > >> > >>Signed-off-by: Larry Finger > >>Cc: Stable > >>Cc: Haggi Eran > > > >Hi, > > > >I haven't been using kernel v4.1 so I haven't seen this warning, but looking > >at the code it seems to originate from the two recent patches to remove > >_cancel_timer and _cancel_timer_ex. I see that there's another patch in lkml [1] > >that changes del_timer_sync back to del_timer in more places. Perhaps it > >could prevent other warnings like this in the future. > > > >Regards, > >Haggai > > > >[1] https://lkml.org/lkml/2015/5/15/226 > > Yes, the script kiddies make changes they do not understand and > screw everything up. Unfortunately, I did not catch these in review. > I think I will submit V2 and blast the contributor. Don't blast the contributor... These are special intern patches that dont' go through the normal review process. The intern process is over this year. The lack of normal review introduced a number of bugs this year. I always complain to Greg about it and he says that I should join the intern mailing list if I care so much. regards, dan carpenter