improve a bit some error messages

git-svn-id: http://www.frescor.org/private/svn/frescor/fna/trunk@1556 35b4ef3e-fd22-0410-ab77-dab3279adceb

diff --git a/src_frescan/frescan.c b/src_frescan/frescan.c
index 30f02c2ed04827642710d4ccdb884d4c7846d9d5..c2f5c7ec18b1e81f8b3a37cf850318ebf4a53cb0 100644 (file)
--- a/src_frescan/frescan.c
+++ b/src_frescan/frescan.c
@@ -121,6 +121,7 @@ int frescan_init(frescan_init_params_t *params)
         fd = open (can_path, O_RDWR);
         if (fd == -1) {
                 FRESCAN_ERROR ("could not open /dev/can%u\n", params->net);
+                FRESCAN_ERROR ("hint: check driver or card installation\n");
                 return -1;
         }
 
@@ -486,10 +487,15 @@ static int frescan_hook_frame_recv (const struct can_chip_t *chip,
         DEBUG(FRESCAN_RX_HOOK_ENABLE_DEBUG || FRESCAN_FRAG_ENABLE_DEBUG,
               "prio:%u dest:%u src:%u chan:%u id:%u flag:%u\n",
               prio, dest, src, channel, frag_id, frag_flag);
-
         DEBUG(FRESCAN_RX_HOOK_ENABLE_DEBUG,
               "enqueue the packet in ID queue\n");
+
         packet = frescan_packets_alloc();
+        if (packet == NULL) {
+                FRESCAN_ERROR("could not allocate packet\n");
+                return -1;
+        }
+
         packet->frame = frame;
 
         if (frag_id == FRESCAN_MX_IDS) {
@@ -503,6 +509,11 @@ static int frescan_hook_frame_recv (const struct can_chip_t *chip,
                                 FRESCAN_FRAG_ENABLE_DEBUG,
                 "allocate head for id=%u\n", frag_id);
                 head = frescan_packets_alloc();
+                if (head == NULL) {
+                        FRESCAN_ERROR("could not allocate packet\n");
+                        return -1;
+                }
+
                 INIT_LIST_HEAD(&head->msg_list);
 
                 if (frag_id == FRESCAN_MX_IDS) {

diff --git a/src_frescan/frescan_bwres_fna.c b/src_frescan/frescan_bwres_fna.c
index 4dc11a93d7a653765ba673be3d697fa558e107b0..cfff706d620fec2d8cfb54ed0b9ab0c48d7abe14 100644 (file)
--- a/src_frescan/frescan_bwres_fna.c
+++ b/src_frescan/frescan_bwres_fna.c
@@ -110,12 +110,18 @@ int frescan_fna_init(const frsh_resource_id_t resource_id)
               "Initializing FRESCAN FNA node%u\n", init_params.node);
 
         ret = frescan_init(&init_params);
-        if (ret != 0) FRESCAN_ERROR ("could not init FRESCAN");
+        if (ret != 0) {
+                FRESCAN_ERROR ("could not init FRESCAN\n");
+                return ret;
+        }
 
         DEBUG(FRESCAN_BWRES_FNA_ENABLE_DEBUG, "Initializing BWRES\n");
 
         ret = frescan_bwres_init(init_params.net);
-        if (ret != 0) FRESCAN_ERROR ("could not init BWRES");
+        if (ret != 0) {
+                FRESCAN_ERROR ("could not init BWRES\n");
+                return ret;
+        }
 
         return 0;
 }

diff --git a/src_frescan/frescan_debug.h b/src_frescan/frescan_debug.h
index 0f2222b373e1fc973d3373868bf60fc1106c136b..32546c73c606395202a687b5c3bede83083b38e8 100644 (file)
--- a/src_frescan/frescan_debug.h
+++ b/src_frescan/frescan_debug.h
@@ -78,9 +78,14 @@
  * DEBUG macros
  **/
 
-#define DEBUG(enable,x,args...) if(enable) printc("DEBUG (%s): " x, __func__ , ##args)
-#define FRESCAN_ERROR(x,args...) {printe("ERROR (%s:%u): " x, __func__ , __LINE__ , ##args); exit(-1);}
-#define FRESCAN_WARNING(x,args...) printe("WARNING (%s): " x, __func__ , ##args)
+#define DEBUG(enable,x,args...) \
+        if(enable) printc("DEBUG (%s): " x, __func__ , ##args)
+
+#define FRESCAN_ERROR(x,args...) \
+        printe("ERROR (%s:%u): " x, __func__ , __LINE__ , ##args)
+
+#define FRESCAN_WARNING(x,args...) \
+        printe("WARNING (%s): " x, __func__ , ##args)
 
 /**
  * DEBUGGING FLAGS to enable/disable debugging messages

diff --git a/src_frescan/frescan_packets.c b/src_frescan/frescan_packets.c
index c40422567abbaf447f002160aea8007974317964..5fc5520ad858459a639fbe731c826f69fd7279cb 100644 (file)
--- a/src_frescan/frescan_packets.c
+++ b/src_frescan/frescan_packets.c
@@ -111,6 +111,7 @@ frescan_packet_t *frescan_packets_alloc() {
         pos = freelist_alloc(&the_packet_pool_freelist);
         if (pos == -1) {
                 FRESCAN_ERROR("could not allocate packet\n");
+                FRESCAN_ERROR("hint: configure FRESCAN_MX_PACKETS\n");
                 return NULL;
         }