From patchwork Mon Nov 25 23:09:37 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Maciej_=C5=BBenczykowski?= X-Patchwork-Id: 1200601 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="h6fbGlyI"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 47MN7Q1wrHz9sR5 for ; Tue, 26 Nov 2019 10:10:58 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726033AbfKYXJs (ORCPT ); Mon, 25 Nov 2019 18:09:48 -0500 Received: from mail-pg1-f194.google.com ([209.85.215.194]:41242 "EHLO mail-pg1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725940AbfKYXJr (ORCPT ); Mon, 25 Nov 2019 18:09:47 -0500 Received: by mail-pg1-f194.google.com with SMTP id 207so7946654pge.8 for ; Mon, 25 Nov 2019 15:09:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=G0cPyC006hnvLt+vhRMpzYR5TfT6GaLflCtS07u8efA=; b=h6fbGlyIkP8nO/BoM5xdZ9hsIq3bUfYcrqcGtgRCzgigiEmhtmP2AuKqkbX8jbrNaq VcG2d7eIi5d8D9tQzdh6/chck/bjuNw0NCZsGQ56fwQoXqxYaMmpmO1bmwdvrKoaJovR 7eJs/4g5BLZpDomF/1mSHH8zJ3PtE7e0UCSEyNlKnYCVtFgRncHIJa/Ckb3ZK1vMPE8F COCMp/SlQIhUOeKzmrWV33Tpd0MLBIVMybW0/dyydmz2E9u1AMsKLG9AlDdYT9xzj4di K/AIf/6+9daStqahJfUtEct7XJKQqeQQjtvUYLhNcwCepLC5a35Sbi9iNxVI6m9evqY/ pMGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=G0cPyC006hnvLt+vhRMpzYR5TfT6GaLflCtS07u8efA=; b=h/mLEIi8dhKRlapGlgsezrS8DUfzVjUtNY46ZMrfvFj3L5l5qzvEmZXVngvFnY2nlF StQLUr0ychm8iU7mbSsiedO0or1IOgKyLs6NM4MvGd7FczIWWqDWfZYELs/T5n00jE4V AjB/kXCZlQ3aq+nh0LLmz4lXPGAuRvXWNx8qbMGQe5Gd9KK78sEJkbsEQ40THm+LcT+y iPYm08bnj/AwkgwCGBzwLp1O27SIrgR8bL/hmjRLTMJMlRo6tM0tECpucLx7EB9NmPcj 0COezIZEFVs9sFdU4Qg4Sh1Q/H292p0L+DthMaoItMFEN7fJREuPkdjep5kLDwY40Qml NWMw== X-Gm-Message-State: APjAAAUsEZ6FV++BKmHToGuhqT3SOHFbuTEpjShN96ytAr9YIu1NUqJ9 KxWlc/eFZUDKcsw199OKz9AhRZUi X-Google-Smtp-Source: APXvYqyh1seExqhD9fcWMcf0P6sodLKgpMnxHByFEz4yG55PRmq0a2r2iLtaNgw1vD3FBkd8lqENXA== X-Received: by 2002:a65:6906:: with SMTP id s6mr18719779pgq.26.1574723386786; Mon, 25 Nov 2019 15:09:46 -0800 (PST) Received: from athina.mtv.corp.google.com ([2620:15c:211:0:c786:d9fd:ab91:6283]) by smtp.gmail.com with ESMTPSA id u20sm9716324pgo.50.2019.11.25.15.09.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 25 Nov 2019 15:09:45 -0800 (PST) From: =?utf-8?q?Maciej_=C5=BBenczykowski?= To: =?utf-8?q?Maciej_=C5=BBenczykowski?= , "David S . Miller" Cc: netdev@vger.kernel.org Subject: [PATCH v2] net-sctp: replace some sock_net(sk) with just 'net' Date: Mon, 25 Nov 2019 15:09:37 -0800 Message-Id: <20191125230937.172098-1-zenczykowski@gmail.com> X-Mailer: git-send-email 2.24.0.432.g9d3f5f5b63-goog In-Reply-To: <20191125.105022.2027962925589066709.davem@davemloft.net> References: <20191125.105022.2027962925589066709.davem@davemloft.net> MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Maciej Żenczykowski It already existed in part of the function, but move it to a higher level and use it consistently throughout. Safe since sk is never written to. Signed-off-by: Maciej Żenczykowski Acked-by: Marcelo Ricardo Leitner --- net/sctp/socket.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/net/sctp/socket.c b/net/sctp/socket.c index 83e4ca1fabda..e4c398db07a0 100644 --- a/net/sctp/socket.c +++ b/net/sctp/socket.c @@ -8267,6 +8267,7 @@ static int sctp_get_port_local(struct sock *sk, union sctp_addr *addr) struct sctp_sock *sp = sctp_sk(sk); bool reuse = (sk->sk_reuse || sp->reuse); struct sctp_bind_hashbucket *head; /* hash list */ + struct net *net = sock_net(sk); kuid_t uid = sock_i_uid(sk); struct sctp_bind_bucket *pp; unsigned short snum; @@ -8282,7 +8283,6 @@ static int sctp_get_port_local(struct sock *sk, union sctp_addr *addr) /* Search for an available port. */ int low, high, remaining, index; unsigned int rover; - struct net *net = sock_net(sk); inet_get_local_port_range(net, &low, &high); remaining = (high - low) + 1; @@ -8294,12 +8294,12 @@ static int sctp_get_port_local(struct sock *sk, union sctp_addr *addr) rover = low; if (inet_is_local_reserved_port(net, rover)) continue; - index = sctp_phashfn(sock_net(sk), rover); + index = sctp_phashfn(net, rover); head = &sctp_port_hashtable[index]; spin_lock(&head->lock); sctp_for_each_hentry(pp, &head->chain) if ((pp->port == rover) && - net_eq(sock_net(sk), pp->net)) + net_eq(net, pp->net)) goto next; break; next: @@ -8323,10 +8323,10 @@ static int sctp_get_port_local(struct sock *sk, union sctp_addr *addr) * to the port number (snum) - we detect that with the * port iterator, pp being NULL. */ - head = &sctp_port_hashtable[sctp_phashfn(sock_net(sk), snum)]; + head = &sctp_port_hashtable[sctp_phashfn(net, snum)]; spin_lock(&head->lock); sctp_for_each_hentry(pp, &head->chain) { - if ((pp->port == snum) && net_eq(pp->net, sock_net(sk))) + if ((pp->port == snum) && net_eq(pp->net, net)) goto pp_found; } } @@ -8382,7 +8382,7 @@ static int sctp_get_port_local(struct sock *sk, union sctp_addr *addr) pp_not_found: /* If there was a hash table miss, create a new port. */ ret = 1; - if (!pp && !(pp = sctp_bucket_create(head, sock_net(sk), snum))) + if (!pp && !(pp = sctp_bucket_create(head, net, snum))) goto fail_unlock; /* In either case (hit or miss), make sure fastreuse is 1 only