[Home]

Summary:ASTERISK-15700: [patch] Alignment trap on ARM processor on calculating cost of codec
Reporter:Michael Evdokimov (michaelevdokimov)Labels:
Date Opened:2010-02-26 08:40:15.000-0600Date Closed:2010-06-03 22:22:12
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) asterisk.patch
Description:Alignment trap on ARM processor on calculating cost of codec
Comments:By: Leif Madsen (lmadsen) 2010-03-03 09:40:22.000-0600

Can you better explain what the issue is? Thanks!

By: Michael Evdokimov (michaelevdokimov) 2010-03-03 10:05:18.000-0600

at file /include/asterisk/sli.h
linear samples are described. Type "uint8_t" is used. However on "ARM" platform data must be aligned by 2 bytes because at codecs this data is converted to "uint16_t". So I think type "uint16_t" should be used to declare this samples.

By: Leif Madsen (lmadsen) 2010-03-03 13:45:13.000-0600

Thanks for the additional information!

By: Digium Subversion (svnbot) 2010-06-03 21:58:55

Repository: asterisk
Revision: 267862

U   trunk/include/asterisk/slin.h

------------------------------------------------------------------------
r267862 | tilghman | 2010-06-03 21:58:55 -0500 (Thu, 03 Jun 2010) | 5 lines

As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.

(closes issue ASTERISK-15700)
Reported by: michaelevdokimov

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=267862

By: Digium Subversion (svnbot) 2010-06-03 22:20:47

Repository: asterisk
Revision: 267877

U   trunk/include/asterisk/slin.h

------------------------------------------------------------------------
r267877 | tilghman | 2010-06-03 22:20:47 -0500 (Thu, 03 Jun 2010) | 8 lines

As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.

(closes issue ASTERISK-15700)
Reported by: michaelevdokimov
Patches:
      asterisk.patch uploaded by michaelevdokimov (license 997)
Tested by: michaelevdokimov

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=267877

By: Digium Subversion (svnbot) 2010-06-03 22:22:11

Repository: asterisk
Revision: 267883

_U  branches/1.6.2/
U   branches/1.6.2/include/asterisk/slin.h

------------------------------------------------------------------------
r267883 | tilghman | 2010-06-03 22:22:11 -0500 (Thu, 03 Jun 2010) | 22 lines

Merged revisions 267862,267877 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r267862 | tilghman | 2010-06-03 21:58:55 -0500 (Thu, 03 Jun 2010) | 5 lines
 
 As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.
 
 (closes issue ASTERISK-15700)
  Reported by: michaelevdokimov
........
 r267877 | tilghman | 2010-06-03 22:20:47 -0500 (Thu, 03 Jun 2010) | 8 lines
 
 As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.
 
 (closes issue ASTERISK-15700)
  Reported by: michaelevdokimov
  Patches:
        asterisk.patch uploaded by michaelevdokimov (license 997)
  Tested by: michaelevdokimov
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=267883