)]}'
{
  "commit": "efe790502be85c60daa65c8aa51f05c333186e12",
  "tree": "c61c53f5d4a2a0f61e202c8e80ab810e51dba5d7",
  "parents": [
    "0b86a2a1e5807326f8eb44e9919d0baadeda3a69"
  ],
  "author": {
    "name": "Hamish Martin",
    "email": "hamish.martin@alliedtelesis.co.nz",
    "time": "Fri Apr 29 10:40:24 2016 -0400"
  },
  "committer": {
    "name": "David S. Miller",
    "email": "davem@davemloft.net",
    "time": "Sun May 01 21:03:30 2016 -0400"
  },
  "message": "tipc: only process unicast on intended node\n\nWe have observed complete lock up of broadcast-link transmission due to\nunacknowledged packets never being removed from the \u0027transmq\u0027 queue. This\nis traced to nodes having their ack field set beyond the sequence number\nof packets that have actually been transmitted to them.\nConsider an example where node 1 has sent 10 packets to node 2 on a\nlink and node 3 has sent 20 packets to node 2 on another link. We\nsee examples of an ack from node 2 destined for node 3 being treated as\nan ack from node 2 at node 1. This leads to the ack on the node 1 to node\n2 link being increased to 20 even though we have only sent 10 packets.\nWhen node 1 does get around to sending further packets, none of the\npackets with sequence numbers less than 21 are actually removed from the\ntransmq.\nTo resolve this we reinstate some code lost in commit d999297c3dbb (\"tipc:\nreduce locking scope during packet reception\") which ensures that only\nmessages destined for the receiving node are processed by that node. This\nprevents the sequence numbers from getting out of sync and resolves the\npacket leakage, thereby resolving the broadcast-link transmission\nlock-ups we observed.\n\nWhile we are aware that this change only patches over a root problem that\nwe still haven\u0027t identified, this is a sanity test that it is always\nlegitimate to do. It will remain in the code even after we identify and\nfix the real problem.\n\nReviewed-by: Chris Packham \u003cchris.packham@alliedtelesis.co.nz\u003e\nReviewed-by: John Thompson \u003cjohn.thompson@alliedtelesis.co.nz\u003e\nSigned-off-by: Hamish Martin \u003chamish.martin@alliedtelesis.co.nz\u003e\nSigned-off-by: Jon Maloy \u003cjon.maloy@ericsson.com\u003e\nSigned-off-by: David S. Miller \u003cdavem@davemloft.net\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "ace178fd385038523bc498222e9497245ccae4b0",
      "old_mode": 33188,
      "old_path": "net/tipc/node.c",
      "new_id": "9aaa1bc566ae3ec59d03a1ef3c7872ab86a71c0d",
      "new_mode": 33188,
      "new_path": "net/tipc/node.c"
    }
  ]
}