mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-11-29 23:01:03 +01:00
Code Issues Projects Releases Wiki Activity

[MINOR] halog: gain back performance before SKIP_CHAR fix

Browse Source 
The SKIP_CHAR fix caused a measurable performance drop. Since we can
consider all chars below 0x20 as delimiters, we can avoid a cache lookup
which requires a char to pointer conversion.
This commit is contained in:
Willy Tarreau 2011-07-10 18:15:08 +02:00
parent 70c428f7c6
commit df6f0d1e49
1 changed files with 7 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
contrib/halog/halog.c
View File

@ -36,13 +36,8 @@
#define MAXLINE 16384 #define MAXLINE 16384
#define QBITS 4 #define QBITS 4
const char sep[256] = { #define SEP(c) ((unsigned char)(c) <= ' ')
[0] = 1, #define SKIP_CHAR(p,c) do { while (1) { int __c = (unsigned char)*p++; if (__c == c) break; if (__c <= ' ') { p--; break; } } } while (0)
[' '] = 1, ['\t'] = 1,
['\r'] = 1, ['\n'] = 1,
};
#define SKIP_CHAR(p,c) do { while (1) if (sep[(unsigned char)*p]) break; else if (*(p++) == c) break; } while (0)
/* [0] = err/date, [1] = req, [2] = conn, [3] = resp, [4] = data */ /* [0] = err/date, [1] = req, [2] = conn, [3] = resp, [4] = data */
static struct eb_root timers[5] = { static struct eb_root timers[5] = {
@ -515,7 +510,7 @@ int main(int argc, char **argv)
p = b; p = b;
err = 0; err = 0;
f = 0; f = 0;
while (!sep[(unsigned char)*p]) { while (!SEP(*p)) {
if (++f == 4) if (++f == 4)
break; break;
SKIP_CHAR(p, '/'); SKIP_CHAR(p, '/');
@ -539,7 +534,7 @@ int main(int argc, char **argv)
p = b; p = b;
err = 0; err = 0;
f = 0; f = 0;
while (!sep[(unsigned char)*p]) { while (!SEP(*p)) {
tps = str2ic(p); tps = str2ic(p);
if (tps < 0) { if (tps < 0) {
tps = -1; tps = -1;
@ -612,7 +607,7 @@ int main(int argc, char **argv)
p = b; p = b;
err = 0; err = 0;
f = 0; f = 0;
while (!sep[(unsigned char)*p]) { while (!SEP(*p)) {
array[f] = str2ic(p); array[f] = str2ic(p);
if (array[f] < 0) { if (array[f] < 0) {
array[f] = -1; array[f] = -1;
@ -777,7 +772,7 @@ int main(int argc, char **argv)
p = b; p = b;
err = 0; err = 0;
f = 0; f = 0;
while (!sep[(unsigned char)*p]) { while (!SEP(*p)) {
array[f] = str2ic(p); array[f] = str2ic(p);
if (array[f] < 0) { if (array[f] < 0) {
array[f] = -1; array[f] = -1;
@ -845,7 +840,7 @@ int main(int argc, char **argv)
* parse the 5 timers to detect errors, it takes avg 55 ns per line. * parse the 5 timers to detect errors, it takes avg 55 ns per line.
*/ */
e = b; err = 0; f = 0; e = b; err = 0; f = 0;
while (!sep[(unsigned char)*e]) { while (!SEP(*e)) {
array[f] = str2ic(e); array[f] = str2ic(e);
if (array[f] < 0) { if (array[f] < 0) {
array[f] = -1; array[f] = -1;