# \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- List: linuxppc-dev
- 7 messages, 4 authors
- Page: https://ratatoskr.run/linuxppc-dev/2008/09/1156003/t

## \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Mathieu Desnoyers \<hidden\>
- Date: 2008-09-29 15:11:57 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156003.md

```text
We need a marker_synchronize_unregister() before the end of exit() to make sure
every probe callers have exited the non preemptible section and thus are not
executing the probe code anymore.

Signed-off-by: Mathieu Desnoyers <redacted>
CC: Ingo Molnar <redacted>
CC: Jeremy Kerr <jk@ozlabs.org>
CC: <redacted>
CC: Christoph Hellwig <hch@infradead.org>
---
 arch/powerpc/platforms/cell/spufs/sputrace.c |    1 +
 1 file changed, 1 insertion(+)

Index: linux-2.6-lttng/arch/powerpc/platforms/cell/spufs/sputrace.c
===================================================================
--- linux-2.6-lttng.orig/arch/powerpc/platforms/cell/spufs/sputrace.c	2008-07-31 09:34:58.000000000 -0400
+++ linux-2.6-lttng/arch/powerpc/platforms/cell/spufs/sputrace.c	2008-07-31 09:35:15.000000000 -0400
@@ -233,6 +233,7 @@ static void __exit sputrace_exit(void)
 
 	remove_proc_entry("sputrace", NULL);
 	kfree(sputrace_log);
+	marker_synchronize_unregister();
 }
 
 module_init(sputrace_init);
-- 
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68
```

## Re: \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Christoph Hellwig \<hch@infradead.org\>
- Date: 2008-09-29 15:14:18 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156004.md

```text
On Mon, Sep 29, 2008 at 11:11:47AM -0400, Mathieu Desnoyers wrote:
> We need a marker_synchronize_unregister() before the end of exit() to make sure
> every probe callers have exited the non preemptible section and thus are not
> executing the probe code anymore.

Looks good.
```

## Re: \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Jeremy Kerr \<jk@ozlabs.org\>
- Date: 2008-09-30 00:28:42 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156200.md

```text
Mathieu,

> We need a marker_synchronize_unregister() before the end of exit() to
> make sure every probe callers have exited the non preemptible section
> and thus are not executing the probe code anymore.

Looks good - added to spufs.git.

Cheers,


Jeremy
```

## Re: \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Ingo Molnar \<hidden\>
- Date: 2008-09-30 09:56:45 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156342.md

```text

* Jeremy Kerr [ref] wrote:

> Mathieu,
> 
> > We need a marker_synchronize_unregister() before the end of exit() to
> > make sure every probe callers have exited the non preemptible section
> > and thus are not executing the probe code anymore.
> 
> Looks good - added to spufs.git.

that wont work very well as the patch relies on the new 
marker_synchronize_unregister() facility.

	Ingo
```

## Re: \[Cbe-oss-dev\] \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Jeremy Kerr \<jk@ozlabs.org\>
- Date: 2008-09-30 11:26:49 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156369.md

```text
Ingo,

> that wont work very well as the patch relies on the new
> marker_synchronize_unregister() facility.

d'oh, right you are. Should I leave this in your hands to merge?

Cheers,


Jeremy
```

## Re: \[Cbe-oss-dev\] \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Ingo Molnar \<hidden\>
- Date: 2008-09-30 11:31:19 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156372.md

```text

* Jeremy Kerr [ref] wrote:

> Ingo,
> 
> > that wont work very well as the patch relies on the new
> > marker_synchronize_unregister() facility.
> 
> d'oh, right you are. Should I leave this in your hands to merge?

would be nice if you could give your Acked-by for the sputrace bits, 
then we can merge it. It's a oneliner so it shouldnt cause merging 
trouble in linux-next.

	Ingo
```

## Re: \[Cbe-oss-dev\] \[PATCH\] sputrace : use marker\_synchronize\_unregister()

- From: Jeremy Kerr \<jk@ozlabs.org\>
- Date: 2008-09-30 11:36:58 
- Message: https://ratatoskr.run/linuxppc-dev/2008/09/1156373.md

```text
Ingo,

> would be nice if you could give your Acked-by for the sputrace bits,
> then we can merge it. It's a oneliner so it shouldnt cause merging
> trouble in linux-next.

Sure!

Acked-by: Jeremy Kerr <jk@ozlabs.org>


Jeremy
```