Skip to content

Commit

Permalink
net: qrtr: Inject BYE on remote termination
Browse files Browse the repository at this point in the history
Per the QMUX protocol specification a terminating node can send a BYE
control message to signal that the link is going down, upon receiving
this all information about remote services should be discarded and local
clients should be notified.

In the event that the link was brought down abruptly the router is
supposed to act like a BYE message has arrived. As there is no harm in
receiving an extra BYE from the remote this patch implements the latter
by injecting a BYE when the link to the remote is unregistered.

The name service will receive the BYE and can implement the notification
to the local clients.

Cc: Courtney Cavin <[email protected]>
Signed-off-by: Bjorn Andersson <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
andersson authored and davem330 committed Jun 8, 2017
1 parent 64f9eca commit 8acc8ee
Showing 1 changed file with 27 additions and 0 deletions.
27 changes: 27 additions & 0 deletions net/qrtr/qrtr.c
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,8 @@ struct qrtr_node {
struct list_head item;
};

static int qrtr_local_enqueue(struct qrtr_node *node, struct sk_buff *skb);

/* Release node resources and free the node.
*
* Do not call directly, use qrtr_node_release. To be used with
Expand Down Expand Up @@ -291,6 +293,25 @@ static struct sk_buff *qrtr_alloc_resume_tx(u32 src_node,
return skb;
}

/* Allocate and construct a BYE message to signal remote termination */
static struct sk_buff *qrtr_alloc_local_bye(u32 src_node)
{
const int pkt_len = 20;
struct sk_buff *skb;
__le32 *buf;

skb = qrtr_alloc_ctrl_packet(QRTR_TYPE_BYE, pkt_len,
src_node, qrtr_local_nid);
if (!skb)
return NULL;

buf = (__le32 *)skb_put(skb, pkt_len);
memset(buf, 0, pkt_len);
buf[0] = cpu_to_le32(QRTR_TYPE_BYE);

return skb;
}

static struct qrtr_sock *qrtr_port_lookup(int port);
static void qrtr_port_put(struct qrtr_sock *ipc);

Expand Down Expand Up @@ -382,11 +403,17 @@ EXPORT_SYMBOL_GPL(qrtr_endpoint_register);
void qrtr_endpoint_unregister(struct qrtr_endpoint *ep)
{
struct qrtr_node *node = ep->node;
struct sk_buff *skb;

mutex_lock(&node->ep_lock);
node->ep = NULL;
mutex_unlock(&node->ep_lock);

/* Notify the local controller about the event */
skb = qrtr_alloc_local_bye(node->nid);
if (skb)
qrtr_local_enqueue(NULL, skb);

qrtr_node_release(node);
ep->node = NULL;
}
Expand Down

0 comments on commit 8acc8ee

Please sign in to comment.