Thanks for reviewing. I did miss it. > -----Original Messages----- > From: "Dmitry Popov" <[email protected]> > Sent Time: Wednesday, July 30, 2014 > To: "Zhouyi Zhou" <[email protected]> > Cc: [email protected], [email protected], [email protected], > [email protected], [email protected], [email protected], > [email protected], [email protected], [email protected], > [email protected], [email protected], > [email protected], "Zhouyi Zhou" <[email protected]> > Subject: Re: [PATCH net-next] pktgen: add ttl option for pktgen > > On Wed, 30 Jul 2014 16:21:26 +0800 > Zhouyi Zhou <[email protected]> wrote: > > > I think it is useful to add ttl option for pktgen, for example > > if a some ISP want to test its network quality, it could set > > ttl so that the tested links get the packet while end users won't > > get it. > > It seems you're missing > > --- a/net/core/pktgen.c > +++ b/net/core/pktgen.c > @@ -2799,7 +2799,7 @@ static struct sk_buff *fill_packet_ipv4(struct > net_device *odev, > > iph->ihl = 5; > iph->version = 4; > - iph->ttl = 32; > + iph->ttl = pkt_dev->ttl; > iph->tos = pkt_dev->tos; > iph->protocol = IPPROTO_UDP; /* UDP */ > iph->saddr = pkt_dev->cur_saddr;
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
