From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1166461AbdDXIrI (ORCPT ); Mon, 24 Apr 2017 04:47:08 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:37330 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1166368AbdDXIrA (ORCPT ); Mon, 24 Apr 2017 04:47:00 -0400 Date: Mon, 24 Apr 2017 09:46:50 +0100 From: Greg Kroah-Hartman To: weili@codeaurora.org Cc: linux-kernel@vger.kernel.org, vatsa@codeaurora.org Subject: Re: [PATCH] driver-core: remove lock for platform devices during probe Message-ID: <20170424084650.GA14347@kroah.com> References: <1493012536-9240-1-git-send-email-weili@codeaurora.org> <20170424073232.GA19970@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.2 (2017-04-18) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Mon, Apr 24, 2017 at 04:27:44PM +0800, weili@codeaurora.org wrote: > Hi Greg, > > We are optimizing boot time for Linux kernel and try to make some > platform drivers use asynchronous probe(by changing probe type of driver to > PROBE_PREFER_ASYNCHRONOUS) to reduce boot time. However we found the > platform drivers did not probe in parallel because they will lock the same > parent device(platform bus for platform drivers) during probe. So we add > this patch to remove lock of parent for platform device. This will help to > make platform driver probe in parallel and reduce boot time. And does it really reduce boot time? What are the numbers? Why do you have so many platform devices and not "real bus" devices? What does the boot graph look like when you run with and without this patch? Why is the platform bus so "special" to warrant this? Should we perhaps make this an option for any bus to enable/disable? In other words, you need to provide a whole lot more information here, justify why you are doing this type of change, and fix the issue I pointed out, in order to get such a chance accepted. Come on now, you know better than this... greg k-h