Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2015-11-26 Thread James Cowgill
Hi,

I've just seen this bus error happen on a MIPS CI20. Using packed
structs (or any misaligned memory) will likely cause a SIGBUS on those
machines.

Is it possible to fix this on mips machines as well? The define to
check would be __mips__ (which is defined on all mips machines
including 64-bit).

Thanks,
James

signature.asc
Description: This is a digitally signed message part


Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2015-08-25 Thread Adrian Knoth

On 05/15/15 01:27, Martin Langer wrote:


Hi,


Hi!


I run into the same bus error with a freshly installed jessie on my
powerpc (1GHz Powerbook, G4). Nevertheless this small patch fixes the
bus error and jack is running fine now with jessie.


Thanks for reporting back. I've applied the patch to upstream jackd2,
it will be included when we sync the next time (we just synced today
prior to inclusion).

Upstream commit:


https://github.com/jackaudio/jack2/commit/460063d8dc2cb465e22fd538239817a0cb0baec6


Cheers



Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2015-05-14 Thread Martin Langer
Hi,

please apply the patch above from Adrian (identical with the attached one) also 
to jessie.

I run into the same bus error with a freshly installed jessie on my powerpc 
(1GHz Powerbook, G4).
Nevertheless this small patch fixes the bus error and jack is running fine now 
with jessie.

I'm really happy that this patch was already existing...

Thank you very much.

Martin

--- systemdeps.h.ORIGINAL	2014-07-19 15:34:15.0 +0200
+++ systemdeps.h	2015-05-15 00:29:04.945771837 +0200
@@ -120,7 +120,7 @@
 
 #endif /* __APPLE__ || __linux__ || __sun__ || sun */
 
-#if defined(__arm__)
+#if defined(__arm__) || defined(__ppc__) || defined(__powerpc__)
 #undef POST_PACKED_STRUCTURE
 #define POST_PACKED_STRUCTURE
 #endif /* __arm__ */


Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2014-03-21 Thread Adrian Knoth
On 11/04/13 15:49, Benoît Delcour wrote:

Hi!

 I just got myself a dual Powermac G4 out of a garbage bin,
 and immediately installed debian and jackd2.
 It fails with an obscure Bus error; since I also own a raspberry pi,
 I first tried to patch source with the same fix, and it works.
 See Usage of __attribute__((__packed__)) causes bus error on ARM:

OK, got you.

 Modifiying source to #undef  #define it unconditionnally did the trick.
 Bug report tagged as patch, but I don't know the correct flag.
 I can prepare an actual patch if/when I get the __powerpc__ 32bit-only
 thingy

Did you ever find the correct macro? We're close to a new jackd release
and would include this fix, if necessary (and correct).


Cheers


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#728710: [Jack-Devel] Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2014-03-21 Thread Robin Gareus
On 03/21/2014 11:25 AM, Adrian Knoth wrote:
 On 11/04/13 15:49, Benoît Delcour wrote:
 
 Hi!
 
 I just got myself a dual Powermac G4 out of a garbage bin,
 and immediately installed debian and jackd2.
 It fails with an obscure Bus error; since I also own a raspberry pi,
 I first tried to patch source with the same fix, and it works.
 See Usage of __attribute__((__packed__)) causes bus error on ARM:
 
 OK, got you.
 
 Modifiying source to #undef  #define it unconditionnally did the trick.
 Bug report tagged as patch, but I don't know the correct flag.
 I can prepare an actual patch if/when I get the __powerpc__ 32bit-only
 thingy
 
 Did you ever find the correct macro? We're close to a new jackd release
 and would include this fix, if necessary (and correct).

gcc -dM -E -  /dev/null

