On Wed, Dec 11, 2013 at 10:25 AM, Maxim Dounin <mdou...@mdounin.ru> wrote:
> Hello!
>
> On Wed, Dec 11, 2013 at 05:55:16AM -0500, Jeffrey Walton wrote:
>
>> I need to do some additional processing with OpenSSL in a custom
>> module. I noticed ngingx does not set any locks in ngx_ssl_init (from
>> ngx_event_openssl.c).
>>
>> A few questions:
>>
>> 1) Should lock installation be guarded on NGX_THREADS or something else?
>>
>> 2) Where is most appropirate to initialize the locks: ngx_init_threads
>> or ngx_ssl_init (or somewhere else)?
>>
>> 3) Is the project interested in a patch? (Per
>> http://nginx.org/en/docs/contributing_changes.html, thanks Maxim).
>
> There are basically no threads support in nginx (it was
> experimental and broken for a long time with the exception of some
> win32-related stuff), so it's not clear why you need locks at all.
Thanks Maxim. The source code is full of:

    #if (NGX_THREADS)
      ...
    #endif

I thought they were needed. My bad.

Jeff

_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx

Reply via email to