Prev: net/dccp: Use memdup_user
Next: [PATCH 1/2] input: mt: Add method to extract the MT slot state
From: Millton Miller on 25 May 2010 07:50
On Tue, 25 May 2010 at 14:43:56 +0530, K.Prasad wrote:
> Certain architectures (such as PowerPC Book III S) have a need to cleanup
> data-structures before the breakpoint is unregistered. This patch introduces
> an arch-specific hook in release_bp_slot() along with a weak definition in
> the form of a stub funciton.
>
> Signed-off-by: K.Prasad <prasad(a)linux.vnet.ibm.com>
> ---
> kernel/hw_breakpoint.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)


My understanding is weak function definitions must appear in a different C
file than their call sites to work on some toolchains.

Andrew, can you confirm the above statement?

> Index: linux-2.6.ppc64_test/kernel/hw_breakpoint.c
> ===================================================================
> --- linux-2.6.ppc64_test.orig/kernel/hw_breakpoint.c
> +++ linux-2.6.ppc64_test/kernel/hw_breakpoint.c
> @@ -242,6 +242,17 @@ toggle_bp_slot(struct perf_event *bp, bo
> }
>
> /*
> + * Function to perform processor-specific cleanup during unregistration
> + */
> +__weak void arch_unregister_hw_breakpoint(struct perf_event *bp)
> +{
> + /*
> + * A weak stub function here for those archs that don't define
> + * it inside arch/.../kernel/hw_breakpoint.c
> + */
> +}
> +
> +/*
> * Contraints to check before allowing this new breakpoint counter:
> *
> * == Non-pinned counter == (Considered as pinned for now)
> @@ -339,6 +350,7 @@ void release_bp_slot(struct perf_event *
> {
> mutex_lock(&nr_bp_mutex);
>
> + arch_unregister_hw_breakpoint(bp);
> __release_bp_slot(bp);
>
> mutex_unlock(&nr_bp_mutex);
>


Since the weak version is empty, should it just be delcared (in
a header, put the comment there) and not defined?

milton
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: net/dccp: Use memdup_user
Next: [PATCH 1/2] input: mt: Add method to extract the MT slot state