linux-kernel - Re: Suggested addition to epoll interface

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.00.0909151143070.20414@makko.or.mcafeemobile.com>
Date:	Tue, 15 Sep 2009 11:48:09 -0700 (PDT)
From:	Davide Libenzi <davidel@...ilserver.org>
To:	adi hodos <thefatredguy@...il.com>
cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: Suggested addition to epoll interface -
 epoll_post_notification()

On Tue, 15 Sep 2009, adi hodos wrote:

> Hello everyone. Thank you for your time in reading this.
> I have been playing with epoll lately and I would like to make a suggestion
> regarding the epoll interface.
> It would be nice to have a function to wake up a thread/process that is blocked
> on an epoll_wait() call and sent it some
> user defined data in the form of a struct epoll_event parameter ( something
> similar with PostQueuedCompletionStatus() from Windows ).
> My suggestion is for a function like this : int epoll_post_notification( int
> epoll_descriptor , struct epoll_event* event );
> It could be of much help in scenarios like these :
> 
> int e_accept;
> int e_clients;
> 
> typedef enum {
>  keycode_client_connection ,
>  keycode_signal ,
>  keycode_socketio ,
>  keycode_fileio ,
>  keycode_event ,
>  keycode_timer
>  ....
> } event_key_type;
> 
> struct event_key {
>   event_key_type keycode;
> };
> 
> struct eventfd_wrapper {
>  struct eventkey ek;
>  int fd_event;
> };
> 
> struct signalfd_wrapper {
>  struct eventkey ek;
>  int fd_signal;
> };
> 
> struct client {
>  struct event_key ek;
>  int sock;
>  ....
> };
> 
> struct user_posted_event {
>  struct event_key ek;
>  union {
>   void*    ptr_data;
>   int      i_data;
>   uint32_t u32;
>   uint32_t u64;
>  } u;
> };
> 
> 
> // thread A - waits for client connections using an epoll interface
> for ( ; ; ) {
>  epoll_wait( e_accept , ... );
> 
>  int s = accept();
>  struct client* new_client = allocate_newclient();
>  new_client->sock = s;
>  struct user_posted_event* ue = allocate_newuserevent();
>  ue->ek.keycode = keycode_client_connection;
>  ue->u.ptr_data = new_client;
> 
>  struct epoll_event e;
>  e.data.ptr = ue;
>  epoll_post_notification( e_clients , &e );
> }
> 
> // thread b - serves clients
> for( ; ; ) {
>  struct epoll_event[..] e;
>  epoll_wait( e_clients , e , ... );
> 
>  struct event_key* key = ( struct event_key* )( e.data.ptr );
>  if( key->keycode == keycode_client_connection ) {
>    // handle new client connection
>  } else if( key->keycode == keycode_signal ) {
>    // handle signal
>  } else if( .... ) {
>   ...
>  }
> 
> Again , thank you for your time in reviewing this.

Just use a pipe(2) to send data from A to B.
The read-side of the pipe will be in thread B epoll set, thread A would 
prepare and write(2) a "message" (binary struct really), and thread B 
would be wake up and read(2) the message.
no need to add anything.
Do you really have a dedicated thread for accepts though?


- Davide


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/