]> pilppa.com Git - linux-2.6-omap-h63xx.git/commit
ieee1394: raw1394: make write() thread-safe
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Fri, 15 Aug 2008 22:15:16 +0000 (00:15 +0200)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Wed, 15 Oct 2008 20:21:08 +0000 (22:21 +0200)
commitf22e52b89e036fd12b9374212da8b5d4a447bd1e
tree84e8cb2fca80197dc1b2afee85d2c46fd8446af0
parentddfb908d3f905dbb5964d6fbf783e69c417eb13e
ieee1394: raw1394: make write() thread-safe

Application programs should use a libraw1394 handle only in a single
thread.  The raw1394 driver was apparently relying on this, because it
did nothing to protect its fi->state variable from corruption due to
concurrent accesses.

We now serialize the fi->state accesses.  This affects the write() path.
We re-use the state_mutex which was introduced to protect fi->iso_state
accesses in the ioctl() path.  These paths and accesses are independent
of each other, hence separate mutexes could be used.  But I don't see
much benefit in that.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/ieee1394/raw1394.c