ip_append_data函数中一个难以理解的条件判断【已解决】

jiufei19

之前的帖子一直无法解决这个疑问，在此我将此问题单独提出，希望能得到大家的帮助。

多个内核版本均如此，包括V3.x，只是对应代码行数略微有差别而已，在此我以版本V2.6.23为例

在ip_append_data()函数的第997行有如下判断

   ...           ...
   994             if (page && (left = PAGE_SIZE - off) > 0) {
   995                 if (copy >= left)
   996                     copy = left;
   997                 if (page != frag->page) {
   998                     if (i == MAX_SKB_FRAGS) {
   999                         err = -EMSGSIZE;
  1000                         goto error;
  1001                     }

  ...            ...


第997行，为啥要进行此判断，我仔细阅读了代码，假设了各种场景，都只能得到page==frag->page的情况，到底什么样的场景会导致这2者不相等呢？

瀚海书香

回复 1# jiufei19

话说现在都是4.x的代码了，checkout到2.6.23看了一下。

1、弄清楚两个page的含义
    page=sk->sk_sndmsg_page 是指本传输块最近一次分配的页面
    frag->page 是指skb分片数据的最后一页

2、一般情况下，这两个页面是相同的

3、但是在特殊情况下，page跟frag->page是不同的，比如

    看net/ipv4/tcp.c 中的tcp_sndmsg函数，在第789行，申请page成功，但是数据copy失败的情况下。

    page=sk->sk_sndmsg_page 会指向最新分配的page;//796行
    而frag->page 由于数据copy没有成功，并不会更新指向这个page；//只有copy数据成功后，才会更新frag-page, 807行

   

jiufei19

回复 2# 瀚海书香

感谢版主。
不过我有点疑问，我的问题是在ip_append_data()函数中出现的问题，而版主给出的例子是在tcp_sendmsg函数中，我核对了下，tcp_sendmsg好像和ip_append_data没有关系呢？

瀚海书香

回复 3# jiufei19 同一个sock，可以被tcp_sendmsg和ip_append_data调用，考虑如下调用逻辑：

1、申请一个sock
2、调用tcp_sendmsg发送数据【copy失败】
3、调用ip_append_data直接发送raw数据【触发上述场景】




另外，Google的Eric Dumazet将tcp_sendmsg中cache page的管理，将per-socket转换为per-task。

jiufei19

回复 4# 瀚海书香

再次感谢版主的热心解答。
虽然，逻辑上该问题的产生一定是某次使用page进行发送时，因某种原因没有调整frags->page为sk->sk_sndmsg_page才导致的，但是因为我没有找到具体什么应用或内核的什么具体操作会这样，故产生了这一疑问，而版主给出了可能出现这一问题的场景，但是，似乎仍然是一个假想的场景，而非内核中具体一个存在的调用场景，我的理解正确吗？

瀚海书香

回复 5# jiufei19

1、既然ip_append_data这个接口是可以给上层调用的，那么这个接口的内部就要处理这种可能发生的场景。2、看一下这个处理逻辑
     tcp_v4_send_ack --->ip_send_unicast_reply--->ip_appsend_data

jiufei19

回复 6# 瀚海书香

似乎有点明白了，非常感谢版主的解惑

nswcfd

per task的page(s)是怎么在（单个task的）多个sock之间共享的？
会出现同个page存放不同sock的data的情况么？

nswcfd

好像没有理由不可以？