rxrpc: Drop channel number field from rxrpc_call struct
authorDavid Howells <dhowells@redhat.com>
Tue, 23 Aug 2016 14:27:24 +0000 (15:27 +0100)
committerDavid Howells <dhowells@redhat.com>
Tue, 23 Aug 2016 14:27:24 +0000 (15:27 +0100)
Drop the channel number (channel) field from the rxrpc_call struct to
reduce the size of the call struct.  The field is redundant: if the call is
attached to a connection, the channel can be obtained from there by AND'ing
with RXRPC_CHANNELMASK.

Signed-off-by: David Howells <dhowells@redhat.com>
net/rxrpc/ar-internal.h
net/rxrpc/call_object.c
net/rxrpc/conn_client.c
net/rxrpc/conn_object.c
net/rxrpc/rxkad.c

index 0e6bc8227d54f17a60c0f54498ab6155fd9629f8..648060a5df35a18a9458bce0baa5b77939dd8f2f 100644 (file)
@@ -432,7 +432,6 @@ struct rxrpc_call {
        int                     error_report;   /* Network error (ICMP/local transport) */
        int                     error;          /* Local error incurred */
        enum rxrpc_call_state   state : 8;      /* current state of call */
-       u8                      channel;        /* connection channel occupied by this call */
        u16                     service_id;     /* service ID */
        u32                     call_id;        /* call ID on connection  */
        u32                     cid;            /* connection ID plus channel index */
index 008188103fd672375a91d2c8385a4c64027c18ac..4af01805bfc7126029c9074dfe2dc5beb6119ce4 100644 (file)
@@ -317,7 +317,6 @@ struct rxrpc_call *rxrpc_incoming_call(struct rxrpc_sock *rx,
        candidate->conn         = conn;
        candidate->cid          = sp->hdr.cid;
        candidate->call_id      = sp->hdr.callNumber;
-       candidate->channel      = chan;
        candidate->rx_data_post = 0;
        candidate->state        = RXRPC_CALL_SERVER_ACCEPTING;
        candidate->flags        |= (1 << RXRPC_CALL_IS_SERVICE);
@@ -330,7 +329,7 @@ struct rxrpc_call *rxrpc_incoming_call(struct rxrpc_sock *rx,
        call = rcu_dereference_protected(conn->channels[chan].call,
                                         lockdep_is_held(&conn->channel_lock));
 
-       _debug("channel[%u] is %p", candidate->channel, call);
+       _debug("channel[%u] is %p", candidate->cid & RXRPC_CHANNELMASK, call);
        if (call && call->call_id == sp->hdr.callNumber) {
                /* already set; must've been a duplicate packet */
                _debug("extant call [%d]", call->state);
@@ -677,8 +676,8 @@ static void rxrpc_destroy_call(struct work_struct *work)
        struct rxrpc_call *call =
                container_of(work, struct rxrpc_call, destroyer);
 
-       _enter("%p{%d,%d,%p}",
-              call, atomic_read(&call->usage), call->channel, call->conn);
+       _enter("%p{%d,%x,%p}",
+              call, atomic_read(&call->usage), call->cid, call->conn);
 
        ASSERTCMP(call->state, ==, RXRPC_CALL_DEAD);
 
index d8dd8e6bb17265614e2a04baf18010ad4a4217fb..fc32cc67c2dee645454c1c637d101d2a1e626d40 100644 (file)
@@ -280,7 +280,6 @@ attached:
 found_channel:
        _debug("found chan");
        call->conn      = conn;
-       call->channel   = chan;
        call->cid       = conn->proto.cid | chan;
        call->call_id   = ++conn->channels[chan].call_counter;
        conn->channels[chan].call_id = call->call_id;
index 896d84493a0574a425a49c943af4051ff8273c0e..6a5a17efc538aefad15687383fc110570d35cc8b 100644 (file)
@@ -156,9 +156,10 @@ not_found:
 void __rxrpc_disconnect_call(struct rxrpc_call *call)
 {
        struct rxrpc_connection *conn = call->conn;
-       struct rxrpc_channel *chan = &conn->channels[call->channel];
+       struct rxrpc_channel *chan =
+               &conn->channels[call->cid & RXRPC_CHANNELMASK];
 
-       _enter("%d,%d", conn->debug_id, call->channel);
+       _enter("%d,%x", conn->debug_id, call->cid);
 
        if (rcu_access_pointer(chan->call) == call) {
                /* Save the result of the call so that we can repeat it if necessary
index 63afa9e9cc08b2db0cd0a13dfdf98ef71507d404..89f475febfd7fbdf121940d9d302acb5e1c94639 100644 (file)
@@ -275,7 +275,7 @@ static int rxkad_secure_packet(struct rxrpc_call *call,
        memcpy(&iv, call->conn->csum_iv.x, sizeof(iv));
 
        /* calculate the security checksum */
-       x = call->channel << (32 - RXRPC_CIDSHIFT);
+       x = (call->cid & RXRPC_CHANNELMASK) << (32 - RXRPC_CIDSHIFT);
        x |= sp->hdr.seq & 0x3fffffff;
        call->crypto_buf[0] = htonl(sp->hdr.callNumber);
        call->crypto_buf[1] = htonl(x);
@@ -507,7 +507,7 @@ static int rxkad_verify_packet(struct rxrpc_call *call,
        memcpy(&iv, call->conn->csum_iv.x, sizeof(iv));
 
        /* validate the security checksum */
-       x = call->channel << (32 - RXRPC_CIDSHIFT);
+       x = (call->cid & RXRPC_CHANNELMASK) << (32 - RXRPC_CIDSHIFT);
        x |= sp->hdr.seq & 0x3fffffff;
        call->crypto_buf[0] = htonl(call->call_id);
        call->crypto_buf[1] = htonl(x);