Thanks! It seems like network code has preference for net_random() but they are the same now.
Signed-off-by: Robert Olsson <[EMAIL PROTECTED]> Cheers. --ro Stephen Hemminger writes: > Can use random32() now. > > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> > > --- > net/core/pktgen.c | 52 > +++++++++++++++++++--------------------------------- > 1 file changed, 19 insertions(+), 33 deletions(-) > > --- pktgen.orig/net/core/pktgen.c 2007-02-26 14:34:36.000000000 -0800 > +++ pktgen/net/core/pktgen.c 2007-02-26 14:39:53.000000000 -0800 > @@ -464,17 +464,6 @@ > return tmp; > } > > -static inline u32 pktgen_random(void) > -{ > -#if 0 > - __u32 n; > - get_random_bytes(&n, 4); > - return n; > -#else > - return net_random(); > -#endif > -} > - > static inline __u64 getCurMs(void) > { > struct timeval tv; > @@ -2091,7 +2080,7 @@ > int flow = 0; > > if (pkt_dev->cflows) { > - flow = pktgen_random() % pkt_dev->cflows; > + flow = random32() % pkt_dev->cflows; > > if (pkt_dev->flows[flow].count > pkt_dev->lflow) > pkt_dev->flows[flow].count = 0; > @@ -2103,7 +2092,7 @@ > __u32 tmp; > > if (pkt_dev->flags & F_MACSRC_RND) > - mc = pktgen_random() % (pkt_dev->src_mac_count); > + mc = random32() % pkt_dev->src_mac_count; > else { > mc = pkt_dev->cur_src_mac_offset++; > if (pkt_dev->cur_src_mac_offset > > @@ -2129,7 +2118,7 @@ > __u32 tmp; > > if (pkt_dev->flags & F_MACDST_RND) > - mc = pktgen_random() % (pkt_dev->dst_mac_count); > + mc = random32() % pkt_dev->dst_mac_count; > > else { > mc = pkt_dev->cur_dst_mac_offset++; > @@ -2156,24 +2145,23 @@ > for(i = 0; i < pkt_dev->nr_labels; i++) > if (pkt_dev->labels[i] & MPLS_STACK_BOTTOM) > pkt_dev->labels[i] = MPLS_STACK_BOTTOM | > - ((__force __be32)pktgen_random() & > + ((__force __be32)random32() & > htonl(0x000fffff)); > } > > if ((pkt_dev->flags & F_VID_RND) && (pkt_dev->vlan_id != 0xffff)) { > - pkt_dev->vlan_id = pktgen_random() % 4096; > + pkt_dev->vlan_id = random32() & (4096-1); > } > > if ((pkt_dev->flags & F_SVID_RND) && (pkt_dev->svlan_id != 0xffff)) { > - pkt_dev->svlan_id = pktgen_random() % 4096; > + pkt_dev->svlan_id = random32() & (4096 - 1); > } > > if (pkt_dev->udp_src_min < pkt_dev->udp_src_max) { > if (pkt_dev->flags & F_UDPSRC_RND) > - pkt_dev->cur_udp_src = > - ((pktgen_random() % > - (pkt_dev->udp_src_max - pkt_dev->udp_src_min)) + > - pkt_dev->udp_src_min); > + pkt_dev->cur_udp_src = random32() % > + (pkt_dev->udp_src_max - pkt_dev->udp_src_min) > + + pkt_dev->udp_src_min; > > else { > pkt_dev->cur_udp_src++; > @@ -2184,10 +2172,9 @@ > > if (pkt_dev->udp_dst_min < pkt_dev->udp_dst_max) { > if (pkt_dev->flags & F_UDPDST_RND) { > - pkt_dev->cur_udp_dst = > - ((pktgen_random() % > - (pkt_dev->udp_dst_max - pkt_dev->udp_dst_min)) + > - pkt_dev->udp_dst_min); > + pkt_dev->cur_udp_dst = random32() % > + (pkt_dev->udp_dst_max - pkt_dev->udp_dst_min) > + + pkt_dev->udp_dst_min; > } else { > pkt_dev->cur_udp_dst++; > if (pkt_dev->cur_udp_dst >= pkt_dev->udp_dst_max) > @@ -2202,7 +2189,7 @@ > saddr_max))) { > __u32 t; > if (pkt_dev->flags & F_IPSRC_RND) > - t = ((pktgen_random() % (imx - imn)) + imn); > + t = random32() % (imx - imn) + imn; > else { > t = ntohl(pkt_dev->cur_saddr); > t++; > @@ -2223,14 +2210,13 @@ > __be32 s; > if (pkt_dev->flags & F_IPDST_RND) { > > - t = pktgen_random() % (imx - imn) + imn; > + t = random32() % (imx - imn) + imn; > s = htonl(t); > > while (LOOPBACK(s) || MULTICAST(s) > || BADCLASS(s) || ZERONET(s) > || LOCAL_MCAST(s)) { > - t = (pktgen_random() % > - (imx - imn)) + imn; > + t = random32() % (imx - imn) + > imn; > s = htonl(t); > } > pkt_dev->cur_daddr = s; > @@ -2262,7 +2248,7 @@ > > for (i = 0; i < 4; i++) { > pkt_dev->cur_in6_daddr.s6_addr32[i] = > - (((__force __be32)pktgen_random() | > + (((__force __be32)random32() | > pkt_dev->min_in6_daddr.s6_addr32[i]) & > pkt_dev->max_in6_daddr.s6_addr32[i]); > } > @@ -2272,9 +2258,9 @@ > if (pkt_dev->min_pkt_size < pkt_dev->max_pkt_size) { > __u32 t; > if (pkt_dev->flags & F_TXSIZE_RND) { > - t = ((pktgen_random() % > - (pkt_dev->max_pkt_size - pkt_dev->min_pkt_size)) > - + pkt_dev->min_pkt_size); > + t = random32() % > + (pkt_dev->max_pkt_size - pkt_dev->min_pkt_size) > + + pkt_dev->min_pkt_size; > } else { > t = pkt_dev->cur_pkt_size + 1; > if (t > pkt_dev->max_pkt_size) > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html