beremiz: svghmi/svghmi.c@95e0b926a8c3 (annotated)

2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	1	#include <pthread.h>
2776 246ae685ab65 SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2775 diff changeset	2	#include <errno.h>
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	3	#include "iec_types_all.h"
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	4	#include "POUS.h"
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	5	#include "config.h"
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	6	#include "beremiz.h"
2750 2694170cd88e intermediate commit, work in progress Edouard Tisserant parents: diff changeset	7
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	8	#define DEFAULT_REFRESH_PERIOD_MS 100
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	9	#define HMI_BUFFER_SIZE %(buffer_size)d
2776 246ae685ab65 SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2775 diff changeset	10	#define HMI_ITEM_COUNT %(item_count)d
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	11	#define HMI_HASH_SIZE 8
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	12	#define MAX_CONNECTIONS %(max_connections)d
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	13	#define MAX_CON_INDEX MAX_CONNECTIONS - 1
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	14
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	15	static uint8_t hmi_hash[HMI_HASH_SIZE] = {%(hmi_hash_ints)s};
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	16
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	17	/* PLC reads from that buffer */
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	18	static char rbuf[HMI_BUFFER_SIZE];
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	19
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	20	/* PLC writes to that buffer */
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	21	static char wbuf[HMI_BUFFER_SIZE];
887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	22
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	23	/* worst biggest send buffer. FIXME : use dynamic alloc ? */
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	24	static char sbuf[HMI_HASH_SIZE + HMI_BUFFER_SIZE + (HMI_ITEM_COUNT * sizeof(uint32_t))];
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	25	static unsigned int sbufidx;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	26
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	27	%(extern_variables_declarations)s
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	28
2771 361366b891ca WIP on svghmi, now builds and runs. HTTP serving + WS transport ready, missing actual data to transmit and thread to collect it. Edouard Tisserant parents: 2768 diff changeset	29	#define ticktime_ns %(PLC_ticktime)d
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	30	static uint16_t ticktime_ms = (ticktime_ns>1000000)?
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	31	ticktime_ns/1000000:
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	32	1;
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	33
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	34	typedef enum {
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	35	buf_free = 0,
2805 e521e0d133d5 SVGHMI: fixed HMI->PLC dataflow : not updates as expected, and not initialized properly after subscribe. Edouard Tisserant parents: 2802 diff changeset	36	buf_new,
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	37	buf_set,
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	38	buf_tosend
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	39	} buf_state_t;
31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	40
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	41	static int global_write_dirty = 0;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	42	static long hmitree_rlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	43	static long hmitree_wlock = 0;
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	44
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	45	typedef struct hmi_tree_item_s hmi_tree_item_t;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	46	struct hmi_tree_item_s{
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	47	void *ptr;
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	48	__IEC_types_enum type;
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	49	uint32_t buf_index;
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	50
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	51	/* retrieve/read/recv */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	52	buf_state_t rstate;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	53
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	54	/* publish/write/send */
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	55	buf_state_t wstate[MAX_CONNECTIONS];
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	56
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	57	/* zero means not subscribed */
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	58	uint16_t refresh_period_ms[MAX_CONNECTIONS];
38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	59	uint16_t age_ms[MAX_CONNECTIONS];
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	60
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	61	/* dual linked list for subscriptions */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	62	hmi_tree_item_t *subscriptions_next;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	63	hmi_tree_item_t *subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	64
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	65	/* single linked list for changes from HMI */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	66	hmi_tree_item_t *incoming_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	67
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	68	};
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	69
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	70	#define HMITREE_ITEM_INITIALIZER(cpath,type,buf_index) { \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	71	&(cpath), /ptr/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	72	type, /type/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	73	buf_index, /buf_index/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	74	buf_free, /rstate/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	75	{[0 ... MAX_CON_INDEX] = buf_free}, /wstate/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	76	{[0 ... MAX_CON_INDEX] = 0}, /refresh_period_ms/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	77	{[0 ... MAX_CON_INDEX] = 0}, /age_ms/ \
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	78	NULL, /subscriptions_next/\
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	79	NULL, /subscriptions_prev/\
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	80	NULL} /incoming_next/
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	81
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	82
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	83	/* entry for dual linked list for HMI subscriptions */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	84	/* points to the end of the list */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	85	static hmi_tree_item_t *subscriptions_tail = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	86
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	87	/* entry for single linked list for changes from HMI */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	88	/* points to the end of the list */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	89	static hmi_tree_item_t *incoming_tail = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	90
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	91	static hmi_tree_item_t hmi_tree_items[] = {
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	92	%(variable_decl_array)s
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	93	};
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	94
2767 302347f48193 svghmi.c : deduplicated variable access code borrowed from plc_debug.c. Added targets/var_access.c. Edouard Tisserant parents: 2766 diff changeset	95	#define __Unpack_desc_type hmi_tree_item_t
2766 3f3b1b8ccba4 SVGHMI: Added iterators in svghmi.c copy-pasted form plc_debug.c Edouard Tisserant parents: 2765 diff changeset	96
2767 302347f48193 svghmi.c : deduplicated variable access code borrowed from plc_debug.c. Added targets/var_access.c. Edouard Tisserant parents: 2766 diff changeset	97	%(var_access_code)s
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	98
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	99	static int write_iterator(hmi_tree_item_t *dsc)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	100	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	101	{
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	102	uint32_t session_index = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	103	int value_changed = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	104	void *dest_p = NULL;
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	105	void *value_p = NULL;
75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	106	size_t sz = 0;
3273 08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	107	while(session_index < MAX_CONNECTIONS) {
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	108	if(dsc->wstate[session_index] == buf_set){
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	109	/* if being subscribed */
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	110	if(dsc->refresh_period_ms[session_index]){
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	111	if(dsc->age_ms[session_index] + ticktime_ms < dsc->refresh_period_ms[session_index]){
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	112	dsc->age_ms[session_index] += ticktime_ms;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	113	}else{
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	114	dsc->wstate[session_index] = buf_tosend;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	115	global_write_dirty = 1;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	116	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	117	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	118	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	119
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	120	/* variable is sample only if just subscribed
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	121	or already subscribed and having value change */
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	122	int do_sample = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	123	int just_subscribed = dsc->wstate[session_index] == buf_new;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	124	if(!just_subscribed){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	125	int already_subscribed = dsc->refresh_period_ms[session_index] > 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	126	if(already_subscribed){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	127	if(!value_changed){
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	128	if(!value_p){
75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	129	UnpackVar(dsc, &value_p, NULL, &sz);
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	130	if(__Is_a_string(dsc)){
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	131	sz = ((STRING*)value_p)->len + 1;
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	132	}
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	133	dest_p = &wbuf[dsc->buf_index];
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	134	}
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	135	value_changed = memcmp(dest_p, value_p, sz) != 0;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	136	do_sample = value_changed;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	137	}else{
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	138	do_sample = 1;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	139	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	140	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	141	} else {
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	142	do_sample = 1;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	143	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	144
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	145
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	146	if(do_sample){
3273 08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	147	if(dsc->wstate[session_index] != buf_set && dsc->wstate[session_index] != buf_tosend) {
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	148	if(dsc->wstate[session_index] == buf_new \
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	149	\|\| ticktime_ms > dsc->refresh_period_ms[session_index]){
3273 08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	150	dsc->wstate[session_index] = buf_tosend;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	151	global_write_dirty = 1;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	152	} else {
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	153	dsc->wstate[session_index] = buf_set;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	154	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	155	dsc->age_ms[session_index] = 0;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	156	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	157	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	158
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	159	session_index++;
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	160	}
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	161	/* copy value if changed (and subscribed) */
08a5a019bed2 SVGHMI: finished multiclient support. Still needs more testing. Edouard Tisserant parents: 3271 diff changeset	162	if(value_changed)
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	163	memcpy(dest_p, value_p, sz);
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	164	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	165	// else ... : PLC can't wait, variable will be updated next turn
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	166	return 0;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	167	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	168
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	169	static int send_iterator(uint32_t index, hmi_tree_item_t *dsc, uint32_t session_index)
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	170	{
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	171	if(dsc->wstate[session_index] == buf_tosend)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	172	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	173	uint32_t sz = __get_type_enum_size(dsc->type);
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	174	if(sbufidx + sizeof(uint32_t) + sz <= sizeof(sbuf))
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	175	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	176	void *src_p = &wbuf[dsc->buf_index];
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	177	void *dst_p = &sbuf[sbufidx];
2826 1e5abecc3cde SVGHMI : support for HMI_STRING and HMI_BOOL Edouard Tisserant <edouard.tisserant@gmail.com> parents: 2822 diff changeset	178	if(__Is_a_string(dsc)){
1e5abecc3cde SVGHMI : support for HMI_STRING and HMI_BOOL Edouard Tisserant <edouard.tisserant@gmail.com> parents: 2822 diff changeset	179	sz = ((STRING*)src_p)->len + 1;
1e5abecc3cde SVGHMI : support for HMI_STRING and HMI_BOOL Edouard Tisserant <edouard.tisserant@gmail.com> parents: 2822 diff changeset	180	}
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	181	/* TODO : force into little endian */
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	182	memcpy(dst_p, &index, sizeof(uint32_t));
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	183	memcpy(dst_p + sizeof(uint32_t), src_p, sz);
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	184	dsc->wstate[session_index] = buf_free;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	185	sbufidx += sizeof(uint32_t) /* index */ + sz;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	186	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	187	else
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	188	{
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	189	printf("BUG!!! %%d + %%ld + %%d > %%ld \n", sbufidx, sizeof(uint32_t), sz, sizeof(sbuf));
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	190	return EOVERFLOW;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	191	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	192	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	193
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	194	return 0;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	195	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	196
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	197	static int read_iterator(hmi_tree_item_t *dsc)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	198	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	199	if(dsc->rstate == buf_set)
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	200	{
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	201	void *src_p = &rbuf[dsc->buf_index];
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	202	void *value_p = NULL;
75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	203	size_t sz = 0;
75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	204	UnpackVar(dsc, &value_p, NULL, &sz);
75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	205	memcpy(value_p, src_p, sz);
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	206	dsc->rstate = buf_free;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	207	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	208	return 0;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	209	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	210
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	211	void update_refresh_period(hmi_tree_item_t *dsc, uint32_t session_index, uint16_t refresh_period_ms)
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	212	{
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	213	uint32_t other_session_index = 0;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	214	int previously_subscribed = 0;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	215	int session_only_subscriber = 0;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	216	int session_subscriber = 0;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	217	int needs_subscription_for_session = (refresh_period_ms != 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	218
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	219	while(other_session_index < session_index) {
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	220	previously_subscribed \|= (dsc->refresh_period_ms[other_session_index++] != 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	221	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	222	session_subscriber = (dsc->refresh_period_ms[other_session_index++] != 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	223	while(other_session_index < MAX_CONNECTIONS) {
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	224	previously_subscribed \|= (dsc->refresh_period_ms[other_session_index++] != 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	225	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	226	session_only_subscriber = session_subscriber && !previously_subscribed;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	227	previously_subscribed \|= session_subscriber;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	228
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	229	printf("update_refresh_period %%x,%%x index:%%d session_index:%%d refresh_period_ms:%%d\n",
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	230	dsc,
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	231	hmi_tree_items,
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	232	dsc - hmi_tree_items,
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	233	session_index,
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	234	refresh_period_ms);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	235
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	236	if(needs_subscription_for_session) {
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	237	if(!session_subscriber)
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	238	{
38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	239	dsc->wstate[session_index] = buf_new;
2863 e120d6985a2f SVGHMI: fix wrong updates of HMI variables on each change, especially when not subscribed. Edouard Tisserant parents: 2862 diff changeset	240	}
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	241	/* if not already subscribed */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	242	if(!previously_subscribed){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	243	/* append subsciption to list */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	244	dsc->subscriptions_prev = subscriptions_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	245	subscriptions_tail = dsc;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	246	dsc->subscriptions_next = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	247	}
2805 e521e0d133d5 SVGHMI: fixed HMI->PLC dataflow : not updates as expected, and not initialized properly after subscribe. Edouard Tisserant parents: 2802 diff changeset	248	} else {
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	249	dsc->wstate[session_index] = buf_free;
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	250	/* item is removed from list only when session was the only one remaining */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	251	if (session_only_subscriber) {
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	252	if(dsc->subscriptions_next == NULL){ /* remove tail */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	253	/* re-link tail to previous */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	254	subscriptions_tail = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	255	} else { /* remove entry in between other entries */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	256	/* re-link previous iand next node */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	257	dsc->subscriptions_next->subscriptions_prev = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	258	dsc->subscriptions_prev->subscriptions_next = dsc->subscriptions_next;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	259	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	260	/* unnecessary
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	261	dsc->subscriptions_next = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	262	dsc->subscriptions_prev = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	263	*/
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	264	}
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	265	}
38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	266	dsc->refresh_period_ms[session_index] = refresh_period_ms;
2766 3f3b1b8ccba4 SVGHMI: Added iterators in svghmi.c copy-pasted form plc_debug.c Edouard Tisserant parents: 2765 diff changeset	267	}
3f3b1b8ccba4 SVGHMI: Added iterators in svghmi.c copy-pasted form plc_debug.c Edouard Tisserant parents: 2765 diff changeset	268
3294 e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	269	static void *svghmi_handle;
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	270
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	271	void SVGHMI_SuspendFromPythonThread(void)
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	272	{
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	273	wait_RT_to_nRT_signal(svghmi_handle);
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	274	}
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	275
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	276	void SVGHMI_WakeupFromRTThread(void)
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	277	{
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	278	unblock_RT_to_nRT_signal(svghmi_handle);
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	279	}
2775 3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	280
3281 1fc4274de64e SVGHMI: Fixed halting problem when there is no session opened. Edouard Tisserant parents: 3273 diff changeset	281	int svghmi_continue_collect;
2776 246ae685ab65 SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2775 diff changeset	282
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	283	int __init_svghmi()
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	284	{
3295 0375d801fff7 Runtime+SVGHMI: Add generic wakeup of threads from PLC thread to windows implementation of plc_main.c. Also added nRT_reschedule to abstract sched_yield. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3294 diff changeset	285	memset(rbuf,0,sizeof(rbuf));
0375d801fff7 Runtime+SVGHMI: Add generic wakeup of threads from PLC thread to windows implementation of plc_main.c. Also added nRT_reschedule to abstract sched_yield. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3294 diff changeset	286	memset(wbuf,0,sizeof(wbuf));
2819 3b99c908f43b SVGHMI: change collect/send thread looping condition to fix infinite loop in some cases Edouard Tisserant parents: 2817 diff changeset	287
3281 1fc4274de64e SVGHMI: Fixed halting problem when there is no session opened. Edouard Tisserant parents: 3273 diff changeset	288	svghmi_continue_collect = 1;
2765 887aba5ef178 SVGHMI: svghmi.c now has mutex, iterator, and read/write buffer. Edouard Tisserant parents: 2764 diff changeset	289
3294 e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	290	/* create svghmi_pipe */
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	291	svghmi_handle = create_RT_to_nRT_signal("SVGHMI_pipe");
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	292
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	293	if(!svghmi_handle)
3294 e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	294	return 1;
e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	295
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	296	return 0;
2750 2694170cd88e intermediate commit, work in progress Edouard Tisserant parents: diff changeset	297	}
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	298
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	299	void __cleanup_svghmi()
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	300	{
3281 1fc4274de64e SVGHMI: Fixed halting problem when there is no session opened. Edouard Tisserant parents: 3273 diff changeset	301	svghmi_continue_collect = 0;
2820 d9b5303d43dc SVGHMI : had to move the problem of wkaing up python thread from plc thread to platform specific code. Edouard Tisserant parents: 2819 diff changeset	302	SVGHMI_WakeupFromRTThread();
3294 e3db472b0dfb Runtime+SVGHMI: Added a generic way to wakeup non-real-time threads from real-time PLC thread. Replace SVGHMI specific calls in Linux and Xenomai implementations of plc_main.c. Fixed xenomai build, xeno-config making problems with --no-auto-init argument. Edouard Tisserant <edouard.tisserant@gmail.com> parents: 3281 diff changeset	303	delete_RT_to_nRT_signal(svghmi_handle);
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	304	}
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	305
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	306	void __retrieve_svghmi()
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	307	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	308	if(AtomicCompareExchange(&hmitree_rlock, 0, 1) == 0) {
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	309	hmi_tree_item_t *dsc = incoming_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	310	/* iterate through read list (changes from HMI) */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	311	while(dsc){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	312	read_iterator(dsc);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	313	dsc = dsc->incoming_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	314	/* unnecessary
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	315	dsc->incoming_prev = NULL;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	316	*/
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	317	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	318	/* flush read list */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	319	incoming_tail = NULL;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	320	AtomicCompareExchange(&hmitree_rlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	321	}
2764 b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	322	}
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	323
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	324	void __publish_svghmi()
b75cc2cf4e50 SVGHMI: draft for svghmi.c. It has all PLC variables pointed in HMI tree in an array. Edouard Tisserant parents: 2754 diff changeset	325	{
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	326	global_write_dirty = 0;
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	327
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	328	if(AtomicCompareExchange(&hmitree_wlock, 0, 1) == 0) {
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	329	hmi_tree_item_t *dsc = subscriptions_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	330	while(dsc){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	331	write_iterator(dsc);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	332	dsc = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	333	}
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	334	AtomicCompareExchange(&hmitree_wlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	335	}
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	336
2768 31785529a657 SVGHMI: Intermediate state while working on svghmi.c Edouard Tisserant parents: 2767 diff changeset	337	if(global_write_dirty) {
2820 d9b5303d43dc SVGHMI : had to move the problem of wkaing up python thread from plc thread to platform specific code. Edouard Tisserant parents: 2819 diff changeset	338	SVGHMI_WakeupFromRTThread();
2775 3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	339	}
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	340	}
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	341
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	342	/* PYTHON CALLS */
3271 561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	343	int svghmi_wait(void){
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	344
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	345	SVGHMI_SuspendFromPythonThread();
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	346	}
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	347
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	348	int svghmi_send_collect(uint32_t session_index, uint32_t size, char *ptr){
2775 3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	349
3281 1fc4274de64e SVGHMI: Fixed halting problem when there is no session opened. Edouard Tisserant parents: 3273 diff changeset	350	if(svghmi_continue_collect) {
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	351	int res;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	352	sbufidx = HMI_HASH_SIZE;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	353
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	354	while(AtomicCompareExchange(&hmitree_wlock, 0, 1)){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	355	nRT_reschedule();
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	356	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	357
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	358	hmi_tree_item_t *dsc = subscriptions_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	359	while(dsc){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	360	uint32_t index = dsc - hmi_tree_items;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	361	printf("Send index %%d\n", index);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	362	res = send_iterator(index, dsc, session_index);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	363	if(res != 0){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	364	break;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	365	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	366	dsc = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	367	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	368	if(res == 0)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	369	{
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	370	if(sbufidx > HMI_HASH_SIZE){
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	371	memcpy(&sbuf[0], &hmi_hash[0], HMI_HASH_SIZE);
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	372	*ptr = &sbuf[0];
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	373	*size = sbufidx;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	374	AtomicCompareExchange(&hmitree_wlock, 1, 0);
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	375	return 0;
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	376	}
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	377	AtomicCompareExchange(&hmitree_wlock, 1, 0);
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	378	return ENODATA;
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	379	}
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	380	// printf("collected BAD result %%d\n", res);
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	381	AtomicCompareExchange(&hmitree_wlock, 1, 0);
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	382	return res;
2775 3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	383	}
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	384	else
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	385	{
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	386	return EINTR;
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	387	}
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	388	}
3b93409ba22c SVGHMI: WIP for python<->C data exchange Edouard Tisserant parents: 2771 diff changeset	389
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	390	typedef enum {
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	391	unset = -1,
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	392	setval = 0,
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	393	reset = 1,
2798 ddb2c4668a6b SVGHMI : many details about communication implemented in JS, with side effects. Edouard Tisserant parents: 2789 diff changeset	394	subscribe = 2
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	395	} cmd_from_JS;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	396
3271 561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	397	int svghmi_reset(uint32_t session_index){
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	398	hmi_tree_item_t *dsc = subscriptions_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	399	while(dsc){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	400	update_refresh_period(dsc, session_index, 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	401	dsc = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	402	}
3271 561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	403	return 1;
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	404	}
561dbd1e3e04 SVGHMI: Fixing last commit's multiclient implementation, in case of watchdog. To be continued, since multiclient still fail... Edouard Tisserant parents: 3270 diff changeset	405
2822 9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	406	// Returns :
9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	407	// 0 is OK, <0 is error, 1 is heartbeat
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	408	int svghmi_recv_dispatch(uint32_t session_index, uint32_t size, const uint8_t *ptr){
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	409	const uint8_t* cursor = ptr + HMI_HASH_SIZE;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	410	const uint8_t* end = ptr + size;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	411
2822 9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	412	int was_hearbeat = 0;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	413
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	414	/* match hmitree fingerprint */
2789 ba0dd2ec6dc4 SVGHMI: now built. Edouard Tisserant parents: 2788 diff changeset	415	if(size <= HMI_HASH_SIZE \|\| memcmp(ptr, hmi_hash, HMI_HASH_SIZE) != 0)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	416	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	417	printf("svghmi_recv_dispatch MISMATCH !!\n");
2822 9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	418	return -EINVAL;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	419	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	420
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	421	int ret;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	422	int got_wlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	423	int got_rlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	424	cmd_from_JS cmd_old = unset;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	425	cmd_from_JS cmd = unset;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	426
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	427	while(cursor < end)
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	428	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	429	uint32_t progress;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	430
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	431	cmd_old = cmd;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	432	cmd = *(cursor++);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	433
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	434	if(cmd_old != cmd){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	435	if(got_wlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	436	AtomicCompareExchange(&hmitree_wlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	437	got_wlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	438	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	439	if(got_rlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	440	AtomicCompareExchange(&hmitree_rlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	441	got_rlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	442	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	443	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	444	switch(cmd)
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	445	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	446	case setval:
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	447	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	448	uint32_t index = (uint32_t)(cursor);
2789 ba0dd2ec6dc4 SVGHMI: now built. Edouard Tisserant parents: 2788 diff changeset	449	uint8_t const *valptr = cursor + sizeof(uint32_t);
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	450
2822 9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	451	if(index == heartbeat_index)
9101a72a1da0 SVGHMI: added a watchdog. To ensure that the whole chain is checked, watchdog use a periodic echo of a hearteat variable. JS client code systematically register /HEARTBEAT at 1s update freq, and reacts on updates of /HEARTBEAT by systematically incrementing it. C code catch /HEARTBEAT update and feeds python-implemented watchdog. For now, watchdog does nothing when tiggered Edouard Tisserant parents: 2820 diff changeset	452	was_hearbeat = 1;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	453
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	454	if(index < HMI_ITEM_COUNT)
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	455	{
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	456	hmi_tree_item_t *dsc = &hmi_tree_items[index];
3397 75920c99ffc9 SVGHMI: Adapt svghmi.c to changes in UnpackVar Edouard Tisserant parents: 3374 diff changeset	457	size_t sz = 0;
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	458	void *dst_p = &rbuf[dsc->buf_index];
2829 4c2c50f60730 SVGHMI : HMI_STRING now also supported from HMI to PLC Edouard Tisserant parents: 2828 diff changeset	459
4c2c50f60730 SVGHMI : HMI_STRING now also supported from HMI to PLC Edouard Tisserant parents: 2828 diff changeset	460	if(__Is_a_string(dsc)){
4c2c50f60730 SVGHMI : HMI_STRING now also supported from HMI to PLC Edouard Tisserant parents: 2828 diff changeset	461	sz = ((STRING*)valptr)->len + 1;
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	462	} else {
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	463	UnpackVar(dsc, NULL, NULL, &sz);
2829 4c2c50f60730 SVGHMI : HMI_STRING now also supported from HMI to PLC Edouard Tisserant parents: 2828 diff changeset	464	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	465
2802 64e6f73b9859 SVGHMI - Fixed svghmi.{c,js} about HMI -> PLC data unpack. Edouard Tisserant parents: 2799 diff changeset	466	if((valptr + sz) <= end)
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	467	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	468	// rescheduling spinlock until free
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	469	if(!got_rlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	470	while(AtomicCompareExchange(&hmitree_rlock, 0, 1)){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	471	nRT_reschedule();
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	472	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	473	got_rlock=1;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	474	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	475
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	476	memcpy(dst_p, valptr, sz);
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	477
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	478	/* check that rstate is not already buf_set */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	479	if(dsc->rstate != buf_set){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	480	dsc->rstate = buf_set;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	481	/* append entry to read list (changes from HMI) */
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	482	dsc->incoming_prev = incoming_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	483	incoming_tail = dsc;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	484	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	485
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	486	progress = sz + sizeof(uint32_t) /* index */;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	487	}
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	488	else
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	489	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	490	ret = -EINVAL;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	491	goto exit_free;
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	492	}
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	493	}
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	494	else
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	495	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	496	ret = -EINVAL;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	497	goto exit_free;
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	498	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	499	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	500	break;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	501
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	502	case reset:
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	503	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	504	progress = 0;
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	505	if(!got_wlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	506	while(AtomicCompareExchange(&hmitree_wlock, 0, 1)){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	507	nRT_reschedule();
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	508	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	509	got_wlock = 1;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	510	}
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	511	{
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	512	hmi_tree_item_t *dsc = subscriptions_tail;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	513	while(dsc){
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	514	update_refresh_period(dsc, session_index, 0);
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	515	dsc = dsc->subscriptions_prev;
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	516	}
95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	517	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	518	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	519	break;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	520
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	521	case subscribe:
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	522	{
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	523	uint32_t index = (uint32_t)(cursor);
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	524	uint16_t refresh_period_ms = (uint32_t)(cursor + sizeof(uint32_t));
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	525
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	526	if(index < HMI_ITEM_COUNT)
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	527	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	528	if(!got_wlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	529	while(AtomicCompareExchange(&hmitree_wlock, 0, 1)){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	530	nRT_reschedule();
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	531	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	532	got_wlock = 1;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	533	}
3399 95e0b926a8c3 SVGHMI: optimization of C part : stop traversing the whole HMI tree, use dual linked list for subscriptions and single linked list for changes from HMI. Intermediate commit, still crashing in some cases. Edouard Tisserant parents: 3397 diff changeset	534	hmi_tree_item_t *dsc = &hmi_tree_items[index];
3270 38f7122ccbf9 SVGHMI: Implemented multiserver+multiclient, but only tested with single client and single server for now. To be continued... Edouard Tisserant parents: 2863 diff changeset	535	update_refresh_period(dsc, session_index, refresh_period_ms);
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	536	}
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	537	else
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	538	{
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	539	ret = -EINVAL;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	540	goto exit_free;
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	541	}
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	542
2862 a108677bd3d0 SVGHMI: whitespaces Edouard Tisserant parents: 2829 diff changeset	543	progress = sizeof(uint32_t) /* index */ +
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	544	sizeof(uint16_t) /* refresh period */;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	545	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	546	break;
2799 f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	547	default:
f5da343b9b63 SVGHMI: Many fixes. Subscriptions to HMItree seems to be working, and dispatch function is called in JS with good data. Bidirectional communication now really working. Edouard Tisserant parents: 2798 diff changeset	548	printf("svghmi_recv_dispatch unknown %%d\n",cmd);
2788 2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	549
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	550	}
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	551	cursor += progress;
2ed9ff826d03 SVGHMI: Work in progress. C side mostly implemented, neither built nor tested. Edouard Tisserant parents: 2779 diff changeset	552	}
3374 9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	553	ret = was_hearbeat;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	554
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	555	exit_free:
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	556	if(got_wlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	557	AtomicCompareExchange(&hmitree_wlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	558	got_wlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	559	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	560	if(got_rlock){
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	561	AtomicCompareExchange(&hmitree_rlock, 1, 0);
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	562	got_rlock = 0;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	563	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	564	return ret;
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	565	}
9a82918e063c SVGHMI: optimize HMI tree handling C code to lower CPU usage when traversing large trees Edouard Tisserant parents: 3295 diff changeset	566

author	Edouard Tisserant
	Thu, 23 Dec 2021 11:36:37 +0100
branch	RuntimeLists
changeset 3399	95e0b926a8c3
parent 3397	75920c99ffc9
child 3400	c2b46d0965ca
permissions	-rw-r--r--