c - FD_SET, FD_CLR ... 是原子操作吗？

Question

我目前正在处理的应用程序是一个服务器，它将使用 select() 管理与客户端的连接，每次服务器收到消息时，它都会打开一个新线程以读取套接字。在此期间，套接字的文件描述符从集合中删除，并将在读取结束时添加。这是代码示例

struct s_handle {
 int sock;
 fd_set * rdfs;
};




int main(){
 ...
 fd_set rdfs;
 ...
 while(1){
 ....
  select(nb_fd,&rdfs,NULL,NULL,NULL)
  for_each(peer){
   if(FD_ISSET(peer->sock,&rdfs)){
     struct s_handle * h = malloc(sizeof(struct s_handle));
     h->sock = peer->sock;
     h->rdfs = &rdfs;
     FD_CLR(peer->sock,&rdfs);
     pthread_create(thread,NULL,handle,(void *)&h);
   }
  }
 ...
 }
 ...
 }

void* handle(void* argss){
 struct s_handle * temp = (struct s_handle *) argss;
 ...
 FD_SET(temp->sock,temp->rdfs);
}

FD_SET、FD_ISSET 和 FD_CLR 是原子操作，还是我需要用互斥量锁定 rdfs？

如果需要互斥量，如何避免死锁？

Answer 1

首先，您不应该那样创建线程。创建线程是一项相当高的开销操作，应该只在您需要更多线程时使用，而不仅仅是因为您必须做更多的工作。

是的，您确实需要使用互斥锁来保护 FD_* 函数。通常的解决方案是拥有一个只在执行 FD_* 操作所需的瞬间持有的互斥锁。在调用 select 之前，您获取互斥量，复制描述符集，然后释放互斥量。

但一般来说，从读取集中删除套接字并不是一个好主意。将套接字放回读取集中不会更改稍后已经发生的 select。并且您将弄清楚如何从 select 中获取调用 select 的线程以便在新集合上运行。

您可能需要重新考虑您的 I/O 发现方法并使用其中一种标准方法，而不是尝试自己动手。您被迫做出丑陋的权衡，要么因为最近读取了一些套接字并且 select 仍然被阻止，或者不得不重新选择 select 作为您完成了每个套接字的读取。这两种解决方案都不好。

一种更常见的模式是在读取套接字时将套接字保留在集合中，直到所有套接字都已被读取(但不一定已处理其数据)后才返回 select ).