conn_flags definition in ip_vs.h is in contradiction with that in conn.h

[worldpeace365]

conn_flags definition in ip_vs.h is in contradiction with that in conn.h. It will cause the usr.conn_flags was overwrited by conn->flags.
see follow:
for example,
DPVS_CONN_F_IN_TIMER = 0x0200,
#define IP_VS_CONN_F_OUT_SEQ 0x0200 /* must do output seq adjust */

DPVS_CONN_F_NOFASTXMIT = 0x2000,
#define IP_VS_CONN_F_ONE_PACKET 0x2000 /* forward only one packet */

ip_vs.h
/*

• IPVS Connection Flags
• Only flags 0..15 are sent to backup server
  /
  #define IP_VS_CONN_F_FWD_MASK 0x0007 / mask for the fwd methods /
  #define IP_VS_CONN_F_MASQ 0x0000 / masquerading/NAT /
  #define IP_VS_CONN_F_LOCALNODE 0x0001 / local node /
  #define IP_VS_CONN_F_TUNNEL 0x0002 / tunneling /
  #define IP_VS_CONN_F_DROUTE 0x0003 / direct routing /
  #define IP_VS_CONN_F_BYPASS 0x0004 / cache bypass /
  #define IP_VS_CONN_F_FULLNAT 0x0005 / full nat mode /
  #define IP_VS_CONN_F_SNAT 0x0006 / SNAT mode /
  #define IP_VS_CONN_F_SYNC 0x0020 / entry created by sync /
  #define IP_VS_CONN_F_HASHED 0x0040 / hashed entry /
  #define IP_VS_CONN_F_NOOUTPUT 0x0080 / no output packets /
  #define IP_VS_CONN_F_INACTIVE 0x0100 / not established /
  #define IP_VS_CONN_F_OUT_SEQ 0x0200 / must do output seq adjust /
  #define IP_VS_CONN_F_IN_SEQ 0x0400 / must do input seq adjust /
  #define IP_VS_CONN_F_SEQ_MASK 0x0600 / in/out sequence mask /
  #define IP_VS_CONN_F_NO_CPORT 0x0800 / no client port set yet /
  #define IP_VS_CONN_F_TEMPLATE 0x1000 / template, not connection /
  #define IP_VS_CONN_F_ONE_PACKET 0x2000 / forward only one packet */

---

conn.h
enum {
DPVS_CONN_F_HASHED = 0x0040,
DPVS_CONN_F_REDIRECT_HASHED = 0x0080,
DPVS_CONN_F_INACTIVE = 0x0100,
DPVS_CONN_F_IN_TIMER = 0x0200,
DPVS_CONN_F_SYNPROXY = 0x8000,
DPVS_CONN_F_TEMPLATE = 0x1000,
DPVS_CONN_F_NOFASTXMIT = 0x2000,
};

usr conn_flags set
case 'i':
set_option(options, OPT_FORWARD);
ce->dest.user.conn_flags = IP_VS_CONN_F_TUNNEL;
break;
case 'g':
set_option(options, OPT_FORWARD);
ce->dest.user.conn_flags = IP_VS_CONN_F_DROUTE;
break;
case 'b':
set_option(options, OPT_FORWARD);
ce->dest.user.conn_flags = IP_VS_CONN_F_FULLNAT;
break;
case 'J':
set_option(options, OPT_FORWARD);
ce->dest.user.conn_flags = IP_VS_CONN_F_SNAT;
break;
case 'm':
set_option(options, OPT_FORWARD);
ce->dest.user.conn_flags = IP_VS_CONN_F_MASQ;

copy to conn->flags
static int dp_vs_conn_bind_dest(struct dp_vs_conn *conn,
struct dp_vs_dest dest)
{
/ ATTENTION:
* Initial state of conn should be INACTIVE, with conn->inactconns=1 and
* conn->actconns=0. We should not increase conn->actconns except in session
* sync.Generally, the INACTIVE and SYN_PROXY flags are passed down from
* the dest here. */
conn->flags |= rte_atomic16_read(&dest->conn_flags);

[ywc689]

fix by pr #714