You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2014/07/16 05:57:18 UTC
[1/3] git commit: TS-2859: remove DBG macros to not generate warnings
from GCC 4.9
Repository: trafficserver
Updated Branches:
refs/heads/4.2.x 311e1152d -> de2f11742
TS-2859: remove DBG macros to not generate warnings from GCC 4.9
(cherry picked from commit e7586dfee7a04b44e3c378a484cc1e776e847b27)
Conflicts:
CHANGES
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/54869408
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/54869408
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/54869408
Branch: refs/heads/4.2.x
Commit: 548694089763c985761463edce4d31882e06e48a
Parents: 311e115
Author: Marcin Juszkiewicz <mj...@redhat.com>
Authored: Thu May 29 12:15:43 2014 -0700
Committer: Phil Sorber <so...@apache.org>
Committed: Tue Jul 15 21:46:20 2014 -0600
----------------------------------------------------------------------
CHANGES | 2 ++
lib/ts/ink_defs.h | 22 ----------------------
2 files changed, 2 insertions(+), 22 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/54869408/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index e601615..5231d41 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 4.2.2
+ *) [TS-2859] Remove DBG macros to not generate warnings from GCC 4.9.
+
*) [TS-2560] regex_remap lowercase substitutions is not being initialized in
the construtor
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/54869408/lib/ts/ink_defs.h
----------------------------------------------------------------------
diff --git a/lib/ts/ink_defs.h b/lib/ts/ink_defs.h
index b5e8122..ad728a5 100644
--- a/lib/ts/ink_defs.h
+++ b/lib/ts/ink_defs.h
@@ -131,28 +131,6 @@ countof(const T (&)[N]) {
#define ROUNDUP(x, y) ((((x)+((y)-1))/(y))*(y))
#endif
-/* Debugging
-*/
-#ifdef NDEBUG
-
-#define FDBG
-#define DBG(s)
-#define DBG1(s,a)
-#define DBG2(s,a1,a2)
-#define DBG3(s,a1,a2,a3)
-#define DBG4(s,a1,a2,a3,a4)
-
-#else
-
-#define FDBG if (debug_level==1) printf("debug "__FILE__":%d %s : entered\n" ,__LINE__,__FUNCTION__)
-#define DBG(s) if (debug_level==1) printf("debug "__FILE__":%d %s :" s ,__LINE__,__FUNCTION__)
-#define DBG1(s,a) if (debug_level==1) printf("debug "__FILE__":%d %s :" s ,__LINE__,__FUNCTION__, a)
-#define DBG2(s,a1,a2) if (debug_level==1) printf("debug "__FILE__":%d %s :" s ,__LINE__,__FUNCTION__, a1,a2)
-#define DBG3(s,a1,a2,a3) if (debug_level==1) printf("debug "__FILE__":%d %s :" s ,__LINE__,__FUNCTION__, a1,a2,a3)
-#define DBG4(s,a1,a2,a3,a4) if (debug_level==1) printf("debug "__FILE__":%d %s :" s ,__LINE__,__FUNCTION__, a1,a2,a3,a4)
-
-#endif
-
/* Types
*/
typedef void *(*VPVP_PFN) (void *);
[3/3] git commit: TS-2931: plugin metrics fail after a crash
Posted by so...@apache.org.
TS-2931: plugin metrics fail after a crash
If a plugin uses TSStatFindName followed by TSStatCreate, TSStatFindName
can return 0 for all the metric IDs after a traffic_server crash.
This will happen every time with the following conditions:
1. traffic_manager has pulled the stat records from traffic_server
2. traffic_server crashes
When traffic_server comes back up, it pulls the records from
traffic_manager. traffic_manager sends the records including the
rsb_id field. However, RecForceInsert() does not copy the rsb_id
field from the message to the actual record. Subsequent stat lookups
succeed because the stat is registered, but always receive a rsb_id
of 0.
The fix is to ensure that we copy the rsb_id field so that stat
lookups succeed.
(cherry picked from commit d537357da69ee701c7165f0e1f07554e18324e1f)
Conflicts:
CHANGES
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/de2f1174
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/de2f1174
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/de2f1174
Branch: refs/heads/4.2.x
Commit: de2f11742c4e0ae1b4e5ee996dda204c2f59e289
Parents: 19f9417
Author: James Peach <jp...@apache.org>
Authored: Wed Jul 9 13:27:01 2014 -0700
Committer: Phil Sorber <so...@apache.org>
Committed: Tue Jul 15 21:55:57 2014 -0600
----------------------------------------------------------------------
CHANGES | 2 ++
lib/records/P_RecCore.cc | 6 +++---
lib/records/RecCore.cc | 3 +++
proxy/InkAPI.cc | 10 +++++-----
4 files changed, 13 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/de2f1174/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 6756f04..f0f96e5 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 4.2.2
+ *) [TS-2931] Plugin metrics fail after a crash.
+
*) [TS-2776] Core dump inside openssl library
*) [TS-2859] Remove DBG macros to not generate warnings from GCC 4.9.
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/de2f1174/lib/records/P_RecCore.cc
----------------------------------------------------------------------
diff --git a/lib/records/P_RecCore.cc b/lib/records/P_RecCore.cc
index b8498e4..ec563a7 100644
--- a/lib/records/P_RecCore.cc
+++ b/lib/records/P_RecCore.cc
@@ -113,7 +113,7 @@ send_push_message()
if (i_am_the_record_owner(r->rec_type)) {
if (r->sync_required & REC_PEER_SYNC_REQUIRED) {
m = RecMessageMarshal_Realloc(m, r);
- r->sync_required = r->sync_required & ~REC_PEER_SYNC_REQUIRED;
+ r->sync_required &= ~REC_PEER_SYNC_REQUIRED;
send_msg = true;
}
}
@@ -157,10 +157,10 @@ send_pull_message(RecMessageT msg_type)
r = &(g_records[i]);
if (i_am_the_record_owner(r->rec_type) ||
(REC_TYPE_IS_STAT(r->rec_type) && !(r->registered)) ||
- (REC_TYPE_IS_STAT(r->rec_type) && !(r->stat_meta.persist_type != RECP_NON_PERSISTENT))) {
+ (REC_TYPE_IS_STAT(r->rec_type) && (r->stat_meta.persist_type == RECP_NON_PERSISTENT))) {
rec_mutex_acquire(&(r->lock));
m = RecMessageMarshal_Realloc(m, r);
- r->sync_required = r->sync_required & ~REC_PEER_SYNC_REQUIRED;
+ r->sync_required &= ~REC_PEER_SYNC_REQUIRED;
rec_mutex_release(&(r->lock));
}
}
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/de2f1174/lib/records/RecCore.cc
----------------------------------------------------------------------
diff --git a/lib/records/RecCore.cc b/lib/records/RecCore.cc
index dded7cb..c8fbd6c 100644
--- a/lib/records/RecCore.cc
+++ b/lib/records/RecCore.cc
@@ -831,7 +831,10 @@ RecForceInsert(RecRecord * record)
// set the record value
RecDataSet(r->data_type, &(r->data), &(record->data));
RecDataSet(r->data_type, &(r->data_default), &(record->data_default));
+
r->registered = record->registered;
+ r->rsb_id = record->rsb_id;
+
if (REC_TYPE_IS_STAT(r->rec_type)) {
r->stat_meta.persist_type = record->stat_meta.persist_type;
r->stat_meta.data_raw = record->stat_meta.data_raw;
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/de2f1174/proxy/InkAPI.cc
----------------------------------------------------------------------
diff --git a/proxy/InkAPI.cc b/proxy/InkAPI.cc
index a0f752f..f26e6f6 100644
--- a/proxy/InkAPI.cc
+++ b/proxy/InkAPI.cc
@@ -85,11 +85,11 @@
_HDR.m_mime = _HDR.m_http->m_fields_impl;
// Globals for new librecords stats
-volatile int top_stat = 0;
-RecRawStatBlock *api_rsb;
+static volatile int api_rsb_index = 0;
+static RecRawStatBlock * api_rsb;
// Globals for the Sessions/Transaction index registry
-volatile int next_argv_index = 0;
+static volatile int next_argv_index = 0;
struct _STATE_ARG_TABLE {
char* name;
@@ -5548,7 +5548,7 @@ TSHttpArgIndexReserve(const char* name, const char* description, int *arg_idx)
{
sdk_assert(sdk_sanity_check_null_ptr(arg_idx) == TS_SUCCESS);
- int volatile ix = ink_atomic_increment(&next_argv_index, 1);
+ int ix = ink_atomic_increment(&next_argv_index, 1);
if (ix < HTTP_SSN_TXN_MAX_USER_ARG) {
state_arg_table[ix].name = ats_strdup(name);
@@ -6638,7 +6638,7 @@ TSCacheScan(TSCont contp, TSCacheKey key, int KB_per_second)
int
TSStatCreate(const char *the_name, TSRecordDataType the_type, TSStatPersistence persist, TSStatSync sync)
{
- int id = ink_atomic_increment(&top_stat, 1);
+ int id = ink_atomic_increment(&api_rsb_index, 1);
RecRawStatSyncCb syncer = RecRawStatSyncCount;
// TODO: This only supports "int" data types at this point, since the "Raw" stats
[2/3] git commit: TS-2776: Core dump inside openssl library
Posted by so...@apache.org.
TS-2776: Core dump inside openssl library
(cherry picked from commit 9c6bb37b39504056683835038aa17fcaedb3c31d)
Conflicts:
CHANGES
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/19f94176
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/19f94176
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/19f94176
Branch: refs/heads/4.2.x
Commit: 19f941763d8f4df31f1527615d074290d4785a04
Parents: 5486940
Author: Bryan Call <bc...@apache.org>
Authored: Tue Jun 3 09:57:24 2014 -0700
Committer: Phil Sorber <so...@apache.org>
Committed: Tue Jul 15 21:52:02 2014 -0600
----------------------------------------------------------------------
CHANGES | 2 ++
iocore/net/SSLUtils.cc | 5 ++++-
2 files changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/19f94176/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 5231d41..6756f04 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 4.2.2
+ *) [TS-2776] Core dump inside openssl library
+
*) [TS-2859] Remove DBG macros to not generate warnings from GCC 4.9.
*) [TS-2560] regex_remap lowercase substitutions is not being initialized in
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/19f94176/iocore/net/SSLUtils.cc
----------------------------------------------------------------------
diff --git a/iocore/net/SSLUtils.cc b/iocore/net/SSLUtils.cc
index 88ffcfb..56c0182 100644
--- a/iocore/net/SSLUtils.cc
+++ b/iocore/net/SSLUtils.cc
@@ -441,7 +441,10 @@ SSLInitServerContext(
}
#ifdef SSL_MODE_RELEASE_BUFFERS
- SSL_CTX_set_mode(ctx, SSL_MODE_RELEASE_BUFFERS);
+ if (OPENSSL_VERSION_NUMBER > 0x1000107fL) {
+ Debug("ssl", "enabling SSL_MODE_RELEASE_BUFFERS");
+ SSL_CTX_set_mode(ctx, SSL_MODE_RELEASE_BUFFERS);
+ }
#endif
SSL_CTX_set_quiet_shutdown(ctx, 1);