Skip Menu |

This queue is for tickets about the Digest-MD5 CPAN distribution.

Report information
The Basics
Id: 77919
Status: resolved
Priority: 0/
Queue: Digest-MD5
People
Owner: Nobody in particular
Requestors: zefram [...] fysh.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Jun 20 07:20:22 2012 zefram [...] fysh.org - Ticket created
Subject: U32 alignment on 64-bit arch
Date: Wed, 20 Jun 2012 12:20:03 +0100
To: bug-Digest-MD5 [...] rt.cpan.org
From: Zefram <zefram [...] fysh.org>
Digest-MD5's U32 alignment test perceives any 64-bit architecture as an "unusual byte order", and unnecessarily plays safe, losing some performance. Attached patch fixes. -zefram

Message body is not shown because sender requested not to inline it.

  Wed Jun 20 07:50:21 2012 zefram [...] fysh.org - Correspondence added
Subject: Re: [rt.cpan.org #77919] AutoReply: U32 alignment on 64-bit arch
Date: Wed, 20 Jun 2012 12:50:09 +0100
To: Bugs in Digest-MD5 via RT <bug-Digest-MD5 [...] rt.cpan.org>
From: Zefram <zefram [...] fysh.org>
Second thoughts: some of the code in MD5.xs needs to be edited as well to cope with a 64-bit BYTEORDER. I'm somewhat amazed that this didn't show up immediately in the test suite. -zefram
  Sun Mar 27 04:52:36 2016 RURBAN [...] cpan.org - Correspondence added
On Wed Jun 20 07:50:21 2012, zefram@fysh.org wrote: Show quoted text
> Second thoughts: some of the code in MD5.xs needs to be edited as well > to cope with a 64-bit BYTEORDER. I'm somewhat amazed that this didn't > show up immediately in the test suite.
perl5 core took over the same wrong d_u32align probe which fails on all intel cpus, unless the AC bit (alignment check) in EFLAGS is set. See http://orchistro.tistory.com/206 how to enforce a SIGBUS on intel also, e.g. when U32_ALIGNMENT_REQUIRED is wrongly set. I do this to check on fast intel for possible SIGBUS errors for alignment strict platforms (mips,sparc,ppc). See https://github.com/gisle/digest-md5/pull/13 It needs to link with -lperl -- Reini Urban
  Sun Mar 27 04:52:36 2016 The RT System itself - Status changed from 'new' to 'open'
  Mon Oct 05 10:46:01 2020 TODDR [...] cpan.org - Correspondence added
Ticket migrated to github as https://github.com/Dual-Life/digest-md5/issues/8
  Mon Oct 05 10:46:02 2020 TODDR [...] cpan.org - Status changed from 'open' to 'resolved'