Skip to content

Add rdseed instruction#109

Open
vegard wants to merge 1 commit intovmt:masterfrom
vegard:rdseed
Open

Add rdseed instruction#109
vegard wants to merge 1 commit intovmt:masterfrom
vegard:rdseed

Conversation

@vegard
Copy link
Contributor

@vegard vegard commented Jan 1, 2016

The rdseed instruction is really similar to rdrand, compare:

http://www.felixcloutier.com/x86/RDRAND.html
http://www.felixcloutier.com/x86/RDSEED.html

The only difference is /reg=7 instead of /reg=6.

However, there is still something wrong with the operand decoding for
both instructions:

objdump:

 0:   66 0f c7 f3             rdrand %bx
 4:   0f c7 f3                rdrand %ebx
 7:   48 0f c7 f3             rdrand %rbx
 b:   66 0f c7 fb             rdseed %bx
 f:   0f c7 fb                rdseed %ebx
12:   48 0f c7 fb             rdseed %rbx

udcli:

0000000000000000 660fc7f3         rdrand rbx
0000000000000004 0fc7f3           rdrand rbx
0000000000000007 480fc7f3         rdrand rbx
000000000000000b 660fc7fb         rdseed rbx
000000000000000f 0fc7fb           rdseed rbx
0000000000000012 480fc7fb         rdseed rbx

This is still an improvement, however.

The rdseed instruction is really similar to rdrand, compare:

http://www.felixcloutier.com/x86/RDRAND.html
http://www.felixcloutier.com/x86/RDSEED.html

The only difference is /reg=7 instead of /reg=6.

However, there is still something wrong with the operand decoding for
both instructions:

objdump:

   0:   66 0f c7 f3             rdrand %bx
   4:   0f c7 f3                rdrand %ebx
   7:   48 0f c7 f3             rdrand %rbx
   b:   66 0f c7 fb             rdseed %bx
   f:   0f c7 fb                rdseed %ebx
  12:   48 0f c7 fb             rdseed %rbx

udcli:

0000000000000000 660fc7f3         rdrand rbx
0000000000000004 0fc7f3           rdrand rbx
0000000000000007 480fc7f3         rdrand rbx
000000000000000b 660fc7fb         rdseed rbx
000000000000000f 0fc7fb           rdseed rbx
0000000000000012 480fc7fb         rdseed rbx

This is still an improvement, however.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant