Bug 791 - [ANKK:4000] / APV /Webagent + SLB Abnormal MSS value

Review Request #378 — Created July 29, 2024 and submitted

kdutta
APV10
rel_apv_10_7
791
mmiriam, pradeep, prajesh, satyendra, tanya, timlai

Bug 791 - [ANKK:4000] / APV /Webagent + SLB Abnormal MSS value

Configure APV

A2JJNpshK3#sh slb all
slb real tcp "RS_WS" 192.168.162.155 80 0 icmp 3 3
slb mode activeclose on "RS_WS"
slb virtual tcp "VS_WS" 192.168.162.195 80 arp 0
slb virtual health off
slb policy static "VS_WS" "RS_WS"
slb directfwd on
slb directfwd syncache off
slb directfwd statistics on
slb directfwd on "VS_WS"
slb mode ircookie plainname
slb mode icookie always
slb mode activeclose off
slb mode keepport off
slb tcpoption mss "VS_WS" 1200
slb ftps datachannel 0
slb ftp datachannel 20
slb tuxedo portposition 78
slb mode regexcase off
slb mode lineup off "VS_WS"
slb overload persistence off
slb sip persistence timeout 120

A2JJNpshK3#show webagent service
webagent service "w1" 192.168.162.195 2001 0

A2JJNpshK3#show slb tcpoption
slb tcpoption mss "VS_WS" size 1200

A2JJNpshK3#

Test with pacp from Origin Server

Take pcap on origin server you can see received SYN packet has the same mss 1200.


Description From Last Updated

ntohs(orig_mss) >= FASTTCP_AN_LOOPBACK_MSS Should be a better check?

prajeshprajesh

Is this passing interface of the incoming packet, so you can use it in line mtu = real_ifp->if_mtu?

tanyatanya

it called with null in couple of more places. please check those too.

tanyatanya

we set to default mss if vs is not configured, because the mss value from loopback is too large to …

tanyatanya
kdutta
kdutta
kdutta
tanya
  1. 
      
  2. Is this passing interface of the incoming packet, so you can use it in line mtu = real_ifp->if_mtu?

  3. 
      
tanya
  1. Ship It!
  2. 
      
tanya
  1. 
      
  2. it called with null in couple of more places. please check those too.

  3. 
      
prajesh
  1. 
      
  2. ntohs(orig_mss) >= FASTTCP_AN_LOOPBACK_MSS Should be a better check?

    1. I have used the code from different checkin, this part i have not added.

  3. 
      
prajesh
  1. Fix it and ship it,.

  2. 
      
kdutta
kdutta
kdutta
prajesh
  1. Ship It!
  2. 
      
timlai
  1. Ship It!
  2. 
      
kdutta
tanya
  1. 
      
  2. we set to default mss if vs is not configured, because the mss value from loopback is too large to keep anyway.

    1. I have set default value as uint16_t orig_mss, new_mss = htons(FASTTCP_AN_MSS); while declaration. If we read new value then only i am updating with new value else it will use default value.

  3. 
      
tanya
  1. Ship It!
  2. 
      
kdutta
kdutta
kdutta
kdutta
Review request changed

Status: Closed (submitted)

Loading...