From patchwork Sun Mar 31 22:57:21 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rafael Wysocki X-Patchwork-Id: 2368821 Return-Path: X-Original-To: patchwork-linux-pm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 4755BDF264 for ; Sun, 31 Mar 2013 22:51:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755975Ab3CaWu6 (ORCPT ); Sun, 31 Mar 2013 18:50:58 -0400 Received: from hydra.sisk.pl ([212.160.235.94]:48971 "EHLO hydra.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755857Ab3CaWu5 (ORCPT ); Sun, 31 Mar 2013 18:50:57 -0400 Received: from vostro.rjw.lan (aepo243.neoplus.adsl.tpnet.pl [79.191.144.243]) by hydra.sisk.pl (Postfix) with ESMTPSA id 5ED91E3F5D; Mon, 1 Apr 2013 00:49:34 +0200 (CEST) From: "Rafael J. Wysocki" To: Greg Kroah-Hartman Cc: Sasha Levin , Linus Torvalds , Tejun Heo , Dave Jones , LKML , Linux PM list , linux-usb@vger.kernel.org, Sarah Sharp , Lan Tianyu Subject: [PATCH 1/2] USB / PM: Don't try to hide PM QoS flags from usb_port_device_release() Date: Mon, 01 Apr 2013 00:57:21 +0200 Message-ID: <1651223.ep7oZJM5CX@vostro.rjw.lan> User-Agent: KMail/4.9.5 (Linux/3.9.0-rc4+; KDE/4.9.5; x86_64; ; ) In-Reply-To: <1876128.a4gSDF7byc@vostro.rjw.lan> References: <512F8980.4040809@oracle.com> <2829731.BFs3ri3Eud@vostro.rjw.lan> <1876128.a4gSDF7byc@vostro.rjw.lan> MIME-Version: 1.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org From: Rafael J. Wysocki Remove the call to dev_pm_qos_hide_flags(), added by commit 6e30d7cb "usb: Add driver/usb/core/(port.c,hub.h) files", from usb_port_device_release(), because (1) it is completely unnecessary (the flags have been removed already by the PM core during the unregistration of the device object) and (2) it triggers a NULL pointer dereference in sysfs_find_dirent() (dev->kobj->sd is NULL at this point). Signed-off-by: Rafael J. Wysocki Acked-by: Greg Kroah-Hartman --- drivers/usb/core/port.c | 1 - 1 file changed, 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Index: linux-pm/drivers/usb/core/port.c =================================================================== --- linux-pm.orig/drivers/usb/core/port.c +++ linux-pm/drivers/usb/core/port.c @@ -67,7 +67,6 @@ static void usb_port_device_release(stru { struct usb_port *port_dev = to_usb_port(dev); - dev_pm_qos_hide_flags(dev); kfree(port_dev); }