[PATCH] make nginx not swappable

Piotr Sikora piotr at cloudflare.com
Wed May 21 21:52:43 UTC 2014


Hey,

> if there's something wrong with patch itself - I always can fix it.

1. m(un)lockall is POSIX, so you need to stop guarding it with
NGX_LINUX and add tests for it in configure,
2. master process doesn't set rlimit(RLIMIT_MEMLOCK),
3. there is no error handling (logging isn't error handling),
4. it could use a more user-friendly name (i.e. "worker_noswap"),
5. style doesn't match nginx, i.e. indent is 4 spaces, not tab, etc.

Best regards,
Piotr Sikora



More information about the nginx-devel mailing list