public inbox for trinity@vger.kernel.org
 help / color / mirror / Atom feed
* Re: rand64 change
  2013-07-12 21:48 rand64 change Vince Weaver
@ 2013-07-12 21:46 ` Dave Jones
  0 siblings, 0 replies; 2+ messages in thread
From: Dave Jones @ 2013-07-12 21:46 UTC (permalink / raw)
  To: Vince Weaver; +Cc: trinity

On Fri, Jul 12, 2013 at 05:48:37PM -0400, Vince Weaver wrote:
 > 
 > I noticed the following change.  The perf_event_open() syscall has various 
 > 64-bit fields (that are 64-bit even on 32-bit machines) that I set with 
 > rand64().  Do I need to change something to get that to work?

Ah, I forgot about that. No, I'll change it so it returns u64 even on 32bit.

	Dave
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* rand64 change
@ 2013-07-12 21:48 Vince Weaver
  2013-07-12 21:46 ` Dave Jones
  0 siblings, 1 reply; 2+ messages in thread
From: Vince Weaver @ 2013-07-12 21:48 UTC (permalink / raw)
  To: Dave Jones; +Cc: trinity


I noticed the following change.  The perf_event_open() syscall has various 
64-bit fields (that are 64-bit even on 32-bit machines) that I set with 
rand64().  Do I need to change something to get that to work?

Vince

commit c5bcbb108fe920851b853180849153ebcacb300a
Author: Dave Jones <davej@redhat.com>
Date:   Fri Jul 12 16:09:54 2013 -0400

    limit rand64 to 32bits when building on 32bit.
    
    I might later change this to make this function return u64 if necessary,
    but for now, the rand64() consumers on 32bit don't care because
    they work on ulongs.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-07-12 21:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-12 21:48 rand64 change Vince Weaver
2013-07-12 21:46 ` Dave Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox