From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [PATCH] uprobes: remove function declarations from arch/{mips,s390} References: <1472804384-17830-1-git-send-email-marcin.nowakowski@imgtec.com> <20160903105928.GB3917@osiris> From: Marcin Nowakowski Message-ID: Date: Thu, 8 Sep 2016 11:52:51 +0200 MIME-Version: 1.0 In-Reply-To: <20160903105928.GB3917@osiris> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-Path: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: List-Archive: List-Post: To: Andrew Morton Cc: Heiko Carstens , linux-kernel@vger.kernel.org, linux-mips@linux-mips.org, ralf@linux-mips.org, linux-s390@vger.kernel.org, schwidefsky@de.ibm.com List-ID: Hi Andrew, On 03.09.2016 12:59, Heiko Carstens wrote: >> Signed-off-by: Marcin Nowakowski >> --- >> arch/mips/include/asm/uprobes.h | 12 ------------ >> arch/s390/include/asm/uprobes.h | 10 ---------- >> 2 files changed, 22 deletions(-) > > You may either split this patch into two patches (mips/s390) so it can be > applied to the different architecture trees, or send it as single patch to > Andrew Morton, so he can pick it. > > In any case: > > Acked-by: Heiko Carstens Could you please pick up this patch and take it through your tree? It's trivial and shouldn't cause any conflicts for the arch trees, so I think it's simpler to keep it as one. thanks, Marcin >> >> diff --git a/arch/mips/include/asm/uprobes.h b/arch/mips/include/asm/uprobes.h >> index 34c325c..28ab364 100644 >> --- a/arch/mips/include/asm/uprobes.h >> +++ b/arch/mips/include/asm/uprobes.h >> @@ -43,16 +43,4 @@ struct arch_uprobe_task { >> unsigned long saved_trap_nr; >> }; >> >> -extern int arch_uprobe_analyze_insn(struct arch_uprobe *aup, >> - struct mm_struct *mm, unsigned long addr); >> -extern int arch_uprobe_pre_xol(struct arch_uprobe *aup, struct pt_regs *regs); >> -extern int arch_uprobe_post_xol(struct arch_uprobe *aup, struct pt_regs *regs); >> -extern bool arch_uprobe_xol_was_trapped(struct task_struct *tsk); >> -extern int arch_uprobe_exception_notify(struct notifier_block *self, >> - unsigned long val, void *data); >> -extern void arch_uprobe_abort_xol(struct arch_uprobe *aup, >> - struct pt_regs *regs); >> -extern unsigned long arch_uretprobe_hijack_return_addr( >> - unsigned long trampoline_vaddr, struct pt_regs *regs); >> - >> #endif /* __ASM_UPROBES_H */ >> diff --git a/arch/s390/include/asm/uprobes.h b/arch/s390/include/asm/uprobes.h >> index 1411dff..658393c 100644 >> --- a/arch/s390/include/asm/uprobes.h >> +++ b/arch/s390/include/asm/uprobes.h >> @@ -29,14 +29,4 @@ struct arch_uprobe { >> struct arch_uprobe_task { >> }; >> >> -int arch_uprobe_analyze_insn(struct arch_uprobe *aup, struct mm_struct *mm, >> - unsigned long addr); >> -int arch_uprobe_pre_xol(struct arch_uprobe *aup, struct pt_regs *regs); >> -int arch_uprobe_post_xol(struct arch_uprobe *aup, struct pt_regs *regs); >> -bool arch_uprobe_xol_was_trapped(struct task_struct *tsk); >> -int arch_uprobe_exception_notify(struct notifier_block *self, unsigned long val, >> - void *data); >> -void arch_uprobe_abort_xol(struct arch_uprobe *ap, struct pt_regs *regs); >> -unsigned long arch_uretprobe_hijack_return_addr(unsigned long trampoline, >> - struct pt_regs *regs); >> #endif /* _ASM_UPROBES_H */ >> -- >> 2.7.4 >> >