From patchwork Wed Sep 27 09:18:13 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michael Nazzareno Trimarchi X-Patchwork-Id: 9973541 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 7AD746037F for ; Wed, 27 Sep 2017 09:18:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 76EFB29143 for ; Wed, 27 Sep 2017 09:18:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 694952912B; Wed, 27 Sep 2017 09:18:20 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.4 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, RCVD_IN_DNSWL_HI, RCVD_IN_SORBS_SPAM autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7FFDA29143 for ; Wed, 27 Sep 2017 09:18:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752508AbdI0JSR (ORCPT ); Wed, 27 Sep 2017 05:18:17 -0400 Received: from mail-it0-f46.google.com ([209.85.214.46]:49885 "EHLO mail-it0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752292AbdI0JSP (ORCPT ); Wed, 27 Sep 2017 05:18:15 -0400 Received: by mail-it0-f46.google.com with SMTP id c195so6264882itb.4 for ; Wed, 27 Sep 2017 02:18:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amarulasolutions-com.20150623.gappssmtp.com; s=20150623; h=mime-version:from:date:message-id:subject:to:cc; bh=2jtwpZXTDLwRHlql4zLhdpE8ii6uEiAdJoI2tckYBcE=; b=vn/ipbIwGfOKE1ouL/Rx35MQ+FWoqWC3XqvpzbyzO7cLjw43RT0tQl5Dsdo/33cfYh 0DKY+uEz0d+dWlPQ1IY7XgZPCeM0J0i2kdGLOM75e5vWzvtssmrd6E8tYjAMjiegKg9B GPw8+wselKXMm54h5/R02wJnWa4Tk1Z0VA6nkjS442aBcjzOl9UF+rJ+rpVvoAuU5hG0 nkMZfIdNfzecNVEJOUmjexj84zG5Wc1v2NmBYvPUCG5FbnvvcMFYSW0fFbKtABz66CJ+ oPfwVK2hVLvKwXGLhwW0rGN5wRcheVREmAl9cVJB4LBxqiFwdTRUo6900+vUKjNgtkVM Bsng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc; bh=2jtwpZXTDLwRHlql4zLhdpE8ii6uEiAdJoI2tckYBcE=; b=VjmdEHcJgcByTmqATU4p3SBjSHaTcEbdiER1A3iloVSjJxr0zqoZ2ctt9H4x1mgQ6B mEDeDizQkAV+BZpjxyEAkDuwXoceiGuF3hep3cXCoh/NnMAMJgHrIP+RqOLs+x20Sds+ 5Co4GiPkAiDPRaB3eZwhrPbzWYeLz96qKqxdPpfC8+O1jdPnzAjdtdmyQgNrcKcCkOqi UdaUjjimEm2FmLnDxJF2Y/U1lPxkLMW03waJkxhBI5TjCfb7u3MyvLRUQCPD1RWaMiWg VHigA2S5IakY2yxQ6AjVYrDRTJtgDAKt6hAXPeACRqRXLC6fO1TyTDMxRqrnmNcZBCJP 5jSw== X-Gm-Message-State: AHPjjUg7S+WvIEI6h9p71/4saOYTtGlmgj3uWlRCuPcNEvatObCnNFvC 6wP/XlAvStNTT5FYErsgB173dqmQSFLTSdSHvSP5EO8Giu0= X-Google-Smtp-Source: AOwi7QCIAdSaz2ywpbt0+9aqXdw0ri0NTX7T1VKucMEsXx45cVUsBJ9B/IN0ocTM69uIjnFYnlS6YptOhmEKsbYOfjk= X-Received: by 10.36.225.75 with SMTP id n72mr1506338ith.82.1506503894230; Wed, 27 Sep 2017 02:18:14 -0700 (PDT) MIME-Version: 1.0 Received: by 10.79.149.139 with HTTP; Wed, 27 Sep 2017 02:18:13 -0700 (PDT) From: Michael Nazzareno Trimarchi Date: Wed, 27 Sep 2017 11:18:13 +0200 Message-ID: Subject: power_supply_core freezable change_work To: linux-pm@vger.kernel.org Cc: Mark Brown Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi <4>[13502.656095] [] (sbs_read_word_data+0x0/0x70) from [] (sbs_set_battery_mode+0x24/0x6c) <4>[13502.656095] r6:d79b0800 r5:d7123408 r4:00000001 r3:c04194ac <4>[13502.656125] [] (sbs_set_battery_mode+0x0/0x6c) from [] (sbs_get_property+0x180/0x6e4) <4>[13502.656125] r6:0000001b r5:d7123408 r4:d79b0800 r3:c04194ac <4>[13502.656156] [] (sbs_get_property+0x0/0x6e4) from [] (power_supply_show_property+0x4c/0x1c8) <4>[13502.656186] [] (power_supply_show_property+0x0/0x1c8) from [] (power_supply_uevent+0xa8/0x1d4) <4>[13502.656186] r8:ca4b1000 r7:0000001b r6:d7123408 r5:0000000e r4:c068d3dc <4>[13502.656217] [] (power_supply_uevent+0x0/0x1d4) from [] (dev_uevent+0xc8/0x18c) <4>[13502.656247] [] (dev_uevent+0x0/0x18c) from [] (kobject_uevent_env+0x1d4/0x4c4) <4>[13502.656247] r7:d7882880 r6:00000000 r5:c069cda4 r4:00000002 <4>[13502.656278] [] (kobject_uevent_env+0x0/0x4c4) from [] (kobject_uevent+0x14/0x18) <4>[13502.656308] [] (kobject_uevent+0x0/0x18) from [] (power_supply_changed_work+0x94/0xb8) <4>[13502.656308] [] (power_supply_changed_work+0x0/0xb8) from [] (process_one_work+0x158/0x474) <4>[13502.656339] r7:c1ae9e00 r6:c1ae4880 r5:cc480000 r4:ce655e00 <4>[13502.656369] [] (process_one_work+0x0/0x474) from [] (worker_thread+0x144/0x324) <4>[13502.656369] [] (worker_thread+0x0/0x324) from [] (kthread+0x98/0xa4) <4>[13502.656400] [] (kthread+0x0/0xa4) from [] (do_exit+0x0/0x7c4) According to this stack I would like to know if I can queue Michael diff --git a/drivers/power/power_supply_core.c b/drivers/power/power_supply_core.c index f58d1805..3dae683 100644 --- a/drivers/power/power_supply_core.c +++ b/drivers/power/power_supply_core.c @@ -75,7 +75,7 @@ void power_supply_changed(struct power_supply *psy) psy->changed = true; wake_lock(&psy->work_wake_lock); spin_unlock_irqrestore(&psy->changed_lock, flags); - schedule_work(&psy->changed_work); + queue_work(system_freezable_wq, &psy->chaged_work); } EXPORT_SYMBOL_GPL(power_supply_changed);