Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 31 Jan 2012 11:00:32 GMT
From:      igor <igor@soumenkov.com>
To:        freebsd-bugs@FreeBSD.org
Subject:   Re: kern/156567: [kqueue] [patch] Add EV_CLEAR to AIO events in kqueue
Message-ID:  <201201311100.q0VB0WGl066042@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/156567; it has been noted by GNATS.

From: igor <igor@soumenkov.com>
To: <davidxu@FreeBSD.org>
Cc: <bug-followup@FreeBSD.org>, <freebsd-bugs@FreeBSD.org>
Subject: Re: kern/156567: [kqueue] [patch] Add =?UTF-8?Q?EV=5FCLEAR=20to?=
 =?UTF-8?Q?=20AIO=20events=20in=20kqueue?=
Date: Tue, 31 Jan 2012 14:52:10 +0400

 --=_3349fc38fa29cf5858e92f7cc8080a49
 Content-Transfer-Encoding: 7bit
 Content-Type: text/plain; charset=UTF-8
 
   
 
 Hello David, 
 
 The only thing I changed is the value of the flag,
 this is the only meaning of the patch. I do not know who decided to
 place the value there. 
 
 On Tue, 31 Jan 2012 16:04:06 +0800, David Xu
 wrote: 
 
 > On 2012/1/31 4:47, igor wrote:
 > 
 >> Dear all, What can I do
 to get my patch finally applied or rejected (hope not :) ? My PR is not
 even assigned to anyone. Should I do my best to become a committer (how?
 :-) ? The solution I propose finally allows to use kqueue with aio in
 multi-threaded applications. The only reason I created it is because I
 wanted to use it for myself and it worked flawlessly. Please, anyone?
 >
 
 > It is better to allocate a member field from struct sigevent for
 kevent
 > flags, we have spare fields available there, please check the
 union
 > _sigev_un.
 > 
 > Regards,
 > David Xu
 
 -- 
 Igor Soumenkov
   
 --=_3349fc38fa29cf5858e92f7cc8080a49
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/html; charset=UTF-8
 
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">
 <html><body>
 <p>Hello David,</p>
 <p>The only thing I changed is the value of the flag, this is the only mean=
 ing of the patch. I do not know who decided to place the value there.</p>
 <p>On Tue, 31 Jan 2012 16:04:06 +0800, David Xu wrote:</p>
 <blockquote type=3D"cite" style=3D"padding-left:5px; border-left:#1010ff 2p=
 x solid; margin-left:5px; width:100%"><!-- html ignored --><!-- head ignore=
 d --><!-- meta ignored -->
 <pre>On 2012/1/31 4:47, igor wrote:</pre>
 <blockquote type=3D"cite" style=3D"padding-left:5px; border-left:#1010ff 2p=
 x solid; margin-left:5px; width:100%">Dear all, What can I do to get my pat=
 ch finally applied or rejected (hope not :) ? My PR is not even assigned to=
  anyone. Should I do my best to become a committer (how? :-) ? The solution=
  I propose finally allows to use kqueue with aio in multi-threaded applicat=
 ions. The only reason I created it is because I wanted to use it for myself=
  and it worked flawlessly. Please, anyone?</blockquote>
 <pre>It is better to allocate a member field from struct sigevent for keven=
 t
 flags, we have spare fields available there, please check the union
 _sigev_un.
 
 Regards,
 David Xu
 
 </pre>
 </blockquote>
 <p>&nbsp;</p>
 <div>
 <pre>--=20
 Igor Soumenkov</pre>
 </div>
 </body></html>
 
 --=_3349fc38fa29cf5858e92f7cc8080a49--
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201201311100.q0VB0WGl066042>