Messages in this thread | | | From | "Sosnowski, Maciej" <> | Date | Fri, 12 Dec 2008 14:29:47 +0000 | Subject | RE: [PATCH 11/13] dmaengine: kill struct dma_client and supporting infrastructure |
| |
Williams, Dan J wrote: > All users have been converted to either the general-purpose allocator, > dma_find_channel, or dma_request_channel. > > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > --- (...) > /** > - * dma_chans_notify_available - broadcast available channels to the > clients > - */ > -static void dma_clients_notify_available(void) > -{ > - struct dma_client *client; > - > - mutex_lock(&dma_list_mutex); > - > - list_for_each_entry(client, &dma_client_list, global_node) > - dma_client_chan_alloc(client); > - > - mutex_unlock(&dma_list_mutex); > -}
I agree with Guennadi's concern about removing clients' notification of new devices available in the system. I understand that this design is based on polling instead, however polling is always less efficient approach. Do you think that restoring notifications in this redesigned dmaengine would be more painful than limiting clients to polling solution?
Regards, Maciej
| |