Hello, On Thu, 13 Jun 2013, Alexander Frolkin wrote: > Hi, > > > > +{ > > > + __be16 port; > > > + struct tcphdr _tcph, *th; > > > + struct udphdr _udph, *uh; > > > + sctp_sctphdr_t _sctph, *sh; > > > + > > > + switch (svc->protocol) { > > Use iph->protocol instead of svc->protocol because not > > all services have correct protocol. > > In order for this to work, I would have to change > ip_vs_fill_iph_addr_only in ip_vs_sh_schedule to ip_vs_fill_iph_skb (or > add an ip_vs_fill_iph_addr_proto_only function). Obviously, I can do > this only when the flag is enabled. > > I can also get it from skb, but that will make the code more > complicated. > > What do you think? May be I have to remove this ip_vs_fill_iph_addr_only function, we should provide iph to schedulers. I'll post patch this weekend. Regards -- Julian Anastasov <ja@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html