| To: | Victor Kamensky <kamensky@cisco.com> |
|---|---|
| Subject: | Re: [PATCH] MIPS Kprobes: Support branch instructions probing |
| From: | Ralf Baechle <ralf@linux-mips.org> |
| Date: | Thu, 13 Oct 2011 23:59:18 +0100 |
| Cc: | David Daney <david.daney@cavium.com>, manesoni@cisco.com, linux-kernel@vger.kernel.org, linux-mips@linux-mips.org, ananth@in.ibm.com |
| In-reply-to: | <Pine.GSO.4.58.1110131205410.7452@infra-view9.cisco.com> |
| References: | <20111013090749.GB16761@cisco.com> <4E971FD3.2020308@cavium.com> <20111013180714.GA7422@linux-mips.org> <Pine.GSO.4.58.1110131205410.7452@infra-view9.cisco.com> |
| Sender: | linux-mips-bounce@linux-mips.org |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Thu, Oct 13, 2011 at 12:16:27PM -0700, Victor Kamensky wrote: > Yes, it does make a lot of sense. Don't you think we need to do > EXPORT_SYMBOL for __compute_return_epc as well? So it could be used by > klms. > > Actually we have yet another copy of it in mips uprobes code, which > normally is built as klm, if we refactor and export __compute_return_epc > all three places could use the same function. Nothing wrong with exporting __compute_return_epc() as long as there are actually users of the exported symbol. So far it's only used from the kernel proper which is why it's not been exported. However I'd prefer it to be exported as EXPORT_SYMBOL_GPL(). Ralf |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] MIPS Kprobes: Support branch instructions probing, Victor Kamensky |
|---|---|
| Next by Date: | [PATCH 14/21] mips: drop unused Kconfig symbols, Paul Bolle |
| Previous by Thread: | Re: [PATCH] MIPS Kprobes: Support branch instructions probing, Victor Kamensky |
| Next by Thread: | Re: [PATCH] MIPS Kprobes: Support branch instructions probing, Maneesh Soni |
| Indexes: | [Date] [Thread] [Top] [All Lists] |