diff options
| author | Jack Morgenstein <jackm@dev.mellanox.co.il> | 2008-06-19 00:36:38 +0200 |
|---|---|---|
| committer | Roland Dreier <rolandd@cisco.com> | 2008-06-19 00:36:38 +0200 |
| commit | fb77bcef9f7be78e3e11543cb5abbcb1b1fac53e (patch) | |
| tree | f5e96f158e20f426d7c4a08c296ecb9cb3c95ece | |
| parent | RDMA/nes: Fix off-by-one in nes_reg_user_mr() error path (diff) | |
| download | linux-fb77bcef9f7be78e3e11543cb5abbcb1b1fac53e.tar.xz linux-fb77bcef9f7be78e3e11543cb5abbcb1b1fac53e.zip | |
IB/uverbs: Fix check of is_closed flag check in ib_uverbs_async_handler()
Commit 1ae5c187 ("IB/uverbs: Don't store struct file * for event
files") changed the way that closed files are handled in the uverbs
code. However, after the conversion, is_closed flag is checked
incorrectly in ib_uverbs_async_handler(). As a result, no async
events are ever passed to applications.
Found by: Ronni Zimmerman <ronniz@mellanox.co.il>
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
| -rw-r--r-- | drivers/infiniband/core/uverbs_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/core/uverbs_main.c b/drivers/infiniband/core/uverbs_main.c index f806da184b51..caed42bf7ef5 100644 --- a/drivers/infiniband/core/uverbs_main.c +++ b/drivers/infiniband/core/uverbs_main.c @@ -423,7 +423,7 @@ static void ib_uverbs_async_handler(struct ib_uverbs_file *file, unsigned long flags; spin_lock_irqsave(&file->async_file->lock, flags); - if (!file->async_file->is_closed) { + if (file->async_file->is_closed) { spin_unlock_irqrestore(&file->async_file->lock, flags); return; } |