Re: [webkit-dev] Stability problems involving Javascript GC

2010-12-16 Thread Samuel Rivas
On 6 December 2010 22:31, Zoltan Herczeg zherc...@inf.u-szeged.hu wrote:
 Crash in WTF::fastMalloc? Such things only happen if something overwrites
 memory areas belongs to the memory manager (i.e overwrites some bytes
 before or after a block returned by malloc). Try some valgrind equivalent
 on mac to detect those writings into red zones.

How can you use valgrind to help on that? We had some symptoms similar
to this and also came to the conclusion that probably something is
overwriting the structures used by fast malloc, but couldn't find
anything with valgrind. Overwriting in an area that has bee reserved
is not an error vangrind finds, at least not with any options that I
know.

Regards

-- 
Samuel Rivas
RD and Base Development Leader
m:+34 619 92 77 46
t:+34 981 17 33 44 ext:61
www.lambdastream.com
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] Atomic read/write operations and thread safety

2010-12-16 Thread Mikhail Naganov
In MSDN there is a widely cited article about lockless programming on
x86 and Xbox: http://msdn.microsoft.com/en-us/library/ee418650(v=vs.85).aspx

In short: using boolean as a flag for thread termination is OK. Using
boolean as a guard for multi-threaded access to data isn't OK, as read
/ write reordering can be made both by C++ compiler and CPU.

On Thu, Dec 16, 2010 at 00:34, Alexey Proskuryakov a...@webkit.org wrote:

 15.12.2010, в 10:36, Stephan Aßmus написал(а):

 The only thing one needs to watch out for is to declare such a boolean 
 volatile (which I believe this code does, if memory serves). Otherwise the 
 thread which polls the condition may read from a cached location and miss 
 the change. Worst that can happen on a hypothecial architecture where 
 writing a byte is not atomic is that the changed condition takes affect one 
 loop cycle later.


 The behavior of volatile depends on the compiler - there is no guarantee that 
 a memory barrier will be emitted. Historically, volatile existed not for 
 multiprocessor systems, but for special addresses in address space (like 
 memory mapped I/O), where it is important to actually access the address each 
 time, and not than keep a local variable in a boolean.

 There is code in WebKit that doesn't care about the other thread immediately 
 seeing the change of a boolean value, and just hopes that it will propagate 
 soon enough in practice. I remember implementing worker thread termination in 
 that way. This code is of course formally wrong.

 - WBR, Alexey Proskuryakov

 ___
 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


[webkit-dev] Webkit Cairo Port on windows

2010-12-16 Thread Optimus

I have successfully built the cairo port for webkit on windows. 

I have two Questions.
1) If i want to ship the webkit on another machine, what all dll i should
bundle, with webkit.

2) When i tried to run the webkit on another machine, it did not run and
instead i got an error message 
This application has failed to start because the application configuration
is incorrect. Reinstalling the application may fix this problem. 

But when i installed sufari, on that machine, it worked. Do we need sufari
to be installed? I mean as i know we can build a standalone application
using cairo port..?
-- 
View this message in context: 
http://old.nabble.com/Webkit-Cairo-Port-on-windows-tp30472133p30472133.html
Sent from the Webkit mailing list archive at Nabble.com.

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


Re: [webkit-dev] WebKitTools is changing to Tools

2010-12-16 Thread Osztrogonac Csaba

Hi,

Eric Seidel írta:

So that means at least the following people will need to perform restarts:
- Whoever runs the Qt ews

