Re: A little help with cbdata and requestLink

2009-08-06 Thread Mark Nottingham
If I parse that correctly, I think there are no such cases; i.e., all of the paths where fwdContinue doesn't go to peerSelect don't go forward, they just error out. E.g., if I read you correctly, it's not necessary to unlink the request here: case PROTO_INTERNAL:

Re: A little help with cbdata and requestLink

2009-06-29 Thread Mark Nottingham
OK. WRT requestlink, I was unsure what would happen if the request didn't go forward; will moving requestLink at the bottom up to here: fwdState-request = r; /* requestLink? */ do it? Thanks, On 27/06/2009, at 3:05 AM, Alex Rousskov wrote: On 06/16/2009 09:57 PM, Mark Nottingham wrote:

Re: A little help with cbdata and requestLink

2009-06-29 Thread Henrik Nordstrom
fre 2009-06-26 klockan 11:05 -0600 skrev Alex Rousskov: On 06/16/2009 09:57 PM, Mark Nottingham wrote: Thanks. Anybody else have a second to look? Please s/fwdStartFoo/fwdContinue/ and document what it is. Since this is Squid2 you do not have to do it, of course. Your cbdata and request

Re: A little help with cbdata and requestLink

2009-06-29 Thread Alex Rousskov
On 06/29/2009 12:30 AM, Mark Nottingham wrote: OK. WRT requestlink, I was unsure what would happen if the request didn't go forward; will moving requestLink at the bottom up to here: fwdState-request = r; /* requestLink? */ do it? Yes, provided you adjust the code to call unlink if the

Re: A little help with cbdata and requestLink

2009-06-26 Thread Alex Rousskov
On 06/16/2009 09:57 PM, Mark Nottingham wrote: Thanks. Anybody else have a second to look? Please s/fwdStartFoo/fwdContinue/ and document what it is. Since this is Squid2 you do not have to do it, of course. Your cbdata and request manipulations appear technically correct to me. IMHO, the

Re: A little help with cbdata and requestLink

2009-06-16 Thread Mark Nottingham
Thanks. Anybody else have a second to look? On 11/06/2009, at 11:28 PM, Amos Jeffries wrote: Mark Nottingham wrote: Would someone mind taking a quick look at this patch: http://www.squid-cache.org/bugs/attachment.cgi?id=1989 and telling me if I've royally stuffed up with managing fwdState

Re: A little help with cbdata and requestLink

2009-06-11 Thread Amos Jeffries
Mark Nottingham wrote: Would someone mind taking a quick look at this patch: http://www.squid-cache.org/bugs/attachment.cgi?id=1989 and telling me if I've royally stuffed up with managing fwdState and request linking? It's to make miss_access a slow lookup... Looks okay to these

A little help with cbdata and requestLink

2009-06-10 Thread Mark Nottingham
Would someone mind taking a quick look at this patch: http://www.squid-cache.org/bugs/attachment.cgi?id=1989 and telling me if I've royally stuffed up with managing fwdState and request linking? It's to make miss_access a slow lookup... Thanks, -- Mark Nottingham