diff options
| author | Dan Williams <dan.j.williams@intel.com> | 2009-01-19 16:39:10 -0500 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2009-01-19 16:39:10 -0500 |
| commit | 83436a0560e9ef8af2f0796264dde4bed1415359 (patch) | |
| tree | ec195673813725fbfc49c6e56689682b162f42d4 | |
| parent | 169d5f663759ec494aa74a552ce99486235e6e50 (diff) | |
dmaengine: kill some dubious WARN_ONCEs
dma_find_channel and dma_issue_pending_all are good places to warn about
improper api usage. However, warning correctly means synchronizing with
dma_list_mutex, i.e. too much overhead for these fast-path calls.
Reported-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
| -rw-r--r-- | drivers/dma/dmaengine.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/dma/dmaengine.c b/drivers/dma/dmaengine.c index 6df144a65fef..a58993011edb 100644 --- a/drivers/dma/dmaengine.c +++ b/drivers/dma/dmaengine.c | |||
| @@ -329,9 +329,6 @@ struct dma_chan *dma_find_channel(enum dma_transaction_type tx_type) | |||
| 329 | struct dma_chan *chan; | 329 | struct dma_chan *chan; |
| 330 | int cpu; | 330 | int cpu; |
| 331 | 331 | ||
| 332 | WARN_ONCE(dmaengine_ref_count == 0, | ||
| 333 | "client called %s without a reference", __func__); | ||
| 334 | |||
| 335 | cpu = get_cpu(); | 332 | cpu = get_cpu(); |
| 336 | chan = per_cpu_ptr(channel_table[tx_type], cpu)->chan; | 333 | chan = per_cpu_ptr(channel_table[tx_type], cpu)->chan; |
| 337 | put_cpu(); | 334 | put_cpu(); |
| @@ -348,9 +345,6 @@ void dma_issue_pending_all(void) | |||
| 348 | struct dma_device *device; | 345 | struct dma_device *device; |
| 349 | struct dma_chan *chan; | 346 | struct dma_chan *chan; |
| 350 | 347 | ||
| 351 | WARN_ONCE(dmaengine_ref_count == 0, | ||
| 352 | "client called %s without a reference", __func__); | ||
| 353 | |||
| 354 | rcu_read_lock(); | 348 | rcu_read_lock(); |
| 355 | list_for_each_entry_rcu(device, &dma_device_list, global_node) { | 349 | list_for_each_entry_rcu(device, &dma_device_list, global_node) { |
| 356 | if (dma_has_cap(DMA_PRIVATE, device->cap_mask)) | 350 | if (dma_has_cap(DMA_PRIVATE, device->cap_mask)) |