I will be online on Dec 17 at 4PM PST and will restart the Qt EWS. Additionally 
the
master buildbot of University of Szeged ( 
http://webkit.sed.hu/buildbot/waterfall )
will need to be restarted too. I will do it after the patch landed.


On Fri, Dec 10, 2010 at 2:10 PM, Dan Bernstein m...@apple.com wrote:

How about next week, Dec 17 at around 4PM PST?


And Dirk mentioned in the bug, that some Chromium wrapper scripts
will need to be updated. Will this update affect the Chromium master?
( http://build.chromium.org/p/chromium/console ) Who will take care
of the Chromium scripts/master update?

br,
Ossy

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


[webkit-dev] Build Slave Shutdown

2010-12-16 Thread William Siegrist
Our buildbot allows for anonymous people to trigger things on the slaves, and 
it is like this on purpose for ease of use. However, that means it is possible 
for a malicious person to do things like shutdown all of the slaves. That is 
what happened last night around 10:30pm PST, from 66.57.13.12, and that is why 
the slaves are offline. 

If you own a build slave that is shutdown, you need to restart the buildbot 
process on it and check that it reconnects to the master. If you have trouble, 
email me or ping _wms on irc. 

I landed http://trac.webkit.org/changeset/74194 to disable the shutdown 
feature for now. We can move to having user accounts, possibly even use 
trac/svn or bugs credentials, if we feel we need more fine-grained permissions. 

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


Re: [webkit-dev] Stability problems involving Javascript GC

2010-12-16 Thread Joe Mason
 -Original Message-
 From: webkit-dev-boun...@lists.webkit.org [mailto:webkit-dev-
 boun...@lists.webkit.org] On Behalf Of Samuel Rivas
 Sent: Thursday, December 16, 2010 3:03 AM
 To: Zoltan Herczeg
 Cc: webkit-dev Development
 Subject: Re: [webkit-dev] Stability problems involving Javascript GC
 
 On 6 December 2010 22:31, Zoltan Herczeg zherc...@inf.u-szeged.hu
 wrote:
  Crash in WTF::fastMalloc? Such things only happen if something
 overwrites
  memory areas belongs to the memory manager (i.e overwrites some bytes
  before or after a block returned by malloc). Try some valgrind
 equivalent
  on mac to detect those writings into red zones.
 
 How can you use valgrind to help on that? We had some symptoms similar
 to this and also came to the conclusion that probably something is
 overwriting the structures used by fast malloc, but couldn't find
 anything with valgrind. Overwriting in an area that has bee reserved
 is not an error vangrind finds, at least not with any options that I
 know.

It is possible to add macros to the code to help valgrind know which areas are 
being used by a custom allocator.  (See VALGRIND_MALLOCLIKE_BLOCK and 
VALGRIND_FREELIKE_BLOCK, for example.)  I'm not sure if they're useful in this 
situation, but they're worth taking a look at.

Joe 

-
This transmission (including any attachments) may contain confidential 
information, privileged material (including material protected by the 
solicitor-client or other applicable privileges), or constitute non-public 
information. Any use of this information by anyone other than the intended 
recipient is prohibited. If you have received this transmission in error, 
please immediately reply to the sender and delete this information from your 
system. Use, dissemination, distribution, or reproduction of this transmission 
by unintended recipients is not authorized and may be unlawful.
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] Webkit Cairo Port on windows

2010-12-16 Thread Brent Fulgham
Hi,

On Thu, Dec 16, 2010 at 4:27 AM, Optimus vsbho...@gmail.com wrote:
 I have two Questions.
 1) If i want to ship the webkit on another machine, what all dll i should
 bundle, with webkit.

The best answer to this question is to use the 'depends' tool on
Windows to see what DLL's it needs to load.  I believe the following
libraries are sufficient:

1. libxml2.dll
2. libxslt.dll
3. JavaScriptCore.dll
4. ICUUC40.dll
5. ICUIN40.dll
6. CFLite.dll
7. libcurl.dll
8. libeay32.dll
9. ssleay32.dll

However, if you enable QuickTime video support, this will pull in
CoreFoundation and the rest of the apple libraries, as they are needed
by QuickTime.

We've been refactoring the project files a fair amount recently, and
during this process I may have missed some places where Apple-specific
binaries are being linked.

I'll double-check my build environment and make sure the apple
runtimes are not being incorrectly used.

 2) When i tried to run the webkit on another machine, it did not run and
 instead i got an error message
 This application has failed to start because the application configuration
 is incorrect. Reinstalling the application may fix this problem. 

 But when i installed sufari, on that machine, it worked. Do we need sufari
 to be installed? I mean as i know we can build a standalone application
 using cairo port..?

I suspect I may have allowed the 'Apple' build environment to bleed
into the WinCairo side of things.  I'll take a look at it today and
try to get it sorted out.

Thanks for the head's up!

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


Re: [webkit-dev] Build Slave Shutdown

2010-12-16 Thread Adam Barth
I've used the shutdown/restart functionality before to try to unwedge
a broken bot.  The most useful feature, though, is the force build
feature, which I use occasionally late at night when the tree is quiet
to probe for flakiness.

Adam


On Thu, Dec 16, 2010 at 9:05 AM, William Siegrist wsiegr...@apple.com wrote:
 Our buildbot allows for anonymous people to trigger things on the slaves,
 and it is like this on purpose for ease of use. However, that means it is
 possible for a malicious person to do things like shutdown all of the
 slaves. That is what happened last night around 10:30pm PST, from
 66.57.13.12, and that is why the slaves are offline.
 If you own a build slave that is shutdown, you need to restart the buildbot
 process on it and check that it reconnects to the master. If you have
 trouble, email me or ping _wms on irc.
 I landed http://trac.webkit.org/changeset/74194 to disable the shutdown
 feature for now. We can move to having user accounts, possibly even use
 trac/svn or bugs credentials, if we feel we need more fine-grained
 permissions.
 -Bill
 ___
 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


