Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753870AbdHWKUe (ORCPT ); Wed, 23 Aug 2017 06:20:34 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:58362 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753673AbdHWKUc (ORCPT ); Wed, 23 Aug 2017 06:20:32 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 3B6FB602B3 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=kvalo@codeaurora.org From: Kalle Valo To: Arvind Yadav Cc: davem@davemloft.net, Larry.Finger@lwfinger.net, chaoming_li@realsil.com.cn, jon.maloy@ericsson.com, ying.xue@windriver.com, linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH 3/3 v2] net: netlink: constify genl_ops References: <1503482923-21640-1-git-send-email-arvind.yadav.cs@gmail.com> Date: Wed, 23 Aug 2017 13:20:26 +0300 In-Reply-To: <1503482923-21640-1-git-send-email-arvind.yadav.cs@gmail.com> (Arvind Yadav's message of "Wed, 23 Aug 2017 15:38:43 +0530") Message-ID: <87y3qawpp1.fsf@kamboji.qca.qualcomm.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2028 Lines: 41 Arvind Yadav writes: > genl_ops are not supposed to change at runtime. All functions > working with genl_ops provided by work with > const genl_ops. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav > --- > Changes in v2: > Patch Subject was wrong. > > net/tipc/netlink_compat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) I don't think the title is still misleading. Other patches have used "tipc:", I think you should use the same: $ git log --oneline --no-merges -20 net/tipc 41c6d650f653 net: convert sock.sk_refcnt from atomic_t to refcount_t 343eba69c696 net: tipc: Fix a sleep-in-atomic bug in tipc_msg_reverse 844cf763fba6 tipc: make macro tipc_wait_for_cond() smp safe ec8a09fbbeff tipc: refactor function tipc_sk_recv_stream() e9f8b10101c6 tipc: refactor function tipc_sk_recvmsg() c1be7756284b tipc: close the connection if protocol messages contain errors 4e0df4951e9e tipc: improve error validations for sockets in CONNECTING state 42b531de17d2 tipc: Fix missing connection request handling 05ff8378975a tipc: fix socket flow control accounting error at tipc_recv_stream 3364d61c92ec tipc: fix socket flow control accounting error at tipc_send_stream 78302fd40576 tipc: check return value of nlmsg_new fe52145f91fe netlink: pass extended ACK struct where available fceb6435e852 netlink: pass extended ACK struct to parsing functions 66bc1e8d5d1d tipc: allow rdm/dgram socketpairs 70b03759e9ec tipc: add support for stream/seqpacket socketpairs 7efea60dcffc tipc: adjust the policy of holding subscription kref 139bb36f754a tipc: advance the time of deleting subscription from subscriber->subscrp_list 557d054c01da tipc: fix nametbl deadlock at tipc_nametbl_unsubscribe cdfbabfb2f0c net: Work around lockdep limitation in sockets that use sockets 174cd4b1e5fb sched/headers: Prepare to move signal wakeup & sigpending methods from into -- Kalle Valo