Skip to content

Conversation

@simon-ess
Copy link
Contributor

Closes #57 .

@simon-ess simon-ess self-assigned this Jan 31, 2025
DB += siteEnvVars.db
DB += epicsEnvVars.db
DB += epicsCAEnvVars.db
ifdef BASE_7_0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ifdef BASE_7_0 isn't a good way to detect whether PVA has been included. There are many different ways to build EPICS with PVA, it is possible to build it with 3.15 and maybe still with 3.14.12.5, although I wouldn't recommend that. Sites can also disable building the PVA modules in Base 7.0.x by editing modules/Makefile — unusual, but legitimate IMHO.

I would try using one of the variables defined in the installed cfg/CONFIG_PVACCESS_*, cfg/CONFIG_PVXS_* or cfg/CONFIG_QSRV* files.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does handling of the environment variables really benefit from being build-time conditional? Why not just add these to epicsEnvVars.db? Which would also avoid some annoyance to dependent modules caused by renaming an installed .db file.

@anjohnson
Copy link
Member

Not required for these particular environment variable changes, but note the new dbServerStats() routine I just added in epics-base/epics-base#592 which will give access to the RSRV and QSRV2 client and channel counts.

@simon-ess simon-ess changed the title Draft: Add pvaccess support to IOC stats Add pvaccess support to IOC stats Nov 25, 2025
@simon-ess simon-ess merged commit 078205e into master Dec 2, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

IOC Stats should support PVAccess

4 participants