Re: [webkit-dev] Build Slave Shutdown

2010-12-16 Thread Ryosuke Niwa
On Thu, Dec 16, 2010 at 9:05 AM, William Siegrist wsiegr...@apple.comwrote:

 Our buildbot allows for anonymous people to trigger things on the slaves,
 and it is like this on purpose for ease of use. However, that means it is
 possible for a malicious person to do things like shutdown all of the
 slaves. That is what happened last night around 10:30pm PST, from
 66.57.13.12, and that is why the slaves are offline.


Do you know when Mac bots become online again?

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


Re: [webkit-dev] Build Slave Shutdown

2010-12-16 Thread William Siegrist
Forced builds are still enabled. But you normally shutdown slaves you do not 
have shell access to? 

-Bill


On Dec 16, 2010, at 11:50 AM, Adam Barth wrote:

 I've used the shutdown/restart functionality before to try to unwedge
 a broken bot.  The most useful feature, though, is the force build
 feature, which I use occasionally late at night when the tree is quiet
 to probe for flakiness.
 
 Adam
 
 
 On Thu, Dec 16, 2010 at 9:05 AM, William Siegrist wsiegr...@apple.com wrote:
 Our buildbot allows for anonymous people to trigger things on the slaves,
 and it is like this on purpose for ease of use. However, that means it is
 possible for a malicious person to do things like shutdown all of the
 slaves. That is what happened last night around 10:30pm PST, from
 66.57.13.12, and that is why the slaves are offline.
 If you own a build slave that is shutdown, you need to restart the buildbot
 process on it and check that it reconnects to the master. If you have
 trouble, email me or ping _wms on irc.
 I landed http://trac.webkit.org/changeset/74194 to disable the shutdown
 feature for now. We can move to having user accounts, possibly even use
 trac/svn or bugs credentials, if we feel we need more fine-grained
 permissions.
 -Bill
 ___
 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


Re: [webkit-dev] Build Slave Shutdown

2010-12-16 Thread Adam Barth
In the past, we've had trouble where one of the two Leopard slaves (or
whatever) would consistently be failing a dozen or so tests and the
other one would be fine.  This causes the tree to be very confusing
because the tests appear to fail randomly half the time.  I've used
the shutdown button to solve those problems, but perhaps I should have
just talked with you directly instead.

Adam


On Thu, Dec 16, 2010 at 2:39 PM, William Siegrist wsiegr...@apple.com wrote:
 Forced builds are still enabled. But you normally shutdown slaves you do not 
 have shell access to?

 On Dec 16, 2010, at 11:50 AM, Adam Barth wrote:
 I've used the shutdown/restart functionality before to try to unwedge
 a broken bot.  The most useful feature, though, is the force build
 feature, which I use occasionally late at night when the tree is quiet
 to probe for flakiness.

 Adam


 On Thu, Dec 16, 2010 at 9:05 AM, William Siegrist wsiegr...@apple.com 
 wrote:
 Our buildbot allows for anonymous people to trigger things on the slaves,
 and it is like this on purpose for ease of use. However, that means it is
 possible for a malicious person to do things like shutdown all of the
 slaves. That is what happened last night around 10:30pm PST, from
 66.57.13.12, and that is why the slaves are offline.
 If you own a build slave that is shutdown, you need to restart the buildbot
 process on it and check that it reconnects to the master. If you have
 trouble, email me or ping _wms on irc.
 I landed http://trac.webkit.org/changeset/74194 to disable the shutdown
 feature for now. We can move to having user accounts, possibly even use
 trac/svn or bugs credentials, if we feel we need more fine-grained
 permissions.
 -Bill
 ___
 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


Re: [webkit-dev] Build Slave Shutdown

2010-12-16 Thread William Siegrist
You should talk to the slave owner in that case. I just own the master. For 
example, according to http://build.webkit.org/buildslaves/apple-xserve-5, 
apple-xserve-5 is owned by bdash. We should probably make sure there are 
contact details for each slave.

-Bill



