Thanks for reviewing. I did miss it. > -----Original Messages----- > From: "Dmitry Popov" <ixaphire@xxxxxxxxxx> > Sent Time: Wednesday, July 30, 2014 > To: "Zhouyi Zhou" <zhouzhouyi@xxxxxxxxx> > Cc: rdunlap@xxxxxxxxxxxxx, davem@xxxxxxxxxxxxx, minipli@xxxxxxxxxxxxxx, brouer@xxxxxxxxxx, steffen.klassert@xxxxxxxxxxx, fan.du@xxxxxxxxxxxxx, dborkman@xxxxxxxxxx, fengguang.wu@xxxxxxxxx, tgraf@xxxxxxx, linux-doc@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, "Zhouyi Zhou" <yizhouzhou@xxxxxxxxx> > Subject: Re: [PATCH net-next] pktgen: add ttl option for pktgen > > On Wed, 30 Jul 2014 16:21:26 +0800 > Zhouyi Zhou <zhouzhouyi@xxxxxxxxx> 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-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
