[webkit-dev] Single-threaded database/storage solution

[Yong Li]

Hi, Laszlo,

Do you think our single-threaded storage(database) solution can be helpful 
to you? If so, I'm going to change the switch from PLATFORM(WINCE) to 
ENABLE(SINGLE_THREADED) or something like that.

-Yong

----- Original Message ----- 
From: <laszlo.1.gombos at nokia.com>
To: <staikos at kde.org>; <webkit-dev at lists.webkit.org>
Sent: Tuesday, August 18, 2009 3:57 PM
Subject: Re: [webkit-dev] Single-threaded database/storage solution


> 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 at lists.webkit.org
> [mailto:webkit-dev-bounces at 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 at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>