On Dec 16, 2010, at 2:42 PM, Adam Barth wrote:

 In the past, we've had trouble where one of the two Leopard slaves (or
 whatever) would consistently be failing a dozen or so tests and the
 other one would be fine.  This causes the tree to be very confusing
 because the tests appear to fail randomly half the time.  I've used
 the shutdown button to solve those problems, but perhaps I should have
 just talked with you directly instead.
 
 Adam
 
 
 On Thu, Dec 16, 2010 at 2:39 PM, William Siegrist wsiegr...@apple.com wrote:
 Forced builds are still enabled. But you normally shutdown slaves you do not 
 have shell access to?
 
 On Dec 16, 2010, at 11:50 AM, Adam Barth wrote:
 I've used the shutdown/restart functionality before to try to unwedge
 a broken bot.  The most useful feature, though, is the force build
 feature, which I use occasionally late at night when the tree is quiet
 to probe for flakiness.
 
 Adam
 
 
 On Thu, Dec 16, 2010 at 9:05 AM, William Siegrist wsiegr...@apple.com 
 wrote:
 Our buildbot allows for anonymous people to trigger things on the slaves,
 and it is like this on purpose for ease of use. However, that means it is
 possible for a malicious person to do things like shutdown all of the
 slaves. That is what happened last night around 10:30pm PST, from
 66.57.13.12, and that is why the slaves are offline.
 If you own a build slave that is shutdown, you need to restart the buildbot
 process on it and check that it reconnects to the master. If you have
 trouble, email me or ping _wms on irc.
 I landed http://trac.webkit.org/changeset/74194 to disable the shutdown
 feature for now. We can move to having user accounts, possibly even use
 trac/svn or bugs credentials, if we feel we need more fine-grained
 permissions.
 -Bill
 ___
 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


Re: [webkit-dev] Webkit Cairo Port on windows

2010-12-16 Thread Optimus

Hi Brent,

The best answer to this question is to use the 'depends' tool on
Windows to see what DLL's it needs to load.  I believe the following
libraries are sufficient:

1. libxml2.dll
2. libxslt.dll
3. JavaScriptCore.dll
4. ICUUC40.dll
5. ICUIN40.dll
6. CFLite.dll
7. libcurl.dll
8. libeay32.dll
9. ssleay32.dll

However, if you enable QuickTime video support, this will pull in
CoreFoundation and the rest of the apple libraries, as they are needed
by QuickTime.

We've been refactoring the project files a fair amount recently, and
during this process I may have missed some places where Apple-specific
binaries are being linked.

I'll double-check my build environment and make sure the apple
runtimes are not being incorrectly used.

I am already using the tool, but i was doubtful about QuickTime, so then we
have to dissable QuickTime video support.

I suspect I may have allowed the 'Apple' build environment to bleed
into the WinCairo side of things.  I'll take a look at it today and
try to get it sorted out.

Please tell me if so, then i can update my source tree. Is it possible for
me to change build environment locally, if it is, where should i look into? 

Thanks for the head's up!
My pleasure 

-Optimus


-- 
View this message in context: 
http://old.nabble.com/Webkit-Cairo-Port-on-windows-tp30472133p30478437.html
Sent from the Webkit mailing list archive at Nabble.com.

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


[webkit-dev] Question about StringImpl::create

2010-12-16 Thread 谢愈挺
I have a question about StringImpl::create(). Is it can only input latin1 
data? 
 If it can only input the latin1 data, the following code maybe incrorect:
  
 In WTFString.cpp :
 String String::format(const char *format, ...)
 {
  ...
 Vectorchar, 256 buffer;
 ...
   return StringImpl::create(buffer.data(), len);
 }
  
 The buffer.data() maybe utf8 data!
  
 Thanks.
  
 pattin___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


[webkit-dev] Page Cache for web-pages with plugin

2010-12-16 Thread Aneesh Bhasin
Hi All,

I was wondering as to what is the current status for maintaining a
page cache for webpages which contain some plugin (
https://bugs.webkit.org/show_bug.cgi?id=13634) ? The last update on
that bug report said that work is ongoing towards it. If someone could
point me to the right direction here that it will be very helpful..

Thanks and regards,
Aneesh
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] Question about StringImpl::create

2010-12-16 Thread Gavin Barraclough
On Dec 16, 2010, at 9:45 PM, 谢愈挺 wrote:

 I have a question about StringImpl::create(). Is it can only input latin1 
 data?

Yes, this method takes latin1 data all const char* arguments in WTFString are 
handled as latin1.
String::fromUTF8 can be used to create strings with utf8 data.

 If it can only input the latin1 data, the following code maybe incorrect:
 In WTFString.cpp :
 String String::format(const char *format, ...)
 {
  ...
 Vectorchar, 256 buffer;
 ...
   return StringImpl::create(buffer.data(), len);
 }
  
 The buffer.data() maybe utf8 data!

The short answer is that String::format should be considered deprecated - its 
behavior is platform-specific, and we'd like to move away from it.

Since all const char* arguments to the String methods are treated as latin1, 
format is not suitable for use with utf8 data – any code that is calling format 
but that may be working with utf8 data needs to be refactored to stop using 
this method (utf8 should be converted to WebCore Strings first, then use the 
other conversion/concatenation methods available).

cheers,
G.


 Thanks.
  
 pattin
 ___
 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