[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webkit-dev
Subject:    Re: [webkit-dev] Single-threaded database/storage solution
From:       <laszlo.1.gombos () nokia ! com>
Date:       2009-08-18 19:57:33
Message-ID: FFF3EA8D74C64440A0AF2BF3334314E03A734857B2 () NOK-EUMSG-01 ! mgdnok ! nokia ! com
[Download RAW message or body]

Hi All,

QtWebKit has a build time flag called ENABLE_SINGLE_THREADED to turn off \
ENABLE_JSC_MULTIPLE_THREADS and WebCore features that create additional threads. I'm \
interested to make this feature available for QtWebKit (and maybe other ports) as \
well, not just to the WINCE port. See http://trac.webkit.org/changeset/44411.

Regards,
  Laszlo


-----Original Message-----
From: webkit-dev-bounces@lists.webkit.org 
[mailto:webkit-dev-bounces@lists.webkit.org] On Behalf Of ext 
George Staikos
Sent: Tuesday, August 18, 2009 10:36 PM
To: WebKit Development
Subject: Re: [webkit-dev] Single-threaded database/storage solution


On 18-Aug-09, at 1:44 PM, Yong Li wrote:

> Hi, All,
> 
> The current webkit database/storage is implemented with creating new 
> threads. But not all platform/products support mulple- threading. 
> Also, threading can be expensive on some platforms.
> Except this database/storage implementation, WEBKIT platform- 
crossing 
> code can build and run in a single thread. WEBKIT generally uses 
> WebCore::Timer To avoid blocking UI. Another way to avoid blocking UI 
> is to create a separate thread for UI, and WEBKIT code can still run 
> in a single-thread. Now it's broken by database/ storage code, which 
> forces to use multiple threads for WEBKIT. In our WINCE port, we have 
> implemented a single-threaded solution with WebCore::Timer. The patch 
> has been post to https://bugs.webkit.org/
> show_bug.cgi?id=28019 . The patch is a bit out-of-date, due 
to changes 
> in upstream. New patch will come soon. Please give some comments if 
> you're interested.
> 
> There's a macro ENABLE_JSC_MULTIPLE_THREADS used in JSC. Probably 
> there should also be a macro ENABLE_WEBCORE_MULTIPLE_THREADS. Or just 
> use ENABLE_MULTIPLE_THREADS for all multi-threading code.

   I agree with that and I think a cleanup of the patch would 
make it a good addition.  Forcing the use of threads here is not ideal.

--
George Staikos
Torch Mobile Inc.
http://www.torchmobile.com/

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


[prev in list] [next in list] [prev in thread] [next in thread]