From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756681AbdKDMQL (ORCPT ); Sat, 4 Nov 2017 08:16:11 -0400 Received: from mail.kernel.org ([198.145.29.99]:40008 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752362AbdKDMQJ (ORCPT ); Sat, 4 Nov 2017 08:16:09 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 61158218C9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=mhiramat@kernel.org Date: Sat, 4 Nov 2017 21:16:05 +0900 From: Masami Hiramatsu To: Ingo Molnar Cc: Stephen Rothwell , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Peter Zijlstra , Linux-Next Mailing List , Linux Kernel Mailing List Subject: Re: linux-next: build warnings after merge of the tip tree Message-Id: <20171104211605.ce02abd1ea46c248105617bf@kernel.org> In-Reply-To: <20171104080134.7i6vv2avlxqdvvz3@gmail.com> References: <20171102135351.24ee83f9@canb.auug.org.au> <20171104060034.4aae72a524c428c9fcb98152@kernel.org> <20171104080134.7i6vv2avlxqdvvz3@gmail.com> X-Mailer: Sylpheed 3.5.1 (GTK+ 2.24.31; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 4 Nov 2017 09:01:34 +0100 Ingo Molnar wrote: > > * Masami Hiramatsu wrote: > > > > net/sctp/probe.c: In function 'sctpprobe_exit': > > > net/sctp/probe.c:240:2: warning: 'unregister_jprobe' is deprecated [-Wdeprecated-declarations] > > > unregister_jprobe(&sctp_recv_probe); > > > ^ > > > In file included from net/sctp/probe.c:28:0: > > > include/linux/kprobes.h:479:33: note: declared here > > > static inline void __deprecated unregister_jprobe(struct jprobe *p) > > > ^ > > > > > > Introduced by commit > > > > > > 590c84593045 ("kprobes: Disable the jprobes APIs") > > > > > > These days we normally don't deprecate things, just remove them. But we > > > do that *after* fixing up all the usages in the tree, please. > > > > OK, should I remove __deprecated or revert above patch? > > I pinged such users but no response. I can just rewrite it but not sure they can reply. > > Ideal would be to just fix all these places: convert code where the facility > appears to be actively used, remove code where it looks unused. If maintainers > don't reply, I can apply them to a separate branch in -tip. Thanks, that will help me. > > For example I'm pretty sure we can just remove the jprobes usage in SCTP. Actually TCP and DCCP jprobes usages are similar to that SCTP usage (maybe derived from TCP one). For those usages, we can not replace it with kprobe/ftrace because it depends on the arguments of target funcs. For such use-cases, we have 3 options; - Remove entirely feature if possible (like no more used). - Replace it with trace-events, and handle the event from kernel as sched tracer does. - Just introduce trace-events, remove usage, and trace it via ftrace or perf. At a glance, all network probes are just used for printing out the event, so we can just introduce trace-events and remove usage. I will try it. Thank you, -- Masami Hiramatsu