Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-05-02 Thread Greg Kroah-Hartman
On Tue, May 02, 2017 at 10:18:25AM +0800, we...@codeaurora.org wrote: > Hi Greg K-H, > > On 2017-04-25 19:36, Greg Kroah-Hartman wrote: > > On Tue, Apr 25, 2017 at 04:43:33PM +0800, we...@codeaurora.org wrote: > > > Hi Greg K-H, > > > > > > On 2017-04-24 16:46, Greg Kroah-Hartman wrote: > > > >

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-05-01 Thread weili
Hi Greg K-H, On 2017-04-25 19:36, Greg Kroah-Hartman wrote: On Tue, Apr 25, 2017 at 04:43:33PM +0800, we...@codeaurora.org wrote: Hi Greg K-H, On 2017-04-24 16:46, Greg Kroah-Hartman wrote: > And does it really reduce boot time? What are the numbers? Yes, it really reduce boot time. After m

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-04-25 Thread Greg Kroah-Hartman
On Tue, Apr 25, 2017 at 04:43:33PM +0800, we...@codeaurora.org wrote: > Hi Greg K-H, > > On 2017-04-24 16:46, Greg Kroah-Hartman wrote: > > > And does it really reduce boot time? What are the numbers? > Yes, it really reduce boot time. After making most time-consuming platform > driver using as

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-04-25 Thread weili
Hi Greg K-H, On 2017-04-24 16:46, Greg Kroah-Hartman wrote: And does it really reduce boot time? What are the numbers? Yes, it really reduce boot time. After making most time-consuming platform driver using async probe and also applying this patch, we see the driver run in parallel with o

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-04-24 Thread Greg Kroah-Hartman
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:

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-04-24 Thread weili
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

Re: [PATCH] driver-core: remove lock for platform devices during probe

2017-04-24 Thread Greg Kroah-Hartman
On Mon, Apr 24, 2017 at 01:42:16PM +0800, Wei Li wrote: > During driver probe procedure, lock on the parent of > platform devices could be removed to make probe in > parallel. > > Signed-off-by: Wei Li Why? Why does this matter? > --- > drivers/base/dd.c | 6 -- > 1 file changed, 4 insert

[PATCH] driver-core: remove lock for platform devices during probe

2017-04-23 Thread Wei Li
During driver probe procedure, lock on the parent of platform devices could be removed to make probe in parallel. Signed-off-by: Wei Li --- drivers/base/dd.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/base/dd.c b/drivers/base/dd.c index a1fbf55..e238fbc 100