[patch] reduce stack footprint of functions in drivers/net/sk98lin/skgepnmi.c
From: Ingo Molnar
Date: Sat Jul 09 2005 - 09:04:27 EST
this patch reduces the stack footprint of Vpd() from 1018 bytes to 28
bytes, SkPnmiGetStruct() from 744 bytes to 92 bytes, GetVpdKeyArr() from
552 bytes to 48 bytes, and General() from 364 bytes to 112 bytes.
build and boot tested. (but i likely did not excercise all the possible
codepaths affected by this patch)
Signed-off-by: Ingo Molnar <mingo@xxxxxxx>
Index: linux/drivers/net/sk98lin/skgepnmi.c
===================================================================
--- linux.orig/drivers/net/sk98lin/skgepnmi.c
+++ linux/drivers/net/sk98lin/skgepnmi.c
@@ -722,13 +722,20 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
unsigned int InstanceCnt;
SK_U32 Instance;
unsigned int TmpLen;
- char KeyArr[SK_PNMI_VPD_ENTRIES][SK_PNMI_VPD_KEY_SIZE];
+ char **KeyArr;
SK_DBG_MSG(pAC, SK_DBGMOD_PNMI, SK_DBGCAT_CTRL,
("PNMI: SkPnmiGetStruct: Called, BufLen=%d, NetIndex=%d\n",
*pLen, NetIndex));
+ KeyArr = kmalloc(SK_PNMI_VPD_ENTRIES*SK_PNMI_VPD_KEY_SIZE, GFP_KERNEL);
+ if (!KeyArr)
+ return (SK_PNMI_ERR_GENERAL);
+
+#define return_free(code) \
+ do { kfree(KeyArr); return (code); } while (0)
+
if (*pLen < SK_PNMI_STRUCT_SIZE) {
if (*pLen >= SK_PNMI_MIN_STRUCT_SIZE) {
@@ -738,14 +745,14 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
}
*pLen = SK_PNMI_STRUCT_SIZE;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/*
* Check NetIndex
*/
if (NetIndex >= pAC->Rlmt.NumNets) {
- return (SK_PNMI_ERR_UNKNOWN_NET);
+ return_free (SK_PNMI_ERR_UNKNOWN_NET);
}
/* Update statistic */
@@ -756,21 +763,21 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_SET_STAT(pBuf, Ret, (SK_U32)(-1));
*pLen = SK_PNMI_MIN_STRUCT_SIZE;
- return (Ret);
+ return_free (Ret);
}
if ((Ret = RlmtUpdate(pAC, IoC, NetIndex)) != SK_PNMI_ERR_OK) {
SK_PNMI_SET_STAT(pBuf, Ret, (SK_U32)(-1));
*pLen = SK_PNMI_MIN_STRUCT_SIZE;
- return (Ret);
+ return_free (Ret);
}
if ((Ret = SirqUpdate(pAC, IoC)) != SK_PNMI_ERR_OK) {
SK_PNMI_SET_STAT(pBuf, Ret, (SK_U32)(-1));
*pLen = SK_PNMI_MIN_STRUCT_SIZE;
- return (Ret);
+ return_free (Ret);
}
/*
@@ -792,7 +799,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_CHECKFLAGS("SkPnmiGetStruct: On return");
SK_PNMI_SET_STAT(pBuf, Ret, (SK_U32)(-1));
*pLen = SK_PNMI_MIN_STRUCT_SIZE;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
/* Retrieve values */
@@ -848,7 +855,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_CHECKFLAGS("SkPnmiGetStruct: On return");
SK_PNMI_SET_STAT(pBuf, Ret, DstOffset);
*pLen = SK_PNMI_MIN_STRUCT_SIZE;
- return (Ret);
+ return_free (Ret);
}
}
}
@@ -860,9 +867,11 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
*pLen = SK_PNMI_STRUCT_SIZE;
SK_PNMI_CHECKFLAGS("SkPnmiGetStruct: On return");
SK_PNMI_SET_STAT(pBuf, SK_PNMI_ERR_OK, (SK_U32)(-1));
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
+#undef return_free
+
/*****************************************************************************
*
* SkPnmiPreSetStruct - Presets the management database in SK_PNMI_STRUCT_DATA
@@ -3023,8 +3032,8 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
{
SK_VPD_STATUS *pVpdStatus;
unsigned int BufLen;
- char Buf[256];
- char KeyArr[SK_PNMI_VPD_ENTRIES][SK_PNMI_VPD_KEY_SIZE];
+ char *Buf;
+ char **KeyArr;
char KeyStr[SK_PNMI_VPD_KEY_SIZE];
unsigned int KeyNo;
unsigned int Offset;
@@ -3035,13 +3044,25 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
int Ret;
SK_U32 Val32;
+ KeyArr = kmalloc(SK_PNMI_VPD_ENTRIES*SK_PNMI_VPD_KEY_SIZE, GFP_KERNEL);
+ if (!KeyArr)
+ return (SK_PNMI_ERR_GENERAL);
+ Buf = kmalloc(256, GFP_KERNEL);
+ if (!Buf) {
+ kfree(KeyArr);
+ return (SK_PNMI_ERR_GENERAL);
+ }
+
+#define return_free(code) \
+ do { kfree(KeyArr); kfree(Buf); return(code); } while (0)
+
/*
* Get array of all currently stored VPD keys
*/
Ret = GetVpdKeyArr(pAC, IoC, &KeyArr[0][0], sizeof(KeyArr), &KeyNo);
if (Ret != SK_PNMI_ERR_OK) {
*pLen = 0;
- return (Ret);
+ return_free (Ret);
}
/*
@@ -3072,13 +3093,13 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (Index == KeyNo) {
*pLen = 0;
- return (SK_PNMI_ERR_UNKNOWN_INST);
+ return_free (SK_PNMI_ERR_UNKNOWN_INST);
}
}
else if (Instance != 1) {
*pLen = 0;
- return (SK_PNMI_ERR_UNKNOWN_INST);
+ return_free (SK_PNMI_ERR_UNKNOWN_INST);
}
}
@@ -3094,7 +3115,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U32)) {
*pLen = sizeof(SK_U32);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/* Get number of free bytes */
pVpdStatus = VpdStat(pAC, IoC);
@@ -3104,7 +3125,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR017MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if ((pVpdStatus->vpd_status & VPD_VALID) == 0) {
@@ -3112,7 +3133,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR018MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Val32 = (SK_U32)pVpdStatus->vpd_free_rw;
@@ -3129,7 +3150,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/* Get value */
@@ -3155,7 +3176,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U32)) {
*pLen = sizeof(SK_U32);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
Val32 = (SK_U32)KeyNo;
@@ -3173,7 +3194,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/*
@@ -3207,14 +3228,14 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR021,
SK_PNMI_ERR021MSG);
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Offset += BufLen + 1;
}
if (*pLen < Offset) {
*pLen = Offset;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/*
@@ -3234,7 +3255,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR022MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
*(pBuf + Offset) = (char)BufLen;
@@ -3248,7 +3269,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < LastIndex - FirstIndex) {
*pLen = LastIndex - FirstIndex;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
for (Offset = 0, Index = FirstIndex;
@@ -3271,7 +3292,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < Offset) {
*pLen = Offset;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
SK_MEMSET(pBuf, 0, Offset);
*pLen = Offset;
@@ -3282,7 +3303,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR023MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
}
else {
@@ -3297,14 +3318,14 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
Id == OID_SKGE_VPD_ACCESS) {
*pLen = 0;
- return (SK_PNMI_ERR_READ_ONLY);
+ return_free (SK_PNMI_ERR_READ_ONLY);
}
SK_ERR_LOG(pAC, SK_ERRCL_SW, SK_PNMI_ERR024,
SK_PNMI_ERR024MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
/*
@@ -3314,7 +3335,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < 1) {
*pLen = 1;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
/*
@@ -3335,7 +3356,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < 4) {
*pLen = 4;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
KeyStr[0] = pBuf[1];
KeyStr[1] = pBuf[2];
@@ -3348,7 +3369,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (!VpdMayWrite(KeyStr)) {
*pLen = 0;
- return (SK_PNMI_ERR_BAD_VALUE);
+ return_free (SK_PNMI_ERR_BAD_VALUE);
}
Offset = (int)pBuf[3] & 0xFF;
@@ -3359,7 +3380,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
/* A preset ends here */
if (Action == SK_PNMI_PRESET) {
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
/* Write the new entry or modify an existing one */
@@ -3367,7 +3388,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (Ret == SK_PNMI_VPD_NOWRITE ) {
*pLen = 0;
- return (SK_PNMI_ERR_BAD_VALUE);
+ return_free (SK_PNMI_ERR_BAD_VALUE);
}
else if (Ret != SK_PNMI_VPD_OK) {
@@ -3375,7 +3396,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR025MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
/*
@@ -3389,7 +3410,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR026MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
break;
@@ -3398,12 +3419,12 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < 3) {
*pLen = 3;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
if (*pLen > 3) {
*pLen = 0;
- return (SK_PNMI_ERR_BAD_VALUE);
+ return_free (SK_PNMI_ERR_BAD_VALUE);
}
KeyStr[0] = pBuf[1];
KeyStr[1] = pBuf[2];
@@ -3424,12 +3445,12 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (Index == KeyNo) {
*pLen = 0;
- return (SK_PNMI_ERR_BAD_VALUE);
+ return_free (SK_PNMI_ERR_BAD_VALUE);
}
if (Action == SK_PNMI_PRESET) {
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
/* Ok, you wanted it and you will get it */
@@ -3440,7 +3461,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR027MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
/*
@@ -3454,19 +3475,22 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR028MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
break;
default:
*pLen = 0;
- return (SK_PNMI_ERR_BAD_VALUE);
+ return_free (SK_PNMI_ERR_BAD_VALUE);
}
}
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
+
}
+#undef return_free
+
/*****************************************************************************
*
* General - OID handler function of various single instance OIDs
@@ -3507,16 +3531,22 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_U64 Val64RxHwErrs = 0;
SK_U64 Val64TxHwErrs = 0;
SK_BOOL Is64BitReq = SK_FALSE;
- char Buf[256];
+ char *Buf;
int MacType;
+ Buf = kmalloc(256, GFP_KERNEL);
+ if (!Buf)
+ return (SK_PNMI_ERR_GENERAL);
+
+#define return_free(code) \
+ do { kfree(Buf); return(code); } while (0)
/*
* Check instance. We only handle single instance variables.
*/
if (Instance != (SK_U32)(-1) && Instance != 1) {
*pLen = 0;
- return (SK_PNMI_ERR_UNKNOWN_INST);
+ return_free (SK_PNMI_ERR_UNKNOWN_INST);
}
/*
@@ -3525,7 +3555,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (Action != SK_PNMI_GET) {
*pLen = 0;
- return (SK_PNMI_ERR_READ_ONLY);
+ return_free (SK_PNMI_ERR_READ_ONLY);
}
MacType = pAC->GIni.GIMacType;
@@ -3541,7 +3571,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
#ifndef SK_NDIS_64BIT_CTR
if (*pLen < sizeof(SK_U32)) {
*pLen = sizeof(SK_U32);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
#else /* SK_NDIS_64BIT_CTR */
@@ -3555,7 +3585,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
* if insufficient space is provided
*/
*pLen = sizeof(SK_U64);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
Is64BitReq = (*pLen < sizeof(SK_U64)) ? SK_FALSE : SK_TRUE;
@@ -3575,7 +3605,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U32)) {
*pLen = sizeof(SK_U32);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
break;
@@ -3583,7 +3613,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U16)) {
*pLen = sizeof(SK_U16);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
break;
@@ -3596,7 +3626,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U8)) {
*pLen = sizeof(SK_U8);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
break;
@@ -3619,7 +3649,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < sizeof(SK_U64)) {
*pLen = sizeof(SK_U64);
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
break;
@@ -3644,7 +3674,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (Ret != SK_PNMI_ERR_OK) {
*pLen = 0;
- return (Ret);
+ return_free (Ret);
}
pAC->Pnmi.MacUpdatedFlag ++;
@@ -3695,7 +3725,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
for (Offset = 0, Index = 0; Offset < Len;
Offset += sizeof(SK_U32), Index ++) {
@@ -3731,7 +3761,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR007MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Len = SK_STRLEN(pAC->Pnmi.pDriverDescription) + 1;
@@ -3741,13 +3771,13 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR029MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
*pBuf = (char)(Len - 1);
SK_MEMCPY(pBuf + 1, pAC->Pnmi.pDriverDescription, Len - 1);
@@ -3761,7 +3791,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR030MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Len = SK_STRLEN(pAC->Pnmi.pDriverVersion) + 1;
@@ -3771,13 +3801,13 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR031MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
*pBuf = (char)(Len - 1);
SK_MEMCPY(pBuf + 1, pAC->Pnmi.pDriverVersion, Len - 1);
@@ -3791,7 +3821,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR053MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Len = SK_STRLEN(pAC->Pnmi.pDriverReleaseDate) + 1;
@@ -3801,13 +3831,13 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR054MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
*pBuf = (char)(Len - 1);
SK_MEMCPY(pBuf + 1, pAC->Pnmi.pDriverReleaseDate, Len - 1);
@@ -3821,7 +3851,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR055MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Len = SK_STRLEN(pAC->Pnmi.pDriverFileName) + 1;
@@ -3831,13 +3861,13 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR056MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
*pBuf = (char)(Len - 1);
SK_MEMCPY(pBuf + 1, pAC->Pnmi.pDriverFileName, Len - 1);
@@ -3858,7 +3888,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR032MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
Len ++;
if (Len > SK_PNMI_STRINGLEN1) {
@@ -3867,12 +3897,12 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR033MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
*pBuf = (char)(Len - 1);
SK_MEMCPY(pBuf + 1, Buf, Len - 1);
@@ -3884,7 +3914,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < 5) {
*pLen = 5;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
Val8 = (SK_U8)pAC->GIni.GIPciHwRev;
pBuf[0] = 4;
@@ -3961,7 +3991,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
if (*pLen < Len) {
*pLen = Len;
- return (SK_PNMI_ERR_TOO_SHORT);
+ return_free (SK_PNMI_ERR_TOO_SHORT);
}
CopyTrapQueue(pAC, pBuf);
*pLen = Len;
@@ -4446,7 +4476,7 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
SK_PNMI_ERR034MSG);
*pLen = 0;
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
if (Id == OID_SKGE_RX_HW_ERROR_CTS ||
@@ -4459,9 +4489,11 @@ SK_U32 NetIndex) /* NetIndex (0..n), in
pAC->Pnmi.MacUpdatedFlag --;
}
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
+#undef return_free
+
/*****************************************************************************
*
* Rlmt - OID handler function of OID_SKGE_RLMT_XXX single instance.
@@ -6535,12 +6567,18 @@ unsigned int KeyArrLen, /* Length of arr
unsigned int *pKeyNo) /* Number of keys */
{
unsigned int BufKeysLen = SK_PNMI_VPD_BUFSIZE;
- char BufKeys[SK_PNMI_VPD_BUFSIZE];
+ char *BufKeys;
unsigned int StartOffset;
unsigned int Offset;
int Index;
int Ret;
+ BufKeys = kmalloc(SK_PNMI_VPD_BUFSIZE, GFP_KERNEL);
+ if (!BufKeys)
+ return (SK_PNMI_ERR_GENERAL);
+
+#define return_free(code) \
+ do { kfree(BufKeys); return (code); } while (0)
SK_MEMSET(pKeyArr, 0, KeyArrLen);
@@ -6554,12 +6592,12 @@ unsigned int *pKeyNo) /* Number of keys
SK_ERR_LOG(pAC, SK_ERRCL_SW, SK_PNMI_ERR014,
SK_PNMI_ERR014MSG);
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
/* If no keys are available return now */
if (*pKeyNo == 0 || BufKeysLen == 0) {
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
/*
* If the key list is too long for us trunc it and give a
@@ -6590,7 +6628,7 @@ unsigned int *pKeyNo) /* Number of keys
SK_ERR_LOG(pAC, SK_ERRCL_SW, SK_PNMI_ERR016,
SK_PNMI_ERR016MSG);
- return (SK_PNMI_ERR_GENERAL);
+ return_free (SK_PNMI_ERR_GENERAL);
}
SK_STRNCPY(pKeyArr + Index * SK_PNMI_VPD_KEY_SIZE,
@@ -6607,9 +6645,11 @@ unsigned int *pKeyNo) /* Number of keys
&BufKeys[StartOffset], SK_PNMI_VPD_KEY_SIZE);
}
- return (SK_PNMI_ERR_OK);
+ return_free (SK_PNMI_ERR_OK);
}
+#undef return_free
+
/*****************************************************************************
*
* SirqUpdate - Let the SIRQ update its internal values
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/