Messages in this thread | | | Date | Tue, 17 Jun 2008 16:21:28 +0300 | From | Eduard - Gabriel Munteanu <> | Subject | Re: [PATCH 2/3] relay: Fix race condition which occurs when reading across CPUs. |
| |
On Tue, 17 Jun 2008 08:55:44 -0400 Mathieu Desnoyers <compudj@krystal.dyndns.org> wrote:
> How about not changing the code, not providing a safe version of > relay_write, but document its use and what kind of locking the > in-kernel user must provide ? > > Mathieu
I would have agreed with you if this was about some other kernel function. But having the user provide locking means more than just bracing writes with spinlocks. Essentially, one also needs to duplicate relay_file_read() and relay_file_read_subbufs(), thus having to know relay's internals.
When I do this, I'll split my changes into two parts, with the first just documenting as you said. Anyway, the non-affine versions are low priority for me right now.
BTW, did you recieve or took a look at the buffer-only channels patch series, the one with CPU hotplug support? Sorry for being impatient, but that early initcall is really core stuff and I'd like a review. :-)
Eduard
| |