Opened 16 years ago

Closed 16 years ago

Last modified 8 months ago

#8300 closed patch (wontfix)

Disable unaligned accesses on Alpha

Reported by: SF/mellum Owned by: fingolfin
Priority: normal Component: Ports
Keywords: Cc:


On Alpha, unaligned accesses don't crash, but are

Ticket imported from: #847463. Ticket imported from: patches/405.

Attachments (1)

alpha-unaligned.patch (628 bytes) - added by SF/mellum 16 years ago.

Download all attachments as: .zip

Change History (8)

Changed 16 years ago by SF/mellum

Attachment: alpha-unaligned.patch added

comment:1 Changed 16 years ago by fingolfin

So what exactly is the point of this patch? What does it do, and
why do you think it should be added to ScummVM?

comment:2 Changed 16 years ago by fingolfin

Status: newpending

comment:3 Changed 16 years ago by SF/mellum

Status: pendingnew

comment:4 Changed 16 years ago by SF/mellum

The point is to avoid slowdowns of a factor of 100 and spamming
the syslog file with warnings. For this, the patch shortcuts
the test
which tries to check whether one can get away with unaligned

The proper method would IMHO be to have a list of architectures
which are known not to have any problems with unaligned
but that was turned down when I suggested it.

comment:5 Changed 16 years ago by fingolfin

Owner: set to fingolfin
Resolution: wontfix
Status: newclosed

comment:6 Changed 16 years ago by fingolfin

A hardcoded list of architectures hardly ever is a "proper way". It
means that you get troubles whenever somebody tries to get stuff
working on an so far untested system.
The reason that autoconf is so successful and flexible is that it
checks for features, not for operating systems/processors/.... I
very much strongly disagree with your statement that "a list of
architectures" is the "proper method" to handle this. It most
definitely is *not* the proper way, isiolated.

However, *augmenting* an automatic detection by a few hard
coded cases is an option. However that shouldn't be implemented
by relying on some preprocessor flag in the C check, rather the
script should check the uname value to detect the Alpha

comment:7 Changed 8 months ago by digitall

Component: Ports
Note: See TracTickets for help on using tickets.