Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-20 Thread Peter Payne
On 10/19/10 18:40, Alex Rousskov wrote: On 10/13/2010 04:21 AM, Peter Payne wrote: BENCHMARKS Firstly, I wanted to address the benchmark questions as it made me curious as to whether there really was an advantage in using /dev/poll. I used the Apache Bench tool (that comes with HTTPD) to do

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-19 Thread Alex Rousskov
On 10/13/2010 04:21 AM, Peter Payne wrote: BENCHMARKS Firstly, I wanted to address the benchmark questions as it made me curious as to whether there really was an advantage in using /dev/poll. I used the Apache Bench tool (that comes with HTTPD) to do my benchmarks. I compiled a 32-bit

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-16 Thread Amos Jeffries
On 14/10/10 13:58, Henrik Nordström wrote: ons 2010-10-13 klockan 22:25 + skrev Amos Jeffries: +1 from me with merge tweaks. Unless anyone has objections I will commit with tweaks at the next opportunity. No objection from me. But have not reviewed the changes outside comm_devpoll.cc.

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-13 Thread Peter Payne
Hello Amos, in response to your comments. BENCHMARKS Firstly, I wanted to address the benchmark questions as it made me curious as to whether there really was an advantage in using /dev/poll. I used the Apache Bench tool (that comes with HTTPD) to do my benchmarks. I compiled a 32-bit

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-13 Thread Peter Payne
Hello Amos, apologies to the dev list for what must appear to be spamming. Please use the attached version of the file. It has a minor performance tweak (only set events that have changed if not clearing any events). Compiled using CC=/tool/sunstudio12/bin/cc and tested again using Apache

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-13 Thread Henrik Nordström
ons 2010-10-13 klockan 15:40 +0100 skrev Peter Payne: Hello Amos, apologies to the dev list for what must appear to be spamming. No apologies needed. We are all for release early an often, and discussing code is what this list is for. Regards Henrik

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-13 Thread Amos Jeffries
On Wed, 13 Oct 2010 15:40:22 +0100, Peter Payne sourcefo...@pirosa.co.uk wrote: Hello Amos, apologies to the dev list for what must appear to be spamming. Completely on topic. No apologies needed. Please use the attached version of the file. It has a minor performance tweak (only set

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-13 Thread Henrik Nordström
ons 2010-10-13 klockan 22:25 + skrev Amos Jeffries: +1 from me with merge tweaks. Unless anyone has objections I will commit with tweaks at the next opportunity. No objection from me. But have not reviewed the changes outside comm_devpoll.cc. Regards Henrik

[PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-11 Thread Peter Payne
Hello Amos, thank you for your response. Please find attached the four files in question. Companies to mention as sponsors are BBC (UK), Siemens IT Solutions and Services (UK). Credit to Peter Payne, Pirosa Limited UK (no e-mail please). Squid bug 3057 was raised by another member of

Re: [PATCH] Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-11 Thread Amos Jeffries
On Mon, 11 Oct 2010 14:58:34 +0100, Peter Payne sourcefo...@pirosa.co.uk wrote: Hello Amos, thank you for your response. Please find attached the four files in question. Companies to mention as sponsors are BBC (UK), Siemens IT Solutions and Services (UK). Credit to Peter Payne,

Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-08 Thread Peter Payne
Dear Mailing List, I have a contribution to make, a C++ file (and autoconf patches) for /dev/poll Solaris support, ported from Squid-2 and in use on the BBC (UK) estate managed by Siemens UK with permission from the BBC to publish to the Squid open-source project. I've not contributed to

Re: Solaris /dev/poll support for Squid 3 (how can I contribute)

2010-10-08 Thread Amos Jeffries
On 08/10/10 23:13, Peter Payne wrote: Dear Mailing List, I have a contribution to make, a C++ file (and autoconf patches) for /dev/poll Solaris support, ported from Squid-2 and in use on the BBC (UK) estate managed by Siemens UK with permission from the BBC to publish to the Squid open-source