diff -r 8ab6687fb94c -r 3adadc800673 telephonyprotocols/rawipnif/rawipnif2/src/IPv6Binder.cpp --- a/telephonyprotocols/rawipnif/rawipnif2/src/IPv6Binder.cpp Thu May 27 14:05:07 2010 +0300 +++ b/telephonyprotocols/rawipnif/rawipnif2/src/IPv6Binder.cpp Fri Jun 11 14:49:29 2010 +0300 @@ -1,4 +1,4 @@ -// Copyright (c) 2006-2009 Nokia Corporation and/or its subsidiary(-ies). +// Copyright (c) 2006-2010 Nokia Corporation and/or its subsidiary(-ies). // All rights reserved. // This component and the accompanying materials are made available // under the terms of "Eclipse Public License v1.0" @@ -20,6 +20,12 @@ @file */ + +#include "OstTraceDefinitions.h" +#ifdef OST_TRACE_COMPILER_IN_USE +#include "IPv6BinderTraces.h" +#endif + #include #include #include "RawIP2Flow.h" @@ -33,15 +39,12 @@ #include #endif -#define LOG_IP_ADDRESS(desc,addr) _LOG_L2C5(_L8(" " desc " = %d:%d:%d:%d from context"), \ - addr.u.iAddr32[3], addr.u.iAddr32[2], addr.u.iAddr32[1], addr.u.iAddr32[0]); -CIPv6Binder::CIPv6Binder(CRawIP2Flow& aFlow, CBttLogger* aTheLogger) +CIPv6Binder::CIPv6Binder(CRawIP2Flow& aFlow) /** * Constructor */ - : CBinderBase(aFlow,aTheLogger), - iTheLogger(aTheLogger), + : CBinderBase(aFlow), iSpeedMetric(KDefaultSpeedMetric) { } @@ -77,8 +80,7 @@ * @return Standard error codes */ { - _LOG_L1C3(_L8("CIPv6Binder::Control [aLevel=%d, aName=%d]"), - aLevel, aName); + OstTraceDefExt2(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_CONTROL_1, "CIPv6Binder::Control [aLevel=%u, aName=%u]",aLevel, aName); if (aLevel == KSOLInterface) { @@ -189,7 +191,7 @@ * @return KErrArgument if an incorrect structure is passed, otherwise KErrNone */ { - _LOG_L1C1(_L8("CIPv6Binder::DeleteContext")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_DELETECONTEXT_1, "CIPv6Binder::DeleteContext"); if (aContextParameters.Length() != sizeof(TContextParameters)) { @@ -223,7 +225,7 @@ */ void CIPv6Binder::UpdateContextConfigL(const TPacketDataConfigBase& aConfig) { - _LOG_L1C1(_L8("CIPv6Binder::UpdateContextConfig")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_1, "CIPv6Binder::UpdateContextConfig"); // Get our IP address from the GPRS context config. TInetAddr address; @@ -249,12 +251,11 @@ if (ret == KErrNone) { iSettings.iLocalAddr = address.Ip6Address(); - LOG_IP_ADDRESS("Got local IP address", iSettings.iLocalAddr); + OstTraceDefExt4(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_2, "Got local IP address = %u:%u:%u:%u from context", iSettings.iLocalAddr.u.iAddr32[3], iSettings.iLocalAddr.u.iAddr32[2], iSettings.iLocalAddr.u.iAddr32[1], iSettings.iLocalAddr.u.iAddr32[0]); } else { - _LOG_L2C2(_L8("Couldn't get IP address from GPRS config (err: %d)"), - ret); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_3, "Couldn't get IP address from GPRS config (err: %d)",ret); // Don't leave on this error: we may still be OK if we read some // settings from CommDB. @@ -276,12 +277,11 @@ if (ret == KErrNone) { iSettings.iPrimaryDns = address.Ip6Address(); - LOG_IP_ADDRESS("Got primary DNS", iSettings.iPrimaryDns); + OstTraceDefExt4(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_4, "Got primary DNS = %u:%u:%u:%u from context", iSettings.iPrimaryDns.u.iAddr32[3], iSettings.iPrimaryDns.u.iAddr32[2], iSettings.iPrimaryDns.u.iAddr32[1], iSettings.iPrimaryDns.u.iAddr32[0]); } else { - _LOG_L2C2(_L8("Couldn't get primary DNS address from GPRS config (err: %d)"), - ret); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_5, "Couldn't get primary DNS address from GPRS config (err: %d)",ret); // Don't leave on this error: we may still be OK if we read some // settings from CommDB. @@ -293,12 +293,11 @@ if (ret == KErrNone) { iSettings.iSecondaryDns = address.Ip6Address(); - LOG_IP_ADDRESS("Got secondary DNS", iSettings.iPrimaryDns); + OstTraceDefExt4(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_6, "Got secondary DNS = %u:%u:%u:%u from context", iSettings.iSecondaryDns.u.iAddr32[3], iSettings.iSecondaryDns.u.iAddr32[2], iSettings.iSecondaryDns.u.iAddr32[1], iSettings.iSecondaryDns.u.iAddr32[0]); } else { - _LOG_L2C2(_L8("Couldn't get secondary DNS address from GPRS config (err: %d)"), - ret); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_7, "Couldn't get secondary DNS address from GPRS config (err: %d)",ret); // Don't leave on this error: we may still be OK if we read some // settings from CommDB. @@ -306,8 +305,8 @@ } else { - LOG_IP_ADDRESS("Using CommDB DNS address - Primary ", iSettings.iPrimaryDns); - LOG_IP_ADDRESS(" - Secondary ", iSettings.iSecondaryDns); + OstTraceDefExt4(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_8, "Using CommDB DNS address - Primary = %u:%u:%u:%u", iSettings.iPrimaryDns.u.iAddr32[3], iSettings.iPrimaryDns.u.iAddr32[2], iSettings.iPrimaryDns.u.iAddr32[1], iSettings.iPrimaryDns.u.iAddr32[0]); + OstTraceDefExt4(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONTEXTCONFIGL_9, " - Secondary = %u:%u:%u:%u", iSettings.iSecondaryDns.u.iAddr32[3], iSettings.iSecondaryDns.u.iAddr32[2], iSettings.iSecondaryDns.u.iAddr32[1], iSettings.iSecondaryDns.u.iAddr32[0]); } } @@ -318,7 +317,7 @@ * @param aConnectionSpeed Our connection speed */ { - _LOG_L1C1(_L8("CIPv6Binder::UpdateConnectionSpeed")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_UPDATECONNECTIONSPEED_1, "CIPv6Binder::UpdateConnectionSpeed"); iSpeedMetric = aConnectionSpeed; } @@ -336,7 +335,7 @@ * @return Standard error codes */ { - _LOG_L1C1(_L8("CIPv6Binder::Send")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_SEND1_1, "CIPv6Binder::Send"); #ifdef __BTT_LOGGING__ LogPacket(static_cast(aPdu)); @@ -362,7 +361,7 @@ * @param aInfo Not used */ { - _LOG_L1C1(_L8("CIPv6Binder::Notification")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_NOTIFICATION_1, "CIPv6Binder::Notification"); return KErrNone; } @@ -374,7 +373,7 @@ * @param aProtocol A pointer to a protocol */ { - _LOG_L1C1(_L8("CIPv6Binder::StartSending()")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_STARTSENDING_1, "CIPv6Binder::StartSending()"); CBinderBase::StartSending(); } @@ -385,8 +384,7 @@ * @param aProtocolCode The protocol type */ { - _LOG_L1C2(_L8("CIPv6Binder::WantsProtocol [aProtocolCode=%X]"), - aProtocolCode); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_WANTSPROTOCOL_1, "CIPv6Binder::WantsProtocol [aProtocolCode=%X]",aProtocolCode); #ifdef RAWIP_HEADER_APPENDED_TO_PACKETS return ((aProtocolCode & 0x00FF) == KIp6FrameType); @@ -409,7 +407,7 @@ * @param aPdu The incoming packet */ { - _LOG_L1C1(_L8("CIPv6Binder::Process")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_PROCESS1_1, "CIPv6Binder::Process"); #ifdef __BTT_LOGGING__ LogPacket(static_cast(aPdu)); @@ -427,12 +425,12 @@ // been bound yet. if (iUpperReceiver && WantsProtocol(protocolCode)) { - _LOG_L1C1(_L8("CIPv6Binder: Packet Sent to TCP/IP Protocol!!!")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_PROCESS1_2, "CIPv6Binder: Packet Sent to TCP/IP Protocol!!!"); iUpperReceiver->Process(static_cast(aPdu)); } else { - _LOG_L2C1(_L8("WARNING: dumping incoming packet, no protocol bound")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_PROCESS1_3, "WARNING: dumping incoming packet, no protocol bound"); aPdu.Free(); } @@ -511,19 +509,18 @@ * @param aPacket The packet */ { - _LOG_L1C1(_L8("CIPv6Binder::LogPacket")); + OstTraceDef0(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_1, "CIPv6Binder::LogPacket"); TInt mBufLength = aPacket.Length() - aPacket.First()->Length(); - _LOG_L3C2(_L8("Analysis of %d byte packet:"), mBufLength); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_2, "Analysis of %d byte packet:", mBufLength); //Note: All the constants used on this method are a pragmatic guess of the //IP header fields. The only porpose of this method is logging. if (mBufLength < 40) { - _LOG_L3C2(_L8(" -doesn't appear to be a valid IPv6 packet (length=%d)") - , mBufLength); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_3, " -doesn't appear to be a valid IPv6 packet (length=%d)", mBufLength); return; } @@ -532,18 +529,14 @@ if ((payloadPtr[0] & 0xF0) != 0x60) { - _LOG_L3C2(_L8(" - doesn't appear to be an IPv6 packet (version=0x%X)"), - (payloadPtr[0] & 0xF0) >> 4); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_4, " - doesn't appear to be an IPv6 packet (version=0x%X)",(payloadPtr[0] & 0xF0) >> 4); return; } - _LOG_L3C2(_L8(" - traffic class: 0x%X"), - ((payloadPtr[0] & 0xF) << 4) | ((payloadPtr[1] & 0xF0) >> 4)); - _LOG_L3C2(_L8(" - flow label: 0x%X"), - ((payloadPtr[1] & 0x0F) << 16) | (payloadPtr[2] << 8) | payloadPtr[3]); - _LOG_L3C2(_L8(" - payload length: 0x%X"), - (payloadPtr[4] << 16) | payloadPtr[5]); - _LOG_L3C2(_L8(" - next header: 0x%08X"), payloadPtr[6]); - _LOG_L3C2(_L8(" - hop limit: 0x%08X"), payloadPtr[7]); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_5, " - traffic class: 0x%X", ((payloadPtr[0] & 0xF) << 4) | ((payloadPtr[1] & 0xF0) >> 4)); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_6, " - flow label: 0x%X", ((payloadPtr[1] & 0x0F) << 16) | (payloadPtr[2] << 8) | payloadPtr[3]); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_7, " - payload length: 0x%X", (payloadPtr[4] << 16) | payloadPtr[5]); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_8, " - next header: 0x%08X", payloadPtr[6]); + OstTraceDef1(OST_TRACE_CATEGORY_DEBUG, TRACE_INTERNALS, CIPV6BINDER_LOGPACKET_9, " - hop limit: 0x%08X", payloadPtr[7]); } #endif // __BTT_LOGGING__