summaryrefslogtreecommitdiffstats
path: root/drivers/macintosh
diff options
context:
space:
mode:
authorJohn Kacur <jkacur@redhat.com>2009-10-11 22:24:25 +0200
committerThomas Gleixner <tglx@linutronix.de>2009-10-14 17:36:54 +0200
commit55e858c8483af427144f33b42b818b30612b82b0 (patch)
tree300663182d53ca9ae77a54dce06066109e927a8f /drivers/macintosh
parentf96d3015e9f7f7fff4cab7ed1d467664cc980061 (diff)
downloadop-kernel-dev-55e858c8483af427144f33b42b818b30612b82b0.zip
op-kernel-dev-55e858c8483af427144f33b42b818b30612b82b0.tar.gz
agp: Remove the BKL from agp_open
- Remove the BKL from agp_open - Perform a few clean-ups. Analysis: --------- int minor is local to the function. The following are protected by agp_fe.agp_mutex struct agp_file_private *priv; struct agp_client *client; Call-outs: kzalloc should be safe to call under the mutex_lock agp_find_client_by_pid: - agp_mmap calls that under agp_fe.agp_mutex which we hold in agp_open - agpioc_reserve_wrap calls it without any locking what-so-ever. - Is that an error? Or is that okay because it has pid that is a unique handle? agp_insert_file_private: - This function only manipulates struct agp_file_private, once again while agp_fe.agp_mutex is held Signed-off-by: John Kacur <jkacur@redhat.com> Acked-by: David Airlie <airlied@linux.ie> LKML-Reference: <alpine.LFD.2.00.0910112216060.12574@localhost.localdomain> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'drivers/macintosh')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud