Experimental Physics and
Industrial Control System

Ø  This description implies that some iocsh callable routine probably needs to make a strcpy() of its parameter name string argument, or to change it from iocshArgString to iocshArgPersistentString in the iocshArg definition (which does that for you).

 

I don’t think that is likely to be the problem for 2 reasons:

-          I don’t think these parameter names are passed via iocsh callable functions, they are hardcoded in the driver.

-          asynPortDriver::createParam(const char *name, asynParamType type, int *index) passes the pointer to the parameter name to one of these paramVal constructors

 

paramVal::paramVal(const char *name):

    type(asynParamNotDefined), status_(asynSuccess), alarmStatus_(0), alarmSeverity_(0),

    valueDefined(false), valueChanged(false)

{

    this->name = epicsStrDup(name);

}

 

paramVal::paramVal(const char *name, asynParamType type):

    type(type), status_(asynSuccess), alarmStatus_(0), alarmSeverity_(0),

    valueDefined(false), valueChanged(false){

    this->name = epicsStrDup(name);

    memset(&this->data, 0, sizeof(this->data));

}

 

Note that both of these copy the name to the paramVal object using epicsStrDup, so they do not assume a persistent pointer.

 

Mark

 

 

 

 

Hi Adrian,

 

Your parameter library looks fine before iocInit, but after iocInit all of the parameter names have changed to either an empty string "", or "BPM1".

 

Does your driver implement the drvUserCreate() method?  If so, that gets called during iocInit and perhaps you have an error in that method.  The names of the parameters should not be changing after they are created.

 

The only parameter whose name has not changed after iocInit is this one:

 

Parameter 28 type=asynInt32, name=enable_compensation, value=0, status=0

 

Perhaps that is a clue?

This description implies that some iocsh callable routine probably needs to make a strcpy() of its parameter name string argument, or to change it from iocshArgString to iocshArgPersistentString in the iocshArg definition (which does that for you).

- Andrew

-- 

Complexity comes for free, Simplicity you have to work for.