adapt to new ownership rules

This commit is contained in:
Michael Davidsaver
2017-07-11 16:00:44 +02:00
parent da30a12b72
commit 1756d70f4d
6 changed files with 70 additions and 142 deletions

View File

@ -19,7 +19,7 @@ bool getS(const epics::pvData::PVStructurePtr& S, const char *name, T& val)
struct BaseChannel : public epics::pvAccess::Channel struct BaseChannel : public epics::pvAccess::Channel
{ {
BaseChannel(const std::string& name, BaseChannel(const std::string& name,
const std::tr1::shared_ptr<epics::pvAccess::ChannelProvider>& prov, const std::tr1::weak_ptr<epics::pvAccess::ChannelProvider>& prov,
const epics::pvAccess::ChannelRequester::shared_pointer& req, const epics::pvAccess::ChannelRequester::shared_pointer& req,
const epics::pvData::StructureConstPtr& dtype const epics::pvData::StructureConstPtr& dtype
) )
@ -27,110 +27,33 @@ struct BaseChannel : public epics::pvAccess::Channel
{} {}
virtual ~BaseChannel() {} virtual ~BaseChannel() {}
epicsMutex lock; mutable epicsMutex lock;
typedef epicsGuard<epicsMutex> guard_t; typedef epicsGuard<epicsMutex> guard_t;
const std::string pvname; const std::string pvname;
std::tr1::shared_ptr<epics::pvAccess::ChannelProvider> provider; const epics::pvAccess::ChannelProvider::weak_pointer provider;
typedef epics::pvAccess::ChannelRequester::shared_pointer requester_t; const requester_type::weak_pointer requester;
requester_t requester;
const epics::pvData::StructureConstPtr fielddesc; const epics::pvData::StructureConstPtr fielddesc;
// assume Requester methods not called after destory() // assume Requester methods not called after destory()
virtual std::string getRequesterName() { guard_t G(lock); return requester->getRequesterName(); } virtual std::string getRequesterName() OVERRIDE
{ return getChannelRequester()->getRequesterName(); }
virtual void destroy() { virtual void destroy() OVERRIDE FINAL {}
std::tr1::shared_ptr<epics::pvAccess::ChannelProvider> prov;
requester_t req;
{
guard_t G(lock);
provider.swap(prov);
requester.swap(req);
}
}
virtual std::tr1::shared_ptr<epics::pvAccess::ChannelProvider> getProvider() { guard_t G(lock); return provider; } virtual std::tr1::shared_ptr<epics::pvAccess::ChannelProvider> getProvider() OVERRIDE FINAL
virtual std::string getRemoteAddress() { guard_t G(lock); return requester->getRequesterName(); } { return epics::pvAccess::ChannelProvider::shared_pointer(provider); }
virtual ConnectionState getConnectionState() { return epics::pvAccess::Channel::CONNECTED; } virtual std::string getRemoteAddress() OVERRIDE
virtual std::string getChannelName() { return pvname; } { return getRequesterName(); }
virtual std::tr1::shared_ptr<epics::pvAccess::ChannelRequester> getChannelRequester() { guard_t G(lock); return requester; }
virtual bool isConnected() { return getConnectionState()==epics::pvAccess::Channel::CONNECTED; }
virtual void getField(epics::pvAccess::GetFieldRequester::shared_pointer const & requester,std::string const & subField) virtual std::string getChannelName() OVERRIDE FINAL { return pvname; }
virtual std::tr1::shared_ptr<epics::pvAccess::ChannelRequester> getChannelRequester() OVERRIDE FINAL
{ return requester_type::shared_pointer(requester); }
virtual void getField(epics::pvAccess::GetFieldRequester::shared_pointer const & requester,std::string const & subField) OVERRIDE
{ requester->getDone(epics::pvData::Status(), fielddesc); } { requester->getDone(epics::pvData::Status(), fielddesc); }
virtual epics::pvAccess::AccessRights getAccessRights(const epics::pvData::PVField::shared_pointer &pvField) virtual void printInfo(std::ostream& out) OVERRIDE {
{ return epics::pvAccess::readWrite; }
virtual epics::pvAccess::ChannelProcess::shared_pointer createChannelProcess(
epics::pvAccess::ChannelProcessRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelProcess::shared_pointer ret;
requester->channelProcessConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"), ret);
return ret;
}
virtual epics::pvAccess::ChannelGet::shared_pointer createChannelGet(
epics::pvAccess::ChannelGetRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelGet::shared_pointer ret;
requester->channelGetConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"),
ret, epics::pvData::StructureConstPtr());
return ret;
}
virtual epics::pvAccess::ChannelPut::shared_pointer createChannelPut(
epics::pvAccess::ChannelPutRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelPut::shared_pointer ret;
requester->channelPutConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"),
ret, epics::pvData::StructureConstPtr());
return ret;
}
virtual epics::pvAccess::ChannelPutGet::shared_pointer createChannelPutGet(
epics::pvAccess::ChannelPutGetRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelPutGet::shared_pointer ret;
requester->channelPutGetConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"),
ret, epics::pvData::StructureConstPtr(), epics::pvData::StructureConstPtr());
return ret;
}
virtual epics::pvAccess::ChannelRPC::shared_pointer createChannelRPC(
epics::pvAccess::ChannelRPCRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelRPC::shared_pointer ret;
requester->channelRPCConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"), ret);
return ret;
}
virtual epics::pvData::Monitor::shared_pointer createMonitor(
epics::pvData::MonitorRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvData::Monitor::shared_pointer ret;
requester->monitorConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"),
ret, epics::pvData::StructureConstPtr());
return ret;
}
virtual epics::pvAccess::ChannelArray::shared_pointer createChannelArray(
epics::pvAccess::ChannelArrayRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest)
{
epics::pvAccess::ChannelArray::shared_pointer ret;
requester->channelArrayConnect(epics::pvData::Status(epics::pvData::Status::STATUSTYPE_FATAL, "Not implemented"),
ret, epics::pvData::Array::const_shared_pointer());
return ret;
}
virtual void printInfo() { printInfo(std::cout); }
virtual void printInfo(std::ostream& out) {
out<<"Channel '"<<pvname<<"' "<<getRemoteAddress()<<"\n"; out<<"Channel '"<<pvname<<"' "<<getRemoteAddress()<<"\n";
} }
}; };
@ -155,7 +78,7 @@ struct BaseMonitor : public epics::pvAccess::Monitor
typedef epicsGuard<epicsMutex> guard_t; typedef epicsGuard<epicsMutex> guard_t;
private: private:
requester_t::shared_pointer requester; const requester_t::weak_pointer requester;
epics::pvData::PVStructurePtr complete; epics::pvData::PVStructurePtr complete;
epics::pvData::BitSet changed, overflow; epics::pvData::BitSet changed, overflow;
@ -186,13 +109,11 @@ public:
epics::pvData::StructureConstPtr dtype(value->getStructure()); epics::pvData::StructureConstPtr dtype(value->getStructure());
epics::pvData::PVDataCreatePtr create(epics::pvData::getPVDataCreate()); epics::pvData::PVDataCreatePtr create(epics::pvData::getPVDataCreate());
BaseMonitor::shared_pointer self(shared_from_this()); BaseMonitor::shared_pointer self(shared_from_this());
requester_t::shared_pointer req; requester_t::shared_pointer req(requester.lock());
{ {
guard_t G(lock); guard_t G(lock);
assert(!complete); // can't call twice assert(!complete); // can't call twice
req = requester;
complete = value; complete = value;
empty.resize(nbuffers); empty.resize(nbuffers);
for(size_t i=0; i<empty.size(); i++) { for(size_t i=0; i<empty.size(); i++) {
@ -200,7 +121,8 @@ public:
} }
} }
epics::pvData::Status sts; epics::pvData::Status sts;
req->monitorConnect(sts, self, dtype); if(req)
req->monitorConnect(sts, self, dtype);
} }
struct no_overflow {}; struct no_overflow {};
@ -218,7 +140,7 @@ public:
if(empty.empty()) return false; if(empty.empty()) return false;
if(p_postone()) if(p_postone())
req = requester; req = requester.lock();
inoverflow = false; inoverflow = false;
} }
if(req) req->monitorEvent(shared_from_this()); if(req) req->monitorEvent(shared_from_this());
@ -240,7 +162,7 @@ public:
} else { } else {
if(p_postone()) if(p_postone())
req = requester; req = requester.lock();
oflow = inoverflow = false; oflow = inoverflow = false;
} }
} }
@ -267,7 +189,7 @@ public:
changed |= updated; changed |= updated;
if(p_postone()) if(p_postone())
req = requester; req = requester.lock();
oflow = inoverflow = false; oflow = inoverflow = false;
} }
} }
@ -292,7 +214,7 @@ public:
changed |= updated; changed |= updated;
if(p_postone()) if(p_postone())
req = requester; req = requester.lock();
oflow = inoverflow = false; oflow = inoverflow = false;
} }
} }
@ -332,7 +254,6 @@ public:
virtual void destroy() virtual void destroy()
{ {
requester_t::shared_pointer req;
bool run; bool run;
{ {
guard_t G(lock); guard_t G(lock);
@ -340,7 +261,6 @@ public:
if(run) { if(run) {
running = false; running = false;
} }
requester.swap(req);
} }
if(run) if(run)
this->onStop(); this->onStop();

View File

@ -313,7 +313,7 @@ struct PDBProcessor
size_t PDBProvider::ninstances; size_t PDBProvider::ninstances;
PDBProvider::PDBProvider() PDBProvider::PDBProvider(const epics::pvAccess::Configuration::shared_pointer &)
{ {
PDBProcessor proc; PDBProcessor proc;
pvd::FieldCreatePtr fcreate(pvd::getFieldCreate()); pvd::FieldCreatePtr fcreate(pvd::getFieldCreate());

View File

@ -3,6 +3,7 @@
#include <dbEvent.h> #include <dbEvent.h>
#include <pv/configuration.h>
#include <pv/pvAccess.h> #include <pv/pvAccess.h>
#include "weakmap.h" #include "weakmap.h"
@ -29,7 +30,7 @@ struct PDBProvider : public epics::pvAccess::ChannelProvider,
{ {
POINTER_DEFINITIONS(PDBProvider); POINTER_DEFINITIONS(PDBProvider);
PDBProvider(); explicit PDBProvider(const epics::pvAccess::Configuration::shared_pointer& =epics::pvAccess::Configuration::shared_pointer());
virtual ~PDBProvider(); virtual ~PDBProvider();
virtual void destroy(); virtual void destroy();
virtual std::string getProviderName(); virtual std::string getProviderName();

View File

@ -168,7 +168,9 @@ void commonGet(PVIF *pvif, pvd::BitSet* changed)
void PDBSingleGet::get() void PDBSingleGet::get()
{ {
commonGet(pvif.get(), changed.get()); commonGet(pvif.get(), changed.get());
requester->getDone(pvd::Status(), shared_from_this(), pvf, changed); requester_type::shared_pointer req(requester.lock());
if(req)
req->getDone(pvd::Status(), shared_from_this(), pvf, changed);
} }
@ -257,13 +259,17 @@ void PDBSinglePut::put(pvd::PVStructure::shared_pointer const & value,
} }
} }
requester->putDone(pvd::Status(), shared_from_this()); requester_type::shared_pointer req(requester.lock());
if(req)
req->putDone(pvd::Status(), shared_from_this());
} }
void PDBSinglePut::get() void PDBSinglePut::get()
{ {
commonGet(pvif.get(), changed.get()); commonGet(pvif.get(), changed.get());
requester->getDone(pvd::Status(), shared_from_this(), pvf, changed); requester_type::shared_pointer req(requester.lock());
if(req)
req->getDone(pvd::Status(), shared_from_this(), pvf, changed);
} }
PDBSingleMonitor::PDBSingleMonitor(const PDBSinglePV::shared_pointer& pv, PDBSingleMonitor::PDBSingleMonitor(const PDBSinglePV::shared_pointer& pv,
@ -276,54 +282,55 @@ PDBSingleMonitor::PDBSingleMonitor(const PDBSinglePV::shared_pointer& pv,
void PDBSingleMonitor::destroy() void PDBSingleMonitor::destroy()
{ {
BaseMonitor::destroy(); BaseMonitor::destroy();
PDBSinglePV::shared_pointer pv;
{
Guard G(lock);
this->pv.swap(pv);
}
} }
void PDBSingleMonitor::onStart() void PDBSingleMonitor::onStart()
{ {
guard_t G(pv->lock); PDBSinglePV::shared_pointer PV(pv.lock());
if(!PV) return;
guard_t G(PV->lock);
pv->scratch.clear(); PV->scratch.clear();
pv->scratch.set(0); PV->scratch.set(0);
if(pv->interested.empty()) { if(PV->interested.empty()) {
// first subscriber // first subscriber
pv->hadevent_VALUE = false; PV->hadevent_VALUE = false;
pv->hadevent_PROPERTY = false; PV->hadevent_PROPERTY = false;
db_event_enable(pv->evt_VALUE.subscript); db_event_enable(PV->evt_VALUE.subscript);
db_event_enable(pv->evt_PROPERTY.subscript); db_event_enable(PV->evt_PROPERTY.subscript);
db_post_single_event(pv->evt_VALUE.subscript); db_post_single_event(PV->evt_VALUE.subscript);
db_post_single_event(pv->evt_PROPERTY.subscript); db_post_single_event(PV->evt_PROPERTY.subscript);
} else if(pv->hadevent_VALUE && pv->hadevent_PROPERTY) { } else if(PV->hadevent_VALUE && PV->hadevent_PROPERTY) {
// new subscriber and already had initial update // new subscriber and already had initial update
post(); post();
} // else new subscriber, but no initial update. so just wait } // else new subscriber, but no initial update. so just wait
shared_pointer self(std::tr1::static_pointer_cast<PDBSingleMonitor>(shared_from_this())); shared_pointer self(std::tr1::static_pointer_cast<PDBSingleMonitor>(shared_from_this()));
pv->interested.insert(self); PV->interested.insert(self);
} }
void PDBSingleMonitor::onStop() void PDBSingleMonitor::onStop()
{ {
guard_t G(pv->lock); PDBSinglePV::shared_pointer PV(pv.lock());
if(PV) return;
guard_t G(PV->lock);
shared_pointer self(std::tr1::static_pointer_cast<PDBSingleMonitor>(shared_from_this())); shared_pointer self(std::tr1::static_pointer_cast<PDBSingleMonitor>(shared_from_this()));
if(pv->interested.erase(self)==0) { if(PV->interested.erase(self)==0) {
fprintf(stderr, "%s: oops\n", __FUNCTION__); fprintf(stderr, "%s: oops\n", __FUNCTION__);
} }
if(pv->interested.empty()) { if(PV->interested.empty()) {
// last subscriber // last subscriber
db_event_disable(pv->evt_VALUE.subscript); db_event_disable(PV->evt_VALUE.subscript);
db_event_disable(pv->evt_PROPERTY.subscript); db_event_disable(PV->evt_PROPERTY.subscript);
} }
} }
void PDBSingleMonitor::requestUpdate() void PDBSingleMonitor::requestUpdate()
{ {
Guard G(pv->lock); PDBSinglePV::shared_pointer PV(pv.lock());
if(PV) return;
guard_t G(PV->lock);
post(); post();
} }

View File

@ -69,15 +69,15 @@ struct PDBSingleChannel : public BaseChannel,
virtual epics::pvAccess::ChannelGet::shared_pointer createChannelGet( virtual epics::pvAccess::ChannelGet::shared_pointer createChannelGet(
epics::pvAccess::ChannelGetRequester::shared_pointer const & channelGetRequester, epics::pvAccess::ChannelGetRequester::shared_pointer const & channelGetRequester,
epics::pvData::PVStructure::shared_pointer const & pvRequest); epics::pvData::PVStructure::shared_pointer const & pvRequest) OVERRIDE FINAL;
virtual epics::pvAccess::ChannelPut::shared_pointer createChannelPut( virtual epics::pvAccess::ChannelPut::shared_pointer createChannelPut(
epics::pvAccess::ChannelPutRequester::shared_pointer const & requester, epics::pvAccess::ChannelPutRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest); epics::pvData::PVStructure::shared_pointer const & pvRequest) OVERRIDE FINAL;
virtual epics::pvData::Monitor::shared_pointer createMonitor( virtual epics::pvData::Monitor::shared_pointer createMonitor(
epics::pvData::MonitorRequester::shared_pointer const & requester, epics::pvData::MonitorRequester::shared_pointer const & requester,
epics::pvData::PVStructure::shared_pointer const & pvRequest); epics::pvData::PVStructure::shared_pointer const & pvRequest) OVERRIDE FINAL;
virtual void printInfo(std::ostream& out); virtual void printInfo(std::ostream& out) OVERRIDE FINAL;
}; };
struct PDBSingleGet : public epics::pvAccess::ChannelGet, struct PDBSingleGet : public epics::pvAccess::ChannelGet,
@ -85,7 +85,7 @@ struct PDBSingleGet : public epics::pvAccess::ChannelGet,
{ {
typedef epics::pvAccess::ChannelGetRequester requester_t; typedef epics::pvAccess::ChannelGetRequester requester_t;
PDBSingleChannel::shared_pointer channel; PDBSingleChannel::shared_pointer channel;
requester_t::shared_pointer requester; requester_t::weak_pointer requester;
epics::pvData::BitSetPtr changed; epics::pvData::BitSetPtr changed;
epics::pvData::PVStructurePtr pvf; epics::pvData::PVStructurePtr pvf;
@ -110,7 +110,7 @@ struct PDBSinglePut : public epics::pvAccess::ChannelPut,
{ {
typedef epics::pvAccess::ChannelPutRequester requester_t; typedef epics::pvAccess::ChannelPutRequester requester_t;
PDBSingleChannel::shared_pointer channel; PDBSingleChannel::shared_pointer channel;
requester_t::shared_pointer requester; requester_t::weak_pointer requester;
epics::pvData::BitSetPtr changed; epics::pvData::BitSetPtr changed;
epics::pvData::PVStructurePtr pvf; epics::pvData::PVStructurePtr pvf;
@ -141,7 +141,7 @@ struct PDBSingleMonitor : public BaseMonitor
{ {
POINTER_DEFINITIONS(PDBSingleMonitor); POINTER_DEFINITIONS(PDBSingleMonitor);
PDBSinglePV::shared_pointer pv; PDBSinglePV::weak_pointer pv;
PDBSingleMonitor(const PDBSinglePV::shared_pointer& pv, PDBSingleMonitor(const PDBSinglePV::shared_pointer& pv,
const requester_t::shared_pointer& requester, const requester_t::shared_pointer& requester,

View File

@ -40,7 +40,7 @@ int main(int argc, char *argv[])
pva::ClientFactory::start(); pva::ClientFactory::start();
pva::ChannelProvider::shared_pointer prov(pva::getChannelProviderRegistry()->getProvider("pva")); pva::ChannelProvider::shared_pointer prov(pva::ChannelProviderRegistry::clients()->getProvider("pva"));
if(!prov) throw std::runtime_error("No pva provider?!?"); if(!prov) throw std::runtime_error("No pva provider?!?");
Destroyer prov_d(prov); Destroyer prov_d(prov);