mirror of
				https://gitlab.freedesktop.org/pipewire/pipewire.git
				synced 2025-11-03 09:01:54 -05:00 
			
		
		
		
	Revert "spa: support: loop: do not call control hooks on blocking invoke"
This reverts commit 9ae89b4247.
All invokes should be paired with a lock/unlock if the loop requires
this. For internal calls of invoke, this will also be true because all
pipewire functions should be called with the lock.
Fixes #4215
			
			
This commit is contained in:
		
							parent
							
								
									8edb6fc8b2
								
							
						
					
					
						commit
						fff52bb7a2
					
				
					 1 changed files with 4 additions and 0 deletions
				
			
		| 
						 | 
					@ -365,10 +365,14 @@ retry:
 | 
				
			||||||
		if (block && queue->ack_fd != -1) {
 | 
							if (block && queue->ack_fd != -1) {
 | 
				
			||||||
			uint64_t count = 1;
 | 
								uint64_t count = 1;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								spa_loop_control_hook_before(&impl->hooks_list);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
			if ((res = spa_system_eventfd_read(impl->system, queue->ack_fd, &count)) < 0)
 | 
								if ((res = spa_system_eventfd_read(impl->system, queue->ack_fd, &count)) < 0)
 | 
				
			||||||
				spa_log_warn(impl->log, "%p: failed to read event fd:%d: %s",
 | 
									spa_log_warn(impl->log, "%p: failed to read event fd:%d: %s",
 | 
				
			||||||
						queue, queue->ack_fd, spa_strerror(res));
 | 
											queue, queue->ack_fd, spa_strerror(res));
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								spa_loop_control_hook_after(&impl->hooks_list);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
			res = item->res;
 | 
								res = item->res;
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
		else {
 | 
							else {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue