srd: add more debugging around probe mapping
[libsigrokdecode.git] / controller.c
1 /*
2  * This file is part of the sigrok project.
3  *
4  * Copyright (C) 2010 Uwe Hermann <uwe@hermann-uwe.de>
5  * Copyright (C) 2012 Bert Vermeulen <bert@biot.com>
6  *
7  * This program is free software: you can redistribute it and/or modify
8  * it under the terms of the GNU General Public License as published by
9  * the Free Software Foundation, either version 3 of the License, or
10  * (at your option) any later version.
11  *
12  * This program is distributed in the hope that it will be useful,
13  * but WITHOUT ANY WARRANTY; without even the implied warranty of
14  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
15  * GNU General Public License for more details.
16  *
17  * You should have received a copy of the GNU General Public License
18  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
19  */
20
21 #include "sigrokdecode.h" /* First, so we avoid a _POSIX_C_SOURCE warning. */
22 #include "sigrokdecode-internal.h"
23 #include "config.h"
24 #include <glib.h>
25 #include <inttypes.h>
26 #include <stdlib.h>
27
28
29 /* List of decoder instances. */
30 static GSList *di_list = NULL;
31
32 /* List of frontend callbacks to receive PD output. */
33 static GSList *callbacks = NULL;
34
35 /* lives in decoder.c */
36 extern GSList *pd_list;
37
38 /* lives in module_sigrokdecode.c */
39 extern PyMODINIT_FUNC PyInit_sigrokdecode(void);
40
41 /* lives in type_logic.c */
42 extern PyTypeObject srd_logic_type;
43
44
45 /**
46  * Initialize libsigrokdecode.
47  *
48  * This initializes the Python interpreter, and creates and initializes
49  * a "sigrok" Python module with a single put() method.
50  *
51  * Then, it searches for sigrok protocol decoder files (*.py) in the
52  * "decoders" subdirectory of the the sigrok installation directory.
53  * All decoders that are found are loaded into memory and added to an
54  * internal list of decoders, which can be queried via srd_list_decoders().
55  *
56  * The caller is responsible for calling the clean-up function srd_exit(),
57  * which will properly shut down libsigrokdecode and free its allocated memory.
58  *
59  * Multiple calls to srd_init(), without calling srd_exit() in between,
60  * are not allowed.
61  *
62  * @return SRD_OK upon success, a (negative) error code otherwise.
63  *         Upon Python errors, return SRD_ERR_PYTHON. If the sigrok decoders
64  *         directory cannot be accessed, return SRD_ERR_DECODERS_DIR.
65  *         If not enough memory could be allocated, return SRD_ERR_MALLOC.
66  */
67 int srd_init(void)
68 {
69         int ret;
70
71         srd_dbg("Initializing libsigrokdecode.");
72
73         /* Add our own module to the list of built-in modules. */
74         PyImport_AppendInittab("sigrokdecode", PyInit_sigrokdecode);
75
76         /* Initialize the python interpreter. */
77         Py_Initialize();
78
79         if ((ret = set_modulepath()) != SRD_OK) {
80                 Py_Finalize();
81                 return ret;
82         }
83
84         if ((ret = srd_load_all_decoders()) != SRD_OK) {
85                 Py_Finalize();
86                 return ret;
87         }
88
89         return SRD_OK;
90 }
91
92
93 /**
94  * Shutdown libsigrokdecode.
95  *
96  * This frees all the memory allocated for protocol decoders and shuts down
97  * the Python interpreter.
98  *
99  * This function should only be called if there was a (successful!) invocation
100  * of srd_init() before. Calling this function multiple times in a row, without
101  * any successful srd_init() calls in between, is not allowed.
102  *
103  * @return SRD_OK upon success, a (negative) error code otherwise.
104  */
105 int srd_exit(void)
106 {
107
108         srd_dbg("Exiting libsigrokdecode.");
109
110         srd_unload_all_decoders();
111         g_slist_free(pd_list);
112
113         /* Py_Finalize() returns void, any finalization errors are ignored. */
114         Py_Finalize();
115
116         return SRD_OK;
117 }
118
119
120 /**
121  * Add an additional search directory for the protocol decoders.
122  *
123  * The specified directory is prepended (not appended!) to Python's sys.path,
124  * in order to search for sigrok protocol decoders in the specified
125  * directories first, and in the generic Python module directories (and in
126  * the current working directory) last. This avoids conflicts if there are
127  * Python modules which have the same name as a sigrok protocol decoder in
128  * sys.path or in the current working directory.
129  *
130  * TODO: add path from env var SIGROKDECODE_PATH, config etc
131  * TODO: Should take directoryname/path as input.
132  *
133  * @return TODO.
134  */
135 int set_modulepath(void)
136 {
137         int ret;
138         gchar *path, *s;
139
140 #ifdef _WIN32
141         gchar **splitted;
142
143         /*
144          * On Windows/MinGW, Python's sys.path needs entries of the form
145          * 'C:\\foo\\bar' instead of '/foo/bar'.
146          */
147
148         splitted = g_strsplit(DECODERS_DIR, "/", 0);
149         path = g_build_pathv("\\\\", splitted);
150         g_strfreev(splitted);
151 #else
152         path = g_strdup(DECODERS_DIR);
153 #endif
154
155         /* TODO: Sanity check on 'path' (length, escape special chars, ...). */
156         s = g_strdup_printf("import sys; sys.path.insert(0, r'%s')", path);
157
158         ret = PyRun_SimpleString(s);
159
160         g_free(path);
161         g_free(s);
162
163         return ret;
164 }
165
166
167 /**
168  * Set options in a decoder instance.
169  *
170  * @param di Decoder instance.
171  * @param options A GHashTable of options to set.
172  *
173  * Handled options are removed from the hash.
174  *
175  * @return SRD_OK upon success, a (negative) error code otherwise.
176  */
177 int srd_instance_set_options(struct srd_decoder_instance *di,
178                 GHashTable *options)
179 {
180         PyObject *py_dec_options, *py_dec_optkeys, *py_di_options, *py_optval;
181         PyObject *py_optlist, *py_classval;
182         Py_UNICODE *py_ustr;
183         unsigned long long int val_ull;
184         int num_optkeys, ret, size, i;
185         char *key, *value;
186
187         if(!PyObject_HasAttrString(di->decoder->py_dec, "options")) {
188                 /* Decoder has no options. */
189                 if (g_hash_table_size(options) == 0) {
190                         /* No options provided. */
191                         return SRD_OK;
192                 } else {
193                         srd_err("Protocol decoder has no options.");
194                         return SRD_ERR_ARG;
195                 }
196                 return SRD_OK;
197         }
198
199         ret = SRD_ERR_PYTHON;
200         key = NULL;
201         py_dec_options = py_dec_optkeys = py_di_options = py_optval = NULL;
202         py_optlist = py_classval = NULL;
203         py_dec_options = PyObject_GetAttrString(di->decoder->py_dec, "options");
204
205         /* All of these are synthesized objects, so they're good. */
206         py_dec_optkeys = PyDict_Keys(py_dec_options);
207         num_optkeys = PyList_Size(py_dec_optkeys);
208         if (!(py_di_options = PyObject_GetAttrString(di->py_instance, "options")))
209                 goto err_out;
210         for (i = 0; i < num_optkeys; i++) {
211                 /* Get the default class value for this option. */
212                 py_str_as_str(PyList_GetItem(py_dec_optkeys, i), &key);
213                 if (!(py_optlist = PyDict_GetItemString(py_dec_options, key)))
214                         goto err_out;
215                 if (!(py_classval = PyList_GetItem(py_optlist, 1)))
216                         goto err_out;
217                 if (!PyUnicode_Check(py_classval) && !PyLong_Check(py_classval)) {
218                         srd_err("Options of type %s are not yet supported.", Py_TYPE(py_classval)->tp_name);
219                         goto err_out;
220                 }
221
222                 if ((value = g_hash_table_lookup(options, key))) {
223                         /* An override for this option was provided. */
224                         if (PyUnicode_Check(py_classval)) {
225                                 if (!(py_optval = PyUnicode_FromString(value))) {
226                                         /* Some UTF-8 encoding error. */
227                                         PyErr_Clear();
228                                         goto err_out;
229                                 }
230                         } else if (PyLong_Check(py_classval)) {
231                                 if (!(py_optval = PyLong_FromString(value, NULL, 0))) {
232                                         /* ValueError Exception */
233                                         PyErr_Clear();
234                                         srd_err("Option %s has invalid value %s: expected integer.",
235                                                         key, value);
236                                         goto err_out;
237                                 }
238                         }
239                         g_hash_table_remove(options, key);
240                 } else {
241                         /* Use the class default for this option. */
242                         if (PyUnicode_Check(py_classval)) {
243                                 /* Make a brand new copy of the string. */
244                                 py_ustr = PyUnicode_AS_UNICODE(py_classval);
245                                 size = PyUnicode_GET_SIZE(py_classval);
246                                 py_optval = PyUnicode_FromUnicode(py_ustr, size);
247                         } else if (PyLong_Check(py_classval)) {
248                                 /* Make a brand new copy of the integer. */
249                                 val_ull = PyLong_AsUnsignedLongLong(py_classval);
250                                 if (val_ull == (unsigned long long)-1) {
251                                         /* OverFlowError exception */
252                                         PyErr_Clear();
253                                         srd_err("Invalid integer value for %s: expected integer.", key);
254                                         goto err_out;
255                                 }
256                                 if (!(py_optval = PyLong_FromUnsignedLongLong(val_ull)))
257                                         goto err_out;
258                         }
259                 }
260
261                 /* If we got here, py_optval holds a known good new reference
262                  * to the instance option to set.
263                  */
264                 if (PyDict_SetItemString(py_di_options, key, py_optval) == -1)
265                         goto err_out;
266         }
267
268         ret = SRD_OK;
269
270 err_out:
271         Py_XDECREF(py_optlist);
272         Py_XDECREF(py_di_options);
273         Py_XDECREF(py_dec_optkeys);
274         Py_XDECREF(py_dec_options);
275         if (key)
276                 g_free(key);
277         if (PyErr_Occurred())
278                 catch_exception("Stray exception in srd_instance_set_options().");
279
280         return ret;
281 }
282
283 /* Helper GComparefunc for g_slist_find_custom() in srd_instance_set_probes() */
284 static gint compare_probe_id(struct srd_probe *a, char *probe_id)
285 {
286
287         return strcmp(a->id, probe_id);
288 }
289
290 /**
291  * Set probes in a decoder instance.
292  *
293  * @param di Decoder instance.
294  * @param probes A GHashTable of probes to set. Key is probe name, value is
295  * the probe number. Samples passed to this instance will be arranged in this
296  * order.
297  *
298  * @return SRD_OK upon success, a (negative) error code otherwise.
299  */
300 int srd_instance_set_probes(struct srd_decoder_instance *di,
301                 GHashTable *new_probes)
302 {
303         GList *l;
304         GSList *sl;
305         struct srd_probe *p;
306         int *new_probemap, new_probenum;
307         char *probe_id, *probenum_str;
308
309         srd_dbg("set probes called for instance %s with list of %d probes",
310                         di->instance_id, g_hash_table_size(new_probes));
311
312         if (g_hash_table_size(new_probes) == 0)
313                 /* No probes provided. */
314                 return SRD_OK;
315
316         if(di->dec_num_probes == 0) {
317                 /* Decoder has no probes. */
318                 srd_err("Protocol decoder %s has no probes to define.",
319                                 di->decoder->name);
320                 return SRD_ERR_ARG;
321         }
322
323         new_probemap = NULL;
324
325         if (!(new_probemap = g_try_malloc(sizeof(int) * di->dec_num_probes))) {
326                 srd_err("Failed to malloc new probe map.");
327                 return SRD_ERR_MALLOC;
328         }
329
330         for (l = g_hash_table_get_keys(new_probes); l; l = l->next) {
331                 probe_id = l->data;
332                 probenum_str = g_hash_table_lookup(new_probes, probe_id);
333                 if (!probenum_str) {
334                         /* Probe name was specified without a value. */
335                         srd_err("No probe number was specified for %s.", probe_id);
336                         g_free(new_probemap);
337                         return SRD_ERR_ARG;
338                 }
339                 new_probenum = strtol(probenum_str, NULL, 10);
340                 if (!(sl = g_slist_find_custom(di->decoder->probes, probe_id,
341                                 (GCompareFunc)compare_probe_id))) {
342                         /* Fall back on optional probes. */
343                         if (!(sl = g_slist_find_custom(di->decoder->extra_probes,
344                                         probe_id, (GCompareFunc)compare_probe_id))) {
345                                 srd_err("Protocol decoder %s has no probe '%s'.",
346                                                 di->decoder->name, probe_id);
347                                 g_free(new_probemap);
348                                 return SRD_ERR_ARG;
349                         }
350                 }
351                 p = sl->data;
352                 new_probemap[p->order] = new_probenum;
353                 srd_dbg("setting probe mapping for %d = probe %d", p->order, new_probenum);
354         }
355         g_free(di->dec_probemap);
356         di->dec_probemap = new_probemap;
357
358         return SRD_OK;
359 }
360
361 /**
362  * Create a new protocol decoder instance.
363  *
364  * @param id Decoder 'id' field.
365  * @param options GHashtable of options which override the defaults set in
366  *          the decoder class.
367  * @return Pointer to a newly allocated struct srd_decoder_instance, or
368  *              NULL in case of failure.
369  */
370 struct srd_decoder_instance *srd_instance_new(const char *decoder_id,
371                 GHashTable *options)
372 {
373         struct srd_decoder *dec;
374         struct srd_decoder_instance *di;
375         int i;
376         char *instance_id;
377
378         srd_dbg("Creating new %s instance.", decoder_id);
379
380         if (!(dec = srd_get_decoder_by_id(decoder_id))) {
381                 srd_err("Protocol decoder %s not found.", decoder_id);
382                 return NULL;
383         }
384
385         if (!(di = g_try_malloc0(sizeof(*di)))) {
386                 srd_err("Failed to malloc instance.");
387                 return NULL;
388         }
389
390         instance_id = g_hash_table_lookup(options, "id");
391         di->decoder = dec;
392         di->instance_id = g_strdup(instance_id ? instance_id : decoder_id);
393         g_hash_table_remove(options, "id");
394
395         /* Prepare a default probe map, where samples come in the
396          * order in which the decoder class defined them.
397          */
398         di->dec_num_probes = g_slist_length(di->decoder->probes) +
399                         g_slist_length(di->decoder->extra_probes);
400         if (di->dec_num_probes) {
401                 if (!(di->dec_probemap = g_try_malloc(sizeof(int) * di->dec_num_probes))) {
402                         srd_err("Failed to malloc probe map.");
403                         g_free(di);
404                         return NULL;
405                 }
406                 for (i = 0; i < di->dec_num_probes; i++)
407                         di->dec_probemap[i] = i;
408         }
409
410         /* Create a new instance of this decoder class. */
411         if (!(di->py_instance = PyObject_CallObject(dec->py_dec, NULL))) {
412                 if (PyErr_Occurred())
413                         catch_exception("failed to create %s instance: ", decoder_id);
414                 g_free(di->dec_probemap);
415                 g_free(di);
416                 return NULL;
417         }
418
419         if (srd_instance_set_options(di, options) != SRD_OK) {
420                 g_free(di->dec_probemap);
421                 g_free(di);
422                 return NULL;
423         }
424
425         /* Instance takes input from a frontend by default. */
426         di_list = g_slist_append(di_list, di);
427
428         return di;
429 }
430
431 int srd_instance_stack(struct srd_decoder_instance *di_from,
432                 struct srd_decoder_instance *di_to)
433 {
434
435         if (!di_from || !di_to) {
436                 srd_err("Invalid from/to instance pair.");
437                 return SRD_ERR_ARG;
438         }
439
440         if (g_slist_find(di_list, di_to)) {
441                 /* Remove from the unstacked list. */
442                 di_list = g_slist_remove(di_list, di_to);
443         }
444
445         /* Stack on top of source di. */
446         di_from->next_di = g_slist_append(di_from->next_di, di_to);
447
448         return SRD_OK;
449 }
450
451 /**
452  * Finds a decoder instance by its instance id, but only in the bottom
453  * level of instances -- instances already stacked on top of another one
454  * will not be found.
455  *
456  * @param instance_id The instance id to be found.
457  *
458  * @return Pointer to struct srd_decoder_instance, or NULL if not found.
459  */
460 struct srd_decoder_instance *srd_instance_find_by_id(char *instance_id)
461 {
462         GSList *l;
463         struct srd_decoder_instance *tmp, *di;
464
465         di = NULL;
466         for (l = di_list; l; l = l->next) {
467                 tmp = l->data;
468                 if (!strcmp(tmp->instance_id, instance_id)) {
469                         di = tmp;
470                         break;
471                 }
472         }
473
474         return di;
475 }
476
477 /**
478  * Finds a decoder instance by its python object, i.e. that instance's
479  * instantiation of the sigrokdecode.Decoder class. This will recurse
480  * to find the instance anywhere in the stack tree.
481  *
482  * @param stack Pointer to a GSList of struct srd_decoder_instance,
483  *              indicating the stack to search. To start searching at the bottom
484  *              level of decoder instances, pass NULL.
485  * @param obj The python class instantiation.
486  *
487  * @return Pointer to struct srd_decoder_instance, or NULL if not found.
488  */
489 struct srd_decoder_instance *srd_instance_find_by_obj(GSList *stack,
490                 PyObject *obj)
491 {
492         GSList *l;
493         struct srd_decoder_instance *tmp, *di;
494
495         di = NULL;
496         for (l = stack ? stack : di_list; di == NULL && l != NULL; l = l->next) {
497                 tmp = l->data;
498                 if (tmp->py_instance == obj)
499                         di = tmp;
500                 else if (tmp->next_di)
501                         di = srd_instance_find_by_obj(tmp->next_di, obj);
502         }
503
504         return di;
505 }
506
507 int srd_instance_start(struct srd_decoder_instance *di, PyObject *args)
508 {
509         PyObject *py_name, *py_res;
510         GSList *l;
511         struct srd_decoder_instance *next_di;
512
513         srd_dbg("Calling start() method on protocol decoder instance %s.",
514                 di->instance_id);
515
516         if (!(py_name = PyUnicode_FromString("start"))) {
517                 srd_err("Unable to build python object for 'start'.");
518                 catch_exception("Protocol decoder instance %s: ", di->instance_id);
519                 return SRD_ERR_PYTHON;
520         }
521
522         if (!(py_res = PyObject_CallMethodObjArgs(di->py_instance,
523                         py_name, args, NULL))) {
524                 catch_exception("Protocol decoder instance %s: ", di->instance_id);
525                 return SRD_ERR_PYTHON;
526         }
527
528         Py_DecRef(py_res);
529         Py_DecRef(py_name);
530
531         /* Start all the PDs stacked on top of this one. Pass along the
532          * metadata all the way from the bottom PD, even though it's only
533          * applicable to logic data for now.
534          */
535         for (l = di->next_di; l; l = l->next) {
536                 next_di = l->data;
537                 srd_instance_start(next_di, args);
538         }
539
540         return SRD_OK;
541 }
542
543 /**
544  * Run the specified decoder function.
545  *
546  * @param start_samplenum The starting sample number for the buffer's sample
547  *                        set, relative to the start of capture.
548  * @param di The decoder instance to call. Must not be NULL.
549  * @param inbuf The buffer to decode. Must not be NULL.
550  * @param inbuflen Length of the buffer. Must be > 0.
551  *
552  * @return SRD_OK upon success, a (negative) error code otherwise.
553  */
554 int srd_instance_decode(uint64_t start_samplenum,
555         struct srd_decoder_instance *di, uint8_t *inbuf, uint64_t inbuflen)
556 {
557         PyObject *py_res;
558         srd_logic *logic;
559         uint64_t end_samplenum;
560
561         srd_dbg("Calling decode() on instance %s with %d bytes starting "
562                 "at sample %d.", di->instance_id, inbuflen, start_samplenum);
563
564         /* Return an error upon unusable input. */
565         if (!di) {
566                 srd_dbg("empty decoder instance");
567                 return SRD_ERR_ARG;
568         }
569         if (!inbuf) {
570                 srd_dbg("NULL buffer pointer");
571                 return SRD_ERR_ARG;
572         }
573         if (inbuflen == 0) {
574                 srd_dbg("empty buffer");
575                 return SRD_ERR_ARG;
576         }
577
578         /* Create new srd_logic object. Each iteration around the PD's loop
579          * will fill one sample into this object.
580          */
581         logic = PyObject_New(srd_logic, &srd_logic_type);
582         Py_INCREF(logic);
583         logic->di = di;
584         logic->start_samplenum = start_samplenum;
585         logic->itercnt = 0;
586         logic->inbuf = inbuf;
587         logic->inbuflen = inbuflen;
588         logic->sample = PyList_New(2);
589         Py_INCREF(logic->sample);
590
591         Py_IncRef(di->py_instance);
592         end_samplenum = start_samplenum + inbuflen / di->data_unitsize;
593         if (!(py_res = PyObject_CallMethod(di->py_instance, "decode",
594                         "KKO", logic->start_samplenum, end_samplenum, logic))) {
595                 catch_exception("Protocol decoder instance %s: ", di->instance_id);
596                 return SRD_ERR_PYTHON; /* TODO: More specific error? */
597         }
598         Py_DecRef(py_res);
599
600         return SRD_OK;
601 }
602
603
604 int srd_session_start(int num_probes, int unitsize, uint64_t samplerate)
605 {
606         PyObject *args;
607         GSList *d;
608         struct srd_decoder_instance *di;
609         int ret;
610
611         srd_dbg("Calling start() on all instances with %d probes, "
612                 "unitsize %d samplerate %d.", num_probes, unitsize, samplerate);
613
614         /* Currently only one item of metadata is passed along to decoders,
615          * samplerate. This can be extended as needed.
616          */
617         if (!(args = Py_BuildValue("{s:l}", "samplerate", (long)samplerate))) {
618                 srd_err("Unable to build python object for metadata.");
619                 return SRD_ERR_PYTHON;
620         }
621
622         /* Run the start() method on all decoders receiving frontend data. */
623         for (d = di_list; d; d = d->next) {
624                 di = d->data;
625                 di->data_num_probes = num_probes;
626                 di->data_unitsize = unitsize;
627                 di->data_samplerate = samplerate;
628                 if ((ret = srd_instance_start(di, args) != SRD_OK))
629                         break;
630         }
631
632         Py_DecRef(args);
633
634         return ret;
635 }
636
637 /* Feed logic samples to decoder session. */
638 int srd_session_feed(uint64_t start_samplenum, uint8_t *inbuf, uint64_t inbuflen)
639 {
640         GSList *d;
641         int ret;
642
643         srd_dbg("Calling decode() on all instances with starting sample "
644                 "number %"PRIu64", %"PRIu64" bytes at 0x%p", start_samplenum,
645                 inbuflen, inbuf);
646
647         for (d = di_list; d; d = d->next) {
648                 if ((ret = srd_instance_decode(start_samplenum, d->data, inbuf,
649                                 inbuflen)) != SRD_OK)
650                         return ret;
651         }
652
653         return SRD_OK;
654 }
655
656
657 int srd_register_callback(int output_type, void *cb)
658 {
659         struct srd_pd_callback *pd_cb;
660
661         srd_dbg("Registering new callback for output type %d.", output_type);
662
663         if (!(pd_cb = g_try_malloc(sizeof(struct srd_pd_callback))))
664                 return SRD_ERR_MALLOC;
665
666         pd_cb->output_type = output_type;
667         pd_cb->callback = cb;
668         callbacks = g_slist_append(callbacks, pd_cb);
669
670         return SRD_OK;
671 }
672
673 void *srd_find_callback(int output_type)
674 {
675         GSList *l;
676         struct srd_pd_callback *pd_cb;
677         void *(cb);
678
679         cb = NULL;
680         for (l = callbacks; l; l = l->next) {
681                 pd_cb = l->data;
682                 if (pd_cb->output_type == output_type) {
683                         cb = pd_cb->callback;
684                         break;
685                 }
686         }
687
688         return cb;
689 }
690
691
692 /* This is the backend function to python sigrokdecode.add() call. */
693 int pd_add(struct srd_decoder_instance *di, int output_type, char *proto_id)
694 {
695         struct srd_pd_output *pdo;
696
697         srd_dbg("Instance %s creating new output type %d for %s.",
698                 di->instance_id, output_type, proto_id);
699
700         if (!(pdo = g_try_malloc(sizeof(struct srd_pd_output))))
701                 return -1;
702
703         /* pdo_id is just a simple index, nothing is deleted from this list anyway. */
704         pdo->pdo_id = g_slist_length(di->pd_output);
705         pdo->output_type = output_type;
706         pdo->di = di;
707         pdo->proto_id = g_strdup(proto_id);
708         di->pd_output = g_slist_append(di->pd_output, pdo);
709
710         return pdo->pdo_id;
711 }
712