[ 
https://issues.apache.org/jira/browse/TS-158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12843708#action_12843708
 ] 

Leif Hedstrom commented on TS-158:
----------------------------------

On RHEL4, we now get a compile error:


if ccache g++ -DHAVE_CONFIG_H  -I. -I. -I../..  -I../../proxy 
-I../../librecords  -I../../libinktomi++  -I../../iocore/eventsystem  
-I../../iocore/net  -I../../iocore/aio  -I../../iocore/hostdb  
-I../../iocore/cache  -I../../iocore/cluster  -I../../iocore/utils  
-I../../iocore/dns  -I../../libev -I../../librecords -I../../proxy/http2 
-I../../proxy/http2/remap -I../../proxy/hdrs -I../../proxy/mgmt2 
-I../../proxy/mgmt2/preparse -I../../proxy/mgmt2/utils -I../../proxy/mgmt2/web2 
-DSYSCONFDIR=\"/opt/ats/etc\" -I/usr/include  -g -pipe -Wall -Werror 
-Wno-char-subscripts -Wno-write-strings -O3 -feliminate-unused-debug-symbols 
-fno-strict-aliasing -Wno-invalid-offsetof  -march=i586 -MT Cache.o -MD -MP -MF 
".deps/Cache.Tpo" -c -o Cache.o Cache.cc; \
then mv -f ".deps/Cache.Tpo" ".deps/Cache.Po"; else rm -f ".deps/Cache.Tpo"; 
exit 1; fi
In file included from P_Cache.h:41,
                 from Cache.cc:27:
P_CacheDir.h: In function `bool dir_compare_tag(Dir*, INK_MD5*)':
P_CacheDir.h:312: warning: comparison between signed and unsigned integer 
expressions
Cache.cc: In member function `int CacheVC::handleReadDone(int, Event*)':
Cache.cc:1798: warning: comparison between signed and unsigned integer 
expressions
Cache.cc: In member function `int CacheVC::handleRead(int, Event*)':
Cache.cc:1909: warning: comparison between signed and unsigned integer 
expressions
Cache.cc: In member function `int CacheVC::removeEvent(int, Event*)':
Cache.cc:2014: warning: comparison between signed and unsigned integer 
expressions
gmake[3]: *** [Cache.o] Error 1
gmake[3]: Leaving directory `/tmp/trafficserver.git/iocore/cache'
gmake[2]: *** [all-recursive] Error 1
gmake[2]: Leaving directory `/tmp/trafficserver.git/iocore'
gmake[1]: *** [all-recursive] Error 1
gmake[1]: Leaving directory `/tmp/trafficserver.git'
gmake: *** [all] Error 2


> cache should not hold parititon lock over callbacks/review locking
> ------------------------------------------------------------------
>
>                 Key: TS-158
>                 URL: https://issues.apache.org/jira/browse/TS-158
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Cache
>            Reporter: John Plevyak
>            Assignee: John Plevyak
>             Fix For: 2.1.0
>
>
> Because we now have far fewer partitions it would be best
> not to hold the partition lock over callbacks to reduce contention
> on this lock and enable better scaling.
> This is a good opportunity to review the locking in the cache as
> well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to