[Vm-dev] Pulse audio Scratch plugin causes seg fault on 64bit
doc at doconnel.f9.co.uk
Wed Aug 8 11:46:00 UTC 2012
On 08/08/12 02:06, David T. Lewis wrote:
> On Thu, Aug 02, 2012 at 05:36:42PM -0400, Amos Blanton wrote:
>> To reproduce:
>> 1. Execute Scratch using current VM and pulse plugin (on Ubuntu 12.04, 64
>> bit system):
>> Executing: /usr/lib/squeak/4.4.7-2357/squeakvm -encoding UTF-8
>> -vm-display-x11 -xshm -plugins
>> /usr/lib/scratch/plugins/:/usr/lib/squeak/4.4.7-2357/ -vm-sound-pulse
>> (There is no pulse plugin in /usr/lib/scratch/plugins in my installation
>> package, so it must be using the one in /usr/lib/squeak/4.4.7-2357)
>> 2. Click "Sounds" tab, click play icon next to meow. That results in:
>> Segmentation fault
>> 10273532  in SoundPlayer class>playLoop
>> 10262332 Semaphore>critical:
>> 10262240 SoundPlayer class>playLoop
>> 10273228  in SoundPlayer class>startPlayerProcessBufferSize:rate:stereo:
>> 10273320  in BlockContext>newProcess
>> Aborted (core dumped)
>> Note that the same package on 32 bit systems does not produce the error
>> (playback and recording both work).
>> The issue is also described here:
>> If someone can take a look and patch this, we'd very much appreciate it!
>> This is the last major (known) issue we've got with Scratch running on
> I have a possible fix for this problem, but need some help to verify.
> The issue is tracked on Mantis http://bugs.squeak.org/view.php?id=7701
> I believe the problem is just some type declaration issues in the support
> code that result in pointers being stored in ints, hence VM crash. A
> potential fix is attached (and also on uploaded to the Mantis issue).
> I need someone to test the 64-bit compiled VM on Scratch with Ubuntu
> and pulse audio and see if the VM crashes are eliminated.
> Bert: Can I send you a compiled VM to test with Scratch/Etoys/Squeak
> and find out if sound still crashes the VM?
> Ian: Regardless of whether this fixes the Scratch problem, the type
> declarations in vm-sound-pulse need to be updated, so please consider
> the attached as a patch.
Tested on Ubuntu 12.04 64-bit, works, thanks Dave!
More information about the Vm-dev