From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751926AbaCVQs6 (ORCPT ); Sat, 22 Mar 2014 12:48:58 -0400 Received: from ring0.de ([5.45.105.125]:56842 "EHLO ring0.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751314AbaCVQsy (ORCPT ); Sat, 22 Mar 2014 12:48:54 -0400 X-Spam-Report: * -0.0 NO_RELAYS Informational: message was not relayed via SMTP * -1.9 BAYES_00 BODY: Spamwahrscheinlichkeit nach Bayes-Test: 0-1% * [score: 0.0000] * -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's * domain * 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily * valid * -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature * -0.0 NO_RECEIVED Informational: message has no Received headers Date: Sat, 22 Mar 2014 17:48:49 +0100 From: Sebastian Reichel To: Dmitry Eremin-Solenikov , David Woodhouse Cc: linux-kernel@vger.kernel.org Subject: Re: [PATCHv2] bq2415x_charger: Fix Atomic Sleep Bug Message-ID: <20140322164849.GA14558@earth.universe> References: <1393278197-27286-1-git-send-email-sre@debian.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="W/nzBZO5zC0uMSeA" Content-Disposition: inline In-Reply-To: <1393278197-27286-1-git-send-email-sre@debian.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --W/nzBZO5zC0uMSeA Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Mon, Feb 24, 2014 at 10:43:17PM +0100, Sebastian Reichel wrote: > Move sysfs_notify and i2c_transfer calls from bq2415x_notifier_call > to bq2415x_timer_work to avoid sleeping in atomic context. >=20 > This fixes the following bug: >=20 > [ 7.667449] Workqueue: events power_supply_changed_work > [ 7.673034] [] (unwind_backtrace+0x0/0xe0) from [] (s= how_stack+0x10/0x14) > [ 7.682098] [] (show_stack+0x10/0x14) from [] (dump_s= tack+0x78/0xac) > [ 7.690704] [] (dump_stack+0x78/0xac) from [] (__sche= dule_bug+0x48/0x60) > [ 7.699645] [] (__schedule_bug+0x48/0x60) from [] (__= schedule+0x74/0x638) > [ 7.708618] [] (__schedule+0x74/0x638) from [] (sched= ule_timeout+0x1dc/0x24c) > [ 7.718017] [] (schedule_timeout+0x1dc/0x24c) from []= (wait_for_common+0x138/0x17c) > [ 7.727966] [] (wait_for_common+0x138/0x17c) from [] = (omap_i2c_xfer+0x340/0x4a0) > [ 7.737640] [] (omap_i2c_xfer+0x340/0x4a0) from [] (_= _i2c_transfer+0x40/0x74) > [ 7.747039] [] (__i2c_transfer+0x40/0x74) from [] (i2= c_transfer+0x6c/0x90) > [ 7.756195] [] (i2c_transfer+0x6c/0x90) from [] (bq24= 15x_i2c_write+0x48/0x78) > [ 7.765563] [] (bq2415x_i2c_write+0x48/0x78) from [] = (bq2415x_set_weak_battery_voltage+0x4c/0x50) > [ 7.776824] [] (bq2415x_set_weak_battery_voltage+0x4c/0x50) fro= m [] (bq2415x_set_mode+0xdc/0x14c) > [ 7.788085] [] (bq2415x_set_mode+0xdc/0x14c) from [] = (bq2415x_notifier_call+0xa8/0xb4) > [ 7.798309] [] (bq2415x_notifier_call+0xa8/0xb4) from [] (notifier_call_chain+0x38/0x68) > [ 7.808715] [] (notifier_call_chain+0x38/0x68) from [= ] (__atomic_notifier_call_chain+0x2c/0x3c) > [ 7.819732] [] (__atomic_notifier_call_chain+0x2c/0x3c) from [<= c005f2a8>] (atomic_notifier_call_chain+0x14/0x18) > [ 7.831420] [] (atomic_notifier_call_chain+0x14/0x18) from [] (power_supply_changed_work+0x6c/0xb8) > [ 7.842864] [] (power_supply_changed_work+0x6c/0xb8) from [] (process_one_work+0x248/0x440) > [ 7.853546] [] (process_one_work+0x248/0x440) from []= (worker_thread+0x208/0x350) > [ 7.863372] [] (worker_thread+0x208/0x350) from [] (k= thread+0xc8/0xdc) > [ 7.872131] [] (kthread+0xc8/0xdc) from [] (ret_from_= fork+0x14/0x3c) ping! -- Sebastian --W/nzBZO5zC0uMSeA Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBCgAGBQJTLb7xAAoJENju1/PIO/qaXXYQAIvdm1dkc+mO7vEMWlyLhWgk hyH6Wo2Zu/Lpv5cfdAHb28y5qdAmL0nC687Ldq7A/M9nqlKF2Nu4YjvCNALZYl1y wzPd65no7l3sNnGSZ9PMni+2vMSryHq2W+l8pgTFh61LkXSyLlcrhamZsOon8do1 mfyARU1taGdosoNK2fgkd5MkHnCIJ9RQleESKEzRbCrbxLDHksTxc7jRelt/f+9e q+QPeuake3svbO4t8htWNeWUjabcF9OpM1V1aenwo6jdpdVqhHdQTcCm8tepbeCI h7Q+Z/l7dv0IBbmXJikLCRLFCMDzrh898/lZB/qGCkbw8y1+pDfExArVVvILtBU9 GWMa+s/CRBJFJZS4IEY8u05A2hAsBTnKeJr6b1CMCjP+bAKxgrc/qYyQhvNcC85k brxQXJVYdx28kIdvf+AvqXNQSDVxsoeZTNv0ssxfx/ALG/5XkuYVJXaYj6PAVcjM zZDwOIoGlSCC81w35L5U4giC+UDG42ziWBYI8PMbJuSh+NlvMb5jpVXMtrdhkndf fAdnU6ifbFEDO3pGilKELE04izL37tPwEMxmmviHKHKBiEBNo8farAPT+/9ogsyf VUlOFE4p6J6k/V9LqvJAtvAhjscpRe4gKe7XVjfdGUY+gHeWKqEcBfgZ1Qr2KO2p YZSZfrdFaL/Cww+A05NK =iN4e -----END PGP SIGNATURE----- --W/nzBZO5zC0uMSeA--