lists all compiler defines, I hazard a guess that __ppc__ is appropriate
(there's also __ppc64__).

But going by compiler defines alone is tricky (OSX for example allows
multi-arch builds, even though osx = 10.5 PPC users are rare these
days). This should be really be set/auto-detected during waf configure.

2c,
robin


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#728710: [Jack-Devel] Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2014-03-21 Thread Benoît Delcour
I will recompile jack this weekend to see if I can work that out.

Here are the defines of gcc:

gcc -dM -E -  /dev/null |grep ppc\|PPC\|powerpc
#define _ARCH_PPC 1
#define __PPC__ 1
#define __powerpc__ 1
#define __PPC 1
#define __powerpc 1
#define PPC 1
#define powerpc 1

Can we assume that __powerpc__ is 32 bit only?

On Fri, 21 Mar 2014 12:35:12 +0100
Robin Gareus ro...@gareus.org wrote:

 On 03/21/2014 11:25 AM, Adrian Knoth wrote:
  On 11/04/13 15:49, Benoît Delcour wrote:
  
  Hi!
  
  I just got myself a dual Powermac G4 out of a garbage bin,
  and immediately installed debian and jackd2.
  It fails with an obscure Bus error; since I also own a raspberry pi,
  I first tried to patch source with the same fix, and it works.
  See Usage of __attribute__((__packed__)) causes bus error on ARM:
  
  OK, got you.
  
  Modifiying source to #undef  #define it unconditionnally did the trick.
  Bug report tagged as patch, but I don't know the correct flag.
  I can prepare an actual patch if/when I get the __powerpc__ 32bit-only
  thingy
  
  Did you ever find the correct macro? We're close to a new jackd release
  and would include this fix, if necessary (and correct).
 
 gcc -dM -E -  /dev/null
 
 lists all compiler defines, I hazard a guess that __ppc__ is appropriate
 (there's also __ppc64__).
 
 But going by compiler defines alone is tricky (OSX for example allows
 multi-arch builds, even though osx = 10.5 PPC users are rare these
 days). This should be really be set/auto-detected during waf configure.
 
 2c,
 robin


-- 
Bricolage pour Tous, vendredi, 22h
Radio Campus, 106,6Mhz
www.campuslille.com


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#728710: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit

2013-11-04 Thread Benoît Delcour
Package: jackd2
Version: 1.9.9.5+20130622git7de15e7a-1
Severity: important
Tags: patch upstream

Content-Type: text/plain; charset=us-ascii
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: Benoit Delcour benoit.delc...@campuslille.com
To: Debian Bug Tracking System sub...@bugs.debian.org
Subject: jackd2: Bus error w/ POST_PACKED_STRUCTURE on powerpc G4 32bit
X-Debbugs-Cc: benoit.delc...@campuslille.com

Package: jackd2
Version: 1.9.9.5+20130622git7de15e7a-2
Severity: important
Tags: upstream patch

TL;DR: the Rasperry Pi Bus error strikes again.
Maybe not for the same causes, but the fix does work.

I just got myself a dual Powermac G4 out of a garbage bin,
and immediately installed debian and jackd2.
It fails with an obscure Bus error; since I also own a raspberry pi,
I first tried to patch source with the same fix, and it works.
See Usage of __attribute__((__packed__)) causes bus error on ARM:

http://trac.jackaudio.org/ticket/295

The issue seems to be related to arm;
however the bug *seems* to hit at the same time:
Jack: JackEngine::NotifyAddClient: name = system
Bus error

The Raspberry bug has been workaround in:
https://github.com/jackaudio/jack2/commit/2f7d512b419cb75cef2e27db872d641de6e7aa0d

+#if defined(__arm__)

+#undef POST_PACKED_STRUCTURE

+#define POST_PACKED_STRUCTURE

+#endif /* __arm__ */

The comment for the commit also states:
Since it's only used for mixed
32/64bit jackd installations and we're not facing such a scenario on
ARM, there's little use to worry more atm.

Modifiying source to #undef  #define it unconditionnally did the trick.
Bug report tagged as patch, but I don't know the correct flag.
I can prepare an actual patch if/when I get the __powerpc__ 32bit-only thingy


-- System Information:
Debian Release: 7.2
  APT prefers stable
  APT policy: (980, 'stable'), (500, 'stable-updates'), (500, 'testing')
Architecture: powerpc (ppc)

Kernel: Linux 3.2.0-4-powerpc-smp (SMP w/2 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages jackd2 depends on:
ii  coreutils  8.13-3.5
ii  debconf [debconf-2.0]  1.5.49
ii  libasound2 1.0.27.2-3
ii  libc6  2.17-93
ii  libdbus-1-31.6.8-1+deb7u1
ii  libexpat1  2.1.0-1
ii  libgcc11:4.7.2-5
ii  libjack-jackd2-0   1.9.9.5+20130622git7de15e7a-2
ii  libopus0   1.1~beta-3
ii  libreadline6   6.2+dfsg-0.1
ii  libsamplerate0 0.1.8-5
ii  libsndfile11.0.25-7
ii  libstdc++6 4.7.2-5
ii  multiarch-support  2.13-38
ii  python 2.7.3-4+deb7u1
ii  python-dbus1.1.1-1

Versions of packages jackd2 recommends:
ii  jackd2-firewire  1.9.9.5+20130622git7de15e7a-2
ii  libpam-modules   1.1.3-7.1
ii  qjackctl 0.3.9-2

Versions of packages jackd2 suggests:
ii  jack-tools   20101210-2
ii  meterbridge  0.9.2-11

-- debconf information:
* jackd/tweak_rt_limits: true

-- System Information:
Debian Release: 7.2
  APT prefers stable
  APT policy: (990, 'stable'), (980, 'testing'), (500, 'stable'), (450, 
'unstable'), (50, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.10-2-rt-amd64 (SMP w/2 CPU cores; PREEMPT)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages jackd2 depends on:
ii  coreutils  8.13-3.5
ii  debconf [debconf-2.0]  1.5.49
ii  libasound2 1.0.27.2-3
ii  libc6  2.17-93
ii  libdbus-1-31.6.16-1
ii  libexpat1  2.1.0-1
ii  libgcc11:4.7.2-5
ii  libjack-jackd2-0   1.9.9.5+20130622git7de15e7a-1
ii  libopus0   1.1~beta-3
ii  libreadline6   6.2+dfsg-0.1
ii  libsamplerate0 0.1.8-5
ii  libsndfile11.0.25-7
ii  libstdc++6 4.7.2-5
ii  multiarch-support  2.13-38
ii  python 2.7.3-4+deb7u1
ii  python-dbus1.1.1-1

Versions of packages jackd2 recommends:
ii  jackd2-firewire  1.9.9.5+20130622git7de15e7a-1
ii  libpam-modules   1.1.3-7.1
ii  qjackctl 0.3.10-2

Versions of packages jackd2 suggests:
ii  jack-tools   20101210-2
ii  meterbridge  0.9.2-11

-- debconf information:
* jackd/tweak_rt_limits: true


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org