svn commit: r255187 - in head/sys: conf crypto/aesni modules/aesni
Konstantin Belousov
kostikbel at gmail.com
Tue Sep 10 11:21:36 UTC 2013
On Mon, Sep 09, 2013 at 11:41:49PM -0700, John-Mark Gurney wrote:
> David O'Brien wrote this message on Mon, Sep 09, 2013 at 23:01 -0700:
> > On Tue, Sep 03, 2013 at 06:31:23PM +0000, John-Mark Gurney wrote:
> > > Log:
> > > Use the fact that the AES-NI instructions can be pipelined to improve
> > > performance... Use SSE2 instructions for calculating the XTS tweek
> > > factor... Let the compiler do more work and handle register allocation
> > > by using intrinsics, now only the key schedule is in assembly...
> >
> > Hi John-Mark,
> > Unfortunately this does not work with /usr/bin/gcc (which I still use as
> > /usr/bin/cc on this old IBM T60 laptop due to clang's enormous pestimation
> > on 'make world' and build times.
> >
> > Please fix this ASAP or back it out out of 10-CURRENT as it does not work
> > with /usr/bin/gcc:
> >
> > 1. /usr/bin/gcc cannot handle the "-maes" option.
> > I committed r255440 to address this.
>
> Please back that out until you have an understand of what the real
> problem is... I'm sad that the commit was approved w/o bothering to
> attempt to figure out the problem...
>
> > 2. /usr/src/sys/modules/aesni/../../crypto/aesni/aesencdec.h:30:23: error: wmmintrin.h: No such file or directory
> > and then a cascade of errors follow.
>
> Sounds like you don't have the latest in-tree gcc... I've been
> compiling the AES module w/ the in tree gcc for a while.. This is
> w/ the changes in r255185...
>
> > For now, I've sent a patch to re@ for approval to remove the 'aesni'
> > module from the build if not using clang. But the LINT build is
> > still broken with /usr/bin/gcc.
>
> I'll strongly object to this change...
Sorry for my mistake, I agree with you.
IMO this change should be reverted.
>
> > > Replace .byte hard coded instructions w/ the proper instructions now
> > > that both clang and gcc support them...
> >
> > Is this out-of-tree latest GCC that supports this?
>
> Nope, in-tree... Sounds like you haven't installed the latest version
> of gcc in tree...
>
> It could be a problem with a bad interaction w/ delete-old as glebius
> added the wmmintrin.h header to the delete-old target in r255354...
>
> I'll take a closer look at this in the morning, but next time please
> attempt to contact the author before taking such actions...
>
> --
> John-Mark Gurney Voice: +1 415 225 5579
>
> "All that I will do, has been done, All that I have, has not."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-src-all/attachments/20130910/dd72d18e/attachment.sig>
More information about the svn-src-all
mailing list