X-Git-Url: http://git.lttng.org./?a=blobdiff_plain;f=genevent%2Fgenevent.c;h=b809a8301197e63b03ecd03dc30a7ae6d8a07f16;hb=590e95792b1bbba36dde969204635106228bfbba;hp=5a628ce88901c5ba7eca1e5f187e7c367d59c065;hpb=d9a67054dc8f62091fa9d05142244d64c0de7f1a;p=lttv.git

diff --git a/genevent/genevent.c b/genevent/genevent.c
index 5a628ce8..b809a830 100644
--- a/genevent/genevent.c
+++ b/genevent/genevent.c
@@ -49,14 +49,14 @@ This program is distributed in the hope that it will be useful,
 
 /* Named types may be referenced from anywhere */
 
-facility * fac;
+facility_t * fac;
 
 unsigned alignment = 0;
 
 int main(int argc, char** argv)
 {
   char *token;
-  parse_file in;
+  parse_file_t in;
   char buffer[BUFFER_SIZE];
   int i;
 
@@ -98,7 +98,7 @@ int main(int argc, char** argv)
 				token = getName(&in);
 			
 				if(strcmp("facility",token) == 0) {
-					fac = memAlloc(sizeof(facility));
+					fac = memAlloc(sizeof(facility_t));
 					fac->name = NULL;
 					fac->description = NULL;
 					sequence_init(&(fac->events));
@@ -139,7 +139,8 @@ int main(int argc, char** argv)
  *Input Params
  *    name         : name of event definition file
  ****************************************************************************/
-void generateFile(char *name){
+void generateFile(char *name)
+{
   char *loadName, *hName, *hIdName, *cName, *tmp, *tmp2;
   FILE * lFp, *hFp, *iFp, *cFp;
   int nbEvent;
@@ -245,7 +246,8 @@ void generateFile(char *name){
  *Output Params
  *    nbEvent           : number of events in the facility
  ****************************************************************************/
-void generateEnumEvent(FILE *fp, char *facName, int * nbEvent, unsigned long checksum) {
+void generateEnumEvent(FILE *fp, char *facName, int * nbEvent,
+                       unsigned long checksum) {
   int pos = 0;
 
   fprintf(fp,"#include <linux/ltt-facilities.h>\n\n");
@@ -258,7 +260,7 @@ void generateEnumEvent(FILE *fp, char *facName, int * nbEvent, unsigned long che
   fprintf(fp,"enum %s_event {\n",facName);
 
   for(pos = 0; pos < fac->events.position;pos++) {
-    fprintf(fp,"\tevent_%s", ((event *)(fac->events.array[pos]))->name);
+    fprintf(fp,"\tevent_%s", ((event_t *)(fac->events.array[pos]))->name);
     if(pos != fac->events.position-1) fprintf(fp,",\n");
   }
   fprintf(fp,"\n};\n\n\n");
@@ -286,15 +288,15 @@ void generateEnumEvent(FILE *fp, char *facName, int * nbEvent, unsigned long che
 static void
 printStruct(FILE * fp, int len, void ** array, char * name, char * facName,
        int * whichTypeFirst, int * hasStrSeq, int * structCount,
-       type_descriptor *type)
+       type_descriptor_t *type)
 {
   int flag = 0;
   int pos;
-  field * fld;
-  type_descriptor * td;
+  field_t * fld;
+  type_descriptor_t * td;
 
   for (pos = 0; pos < len; pos++) {
-    fld  = (field *)array[pos];
+    fld  = (field_t *)array[pos];
     td = fld->type;
     if( td->type == STRING || td->type == SEQUENCE ||
         td->type == ARRAY) {
@@ -369,7 +371,7 @@ generateTypeDefs(FILE * fp, char *facName)
  *    fp                    : file to be written to
  *    fHead                 : enum type
  ****************************************************************************/
-void generateEnumDefinition(FILE * fp, type_descriptor * type){
+void generateEnumDefinition(FILE * fp, type_descriptor_t * type){
   int pos;
 
   if(type->already_printed) return;
@@ -396,15 +398,15 @@ void generateEnumDefinition(FILE * fp, type_descriptor * type){
  *    facName           : name of facility
  ****************************************************************************/
 void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
-  event * ev;
-  field * fld;
-  type_descriptor * td;
+  event_t * ev;
+  field_t * fld;
+  type_descriptor_t * td;
   int pos, pos1;
   int hasStrSeq, flag, structCount, seqCount,strCount, whichTypeFirst=0;
   int args_empty;
 
   for(pos = 0; pos < fac->events.position; pos++){
-    ev = (event *) fac->events.array[pos];
+    ev = (event_t *) fac->events.array[pos];
     //yxx    if(ev->nested)continue;
     fprintf(fp,"/****  structure and trace function for event: %s  ****/\n\n",
         ev->name);
@@ -421,7 +423,7 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     //twice.
     if(ev->type != 0)
       for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-        fld = (field *)ev->type->fields.array[pos1];
+        fld = (field_t *)ev->type->fields.array[pos1];
         if(fld->type->type == ENUM) generateEnumDefinition(fp, fld->type);      
       }
       
@@ -460,7 +462,7 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     
     if(ev->type != 0) {
       for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-        fld  = (field *)ev->type->fields.array[pos1];
+        fld  = (field_t *)ev->type->fields.array[pos1];
         td = fld->type;
         if(!args_empty) fprintf(fp, ", ");
         if(td->type == ARRAY ){
@@ -508,7 +510,7 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
 
     if(ev->type != 0) {
       for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-        fld  = (field *)ev->type->fields.array[pos1];
+        fld  = (field_t *)ev->type->fields.array[pos1];
         td = fld->type;
         if(!args_empty) fprintf(fp, ", ");
         if(td->type == ARRAY ){
@@ -538,13 +540,11 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     fprintf(fp, "\tunsigned int index;\n");
     fprintf(fp, "\tstruct ltt_channel_struct *channel;\n");
     fprintf(fp, "\tstruct ltt_trace_struct *trace;\n");
-    fprintf(fp, "\tunsigned long _flags;\n");
     fprintf(fp, "\tvoid *buff;\n");
-    fprintf(fp, "\tunsigned old_offset;\n");
-    fprintf(fp, "\tunsigned int header_length;\n");
-    fprintf(fp, "\tunsigned int event_length;\n");	// total size (incl hdr)
-		fprintf(fp, "\tunsigned int length;\n");	// Size of the event var data.
-    fprintf(fp, "\tunsigned char _offset;\n");
+    fprintf(fp, "\tunsigned int slot_size;\n");	// total size (incl hdr)
+		fprintf(fp, "\tunsigned int data_size;\n");	// Size of the event var data.
+		fprintf(fp, "\tunsigned int before_hdr_pad;\n");// Dynamic padding before event header
+		fprintf(fp, "\tunsigned int after_hdr_pad;\n");	// Dynamic padding after event header
 		fprintf(fp, "\tstruct rchan_buf *buf;\n");
 		fprintf(fp, "\tstruct timeval delta;\n");
 		fprintf(fp, "\tu64 tsc;\n");
@@ -559,15 +559,75 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
 		 * just won't iterate on any trace). */
     fprintf(fp,
         "\tif(ltt_traces.num_active_traces == 0) return;\n\n");
+		
+    /* Calculate event variable len + event alignment offset.
+     * Assume that the padding for alignment starts at a void*
+     * address. */
+		fprintf(fp, "\t\tdata_size = 0;\n");
 
-    fprintf(fp, "\t/* Disable interrupts. */\n");
-    fprintf(fp, "\tlocal_irq_save(_flags);\n");
-    fprintf(fp, "\tpreempt_disable();\n\n");
+    for(pos1=0;pos1<structCount;pos1++){
+			if(ev->type->alignment > 1) {
+        fprintf(fp,"\t\t\tdata_size += ltt_align(data_size, %u);\n",
+						ev->type->alignment);
+      }
+      fprintf(fp,"\t\t\tdata_size += sizeof(struct %s_%s_%d);\n",
+					ev->name,facName,pos1+1);
+//      if(pos1 != structCount-1) fprintf(fp," + ");
+    }
+
+    //length of buffer : length of all arrays, sequences and strings
+    seqCount = 0;
+    strCount = 0;
+    flag = 0;
+		if(ev->type != 0)
+			for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
+				fld  = (field_t *)ev->type->fields.array[pos1];
+				td = fld->type;
+				if(td->type == SEQUENCE || td->type==STRING || td->type==ARRAY){
+					if(td->type == SEQUENCE) {
+
+						if(td->alignment > 1) {
+							fprintf(fp,"\t\t\tdata_size += ltt_align(data_size, %u);\n",
+									td->alignment);
+						}
+						fprintf(fp,"\t\t\tdata_size += sizeof(%s);\n",
+								uintOutputTypes[td->size]);
+
+						if(td->alignment > 1) {
+							fprintf(fp,"\t\tdata_size += ltt_align(data_size, %u);\n",
+									td->alignment);
+						}
+						fprintf(fp,"\t\tdata_size += sizeof(%s) * seqlength_%d;\n",
+								getTypeStr(td), seqCount);
+
+          } else if(td->type==STRING) {
+            if(td->alignment > 1) {
+							fprintf(fp,"\t\tdata_size += ltt_align(data_size, %u);\n",
+									td->alignment);
+            }
+						strCount++;
+						fprintf(fp,"\t\tdata_size += strlength_%d + 1;\n",
+								strCount);
+          }
+					else if(td->type==ARRAY) {
+            if(td->alignment > 1) {
+							fprintf(fp,"\t\tdata_size += ltt_align(data_size, %u);\n",
+									td->alignment);
+            }
+						fprintf(fp,"\t\tdata_size += sizeof(%s) * %d;\n",
+								getTypeStr(td), td->size);
 
+					if(structCount == 0) flag = 1;
+          }
+				}
+			}
+		
+		/* We use preempt_disable instead of rcu_read_lock because we want to
+		 * use preempt_enable_no_resched : the only way to be called from the
+		 * scheduler code safely! */
+    fprintf(fp, "\tpreempt_disable();\n\n");
 		fprintf(fp, "\tltt_nesting[smp_processor_id()]++;\n");
-		fprintf(fp, "\tbarrier();\n");
 		fprintf(fp, "\tif(ltt_nesting[smp_processor_id()] > 1) goto unlock;\n");
-    fprintf(fp, "\tspin_lock(&ltt_traces.locks[smp_processor_id()]);\n\n");
 		
     if(ev->per_tracefile) {
       fprintf(fp, "\tindex = tracefile_index;\n");
@@ -580,7 +640,7 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     fprintf(fp,"\n");
 
     /* For each trace */
-    fprintf(fp, "\tlist_for_each_entry(trace, &ltt_traces.head, list) {\n");
+    fprintf(fp, "\tlist_for_each_entry_rcu(trace, &ltt_traces.head, list) {\n");
     fprintf(fp, "\t\tif(!trace->active) continue;\n\n");
 
     if(ev->per_trace) {
@@ -592,134 +652,125 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     fprintf(fp, "\t\tchannel = ltt_get_channel_from_index(trace, index);\n");
 		fprintf(fp, "\t\tbuf = channel->rchan->buf[smp_processor_id()];\n");
 		fprintf(fp, "\n");
-		/* Warning : not atomic reservation : event size depends on the current
-		 * address for alignment */
-		/* NOTE : using cmpxchg in reserve with repeat for atomicity */
-    // Replaces _offset
-		/* NOTE2 : as the read old address from memory must come before any
-		 * protected event, let's add a memory barrier() to make sure the compiler
-		 * will not reorder this read. */
-		fprintf(fp, "\t\tdo {\n");
-		fprintf(fp, "\t\t\told_offset = buf->offset;\n");
-		fprintf(fp, "\t\t\tbarrier();\n");
-    fprintf(fp, "\t\t\tptr = (char*)buf->data + old_offset;\n");
 
 
-    fprintf(fp, "\t\t\theader_length = ltt_get_event_header_data(trace, "
-																																"channel,\n"
-								"\t\t\t\t\t\t\t\t\t\tptr, &_offset, &delta, &tsc);\n");
+    /* Relay reserve */
+		fprintf(fp, "\t\tslot_size = 0;\n");
+		fprintf(fp, "\t\tbuff = ltt_reserve_slot(trace, buf, event_length, &slot_size, &tsc,\n"
+			"\t\t&before_hdr_pad, &after_hdr_pad);\n");
+    /* If error, return */
+		fprintf(fp, "if(!buff) return;\n");
 
-    fprintf(fp, "\t\t\tptr += _offset + header_length;\n");
+    /* write data */
 
-    for(pos1=0;pos1<structCount;pos1++){
+    /* Write the header */
+		/* FIXME : delta is not set ! */
+    fprintf(fp, "\n");
+    fprintf(fp, "\t\tltt_write_event_header(trace, channel, buff, \n"
+                "\t\t\t\tltt_facility_%s_%X, event_%s, data_size, before_hdr_pad,\n"
+								"\t\t\t\t&delta, &tsc);\n",
+                facName, checksum, ev->name);
+    fprintf(fp, "\n");
 
-			unsigned align = max(alignment, ev->type->alignment);
+    if(ev->type != 0) {
+      fprintf(fp,"\t\tptr = (char*)buff + before_hdr_pad + header_length + after_hdr_pad;\n");
+
+			/* Write structures */
+			for(pos1=0;pos1<structCount;pos1++){
+				if(ev->type->alignment > 1) {
+					fprintf(fp,"\t\tptr += ltt_align((unsigned long)ptr, %u);\n",
+							ev->type->alignment);
+				}
+				fprintf(fp,"\t\t__%d = (struct %s_%s_%d *)(ptr);\n",
+						pos1+1, ev->name,facName,pos1+1);
+
+				fprintf(fp,"\t\t/* initialize structs */\n");
+
+				for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
+					fld  = (field_t *)ev->type->fields.array[pos1];
+					td = fld->type;
+					if(td->type != ARRAY && td->type != SEQUENCE && td->type != STRING){
+						fprintf(fp, "\t\t__%d->%s = %s;\n", pos1+1, fld->name, fld->name);
+					}
+				}
+				if(structCount) fprintf(fp,"\n");
+			}
+
+		}
+
+
+    //declare a char pointer if needed : starts at the end of the structs.
+    //if(structCount + hasStrSeq > 1) {
+    //  fprintf(fp,"\t\tchar * ptr = (char*)buff + header_length");
+    //  for(pos1=0;pos1<structCount;pos1++){
+    //    fprintf(fp," + sizeof(struct %s_%s_%d)",ev->name, facName,pos1+1);
+    //  }
+    //  if(structCount + hasStrSeq > 1) fprintf(fp,";\n");
+    //}
+
+    // Declare an alias pointer of the struct type to the beginning
+    // of the reserved area, just after the event header.
+    if(ev->type != 0) {
+
+			unsigned align = max(alignment, td->alignment);
       if(align > 1) {
-        fprintf(fp,"\t\t\tptr += (%u - ((unsigned int)ptr&(%u-1)))&(%u-1) ;\n",
-            align, align, align);
+        fprintf(fp,"\t\tptr+=(%u - ((unsigned int)ptr&(%u-1)))&(%u-1);\n",
+              align, align, align);
       }
-      fprintf(fp,"\t\t\tptr += sizeof(struct %s_%s_%d);\n",ev->name,
-          facName,pos1+1);
-//      if(pos1 != structCount-1) fprintf(fp," + ");
+    
+      fprintf(fp, "\t\t__1 = (struct %s_%s_1 *)(ptr);\n",
+          ev->name, facName);
     }
+    //allocate memory for new struct and initialize it
+    //if(whichTypeFirst == 1){ //struct first
+      //for(pos1=0;pos1<structCount;pos1++){  
+      //  if(pos1==0) fprintf(fp,
+      //      "\tstruct %s_%s_1 * __1 = (struct %s_%s_1 *)buff;\n",
+      //      ev->name, facName,ev->name, facName);
+  //MD disabled      else fprintf(fp,
+  //          "\tstruct %s_%s_%d  __%d;\n",
+  //          ev->name, facName,pos1+1,pos1+1);
+      //}      
+    //}else if(whichTypeFirst == 2){
+     // for(pos1=0;pos1<structCount;pos1++)
+     //   fprintf(fp,"\tstruct %s_%s_%d  __%d;\n",
+     //       ev->name, facName,pos1+1,pos1+1);
+    //}
+    fprintf(fp,"\n");
 
-    //length of buffer : length of all arrays, sequences and strings
-    seqCount = 0;
-    strCount = 0;
-    flag = 0;
+    if(structCount) fprintf(fp,"\t\t//initialize structs\n");
+    //flag = 0;
+    //structCount = 0;
 		if(ev->type != 0)
 			for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-				fld  = (field *)ev->type->fields.array[pos1];
+				fld  = (field_t *)ev->type->fields.array[pos1];
 				td = fld->type;
-				if(td->type == SEQUENCE || td->type==STRING || td->type==ARRAY){
-					if(td->type == SEQUENCE) {
-
-						unsigned align = max(alignment, td->alignment);
-						if(align > 1) {
-							fprintf(fp,"\t\tptr+=(%u - ((unsigned int)ptr&(%u-1)))&(%u-1);\n",
-											align, align, align);
-						}
-						fprintf(fp,"\t\tptr += sizeof(%s);\n",uintOutputTypes[td->size]);
-						if(align > 1) {
-							fprintf(fp,"\t\tptr+=(%u - ((unsigned int)ptr&(%u-1)))&(%u-1);\n",
-											align, align, align);
-						}
-						fprintf(fp,"\t\tptr += sizeof(%s) * seqlength_%d;\n\n",
-							getTypeStr(td), seqCount);
+				if(td->type != ARRAY && td->type != SEQUENCE && td->type != STRING){
+					//if(flag == 0){
+					//  flag = 1;  
+					//  structCount++;
+					//  if(structCount > 1) fprintf(fp,"\n");
+					//}
+					fprintf(fp, "\t\t__1->%s = %s;\n", fld->name, fld->name );
 
-          } else if(td->type==STRING) {
-						unsigned align = max(alignment, td->alignment);
-            if(align > 1) {
-              fprintf(fp,"\t\t\tptr += (%u - ((unsigned int)ptr&(%u-1)))&(%u-1)) ;\n",
-                 align, align, align);
-            }
-            fprintf(fp,"ptr += strlength_%d + 1;\n",
-                                                ++strCount);
-          }
-					else if(td->type==ARRAY) {
-						unsigned align = max(alignment, td->alignment);
-            if(align > 1) {
-              fprintf(fp,"\t\t\tptr += (%u - ((unsigned int)ptr&(%u-1)))&(%u-1);\n",
-                 align, align, align);
-            }
-						fprintf(fp,"\t\t\tptr += sizeof(%s) * %d;\n",
-                getTypeStr(td),td->size);
-					if(structCount == 0) flag = 1;
-          }
+				//if(structCount == 1 && whichTypeFirst == 1)
+				//  fprintf(fp, "\t__1->%s =  %s;\n",fld->name,fld->name );
+				//else 
+				//  fprintf(fp, "\t__%d.%s =  %s;\n",structCount ,fld->name,fld->name);
 				}
+				//else flag = 0;
 			}
-    fprintf(fp, "\t\t\tevent_length = (unsigned long)ptr -"
-				"(unsigned long)(buf->data + old_offset);\n");
-		
-		/* let's put some protection before the cmpxchg : the space reservation and
-		 * the get TSC are not dependant from each other. I don't want the compiler
-		 * to reorder those in the wrong order. And relay_reserve is inline, so
-		 * _yes_, the compiler could mess it up. */
-		fprintf(fp, "\t\t\tbarrier();\n");
-		fprintf(fp, "\t\t\tbuff = relay_reserve(channel->rchan, event_length, "
-												"old_offset);\n");
-		fprintf(fp, "\n");
- 		fprintf(fp, "\t\t} while(PTR_ERR(buff) == -EAGAIN);\n");
-		fprintf(fp, "\n");
-
-  
-    /* Reserve the channel */
-    //fprintf(fp, "\t\tbuff = relay_reserve(channel->rchan, event_length);\n");
-    fprintf(fp, "\t\tif(buff == NULL) {\n");
-    fprintf(fp, "\t\t\t/* Buffer is full*/\n");
-    fprintf(fp, "\t\t\t/* for debug BUG(); */\n"); // DEBUG!
-    fprintf(fp, "\t\t\tchannel->events_lost[smp_processor_id()]++;\n");
-    fprintf(fp, "\t\t\tbreak;\n");	 /* don't commit a NULL reservation! */
-    fprintf(fp, "\t\t}\n");
-		
-		/* DEBUG */
-    //fprintf(fp, "\t\tif(resret == 1) {\n");
-		//fprintf(fp, "printk(\"f%%lu e\%%u \", ltt_facility_%s_%X, event_%s);",
-		//		facName, checksum, ev->name);
-    //fprintf(fp, "\t\t}\n");
-
-    /* Write the header */
-    fprintf(fp, "\n");
-		fprintf(fp,"\t\tlength = event_length - _offset - header_length;\n");
-    fprintf(fp, "\n");
-    fprintf(fp, "\t\tltt_write_event_header(trace, channel, buff, \n"
-                "\t\t\t\tltt_facility_%s_%X, event_%s, length, _offset,\n"
-								"\t\t\t\t&delta, &tsc);\n",
-                facName, checksum, ev->name);
-    fprintf(fp, "\n");
+    if(structCount) fprintf(fp,"\n");
 
-    if(ev->type != 0)
-      fprintf(fp, "\t\tptr = (char*)buff + _offset + header_length;\n");
 
-    //copy struct, sequence and string to buffer
-    //FIXME : they always come before the structs.
+    //copy sequence and string to buffer
     seqCount = 0;
     strCount = 0;
     flag = 0;
     structCount = 0;
 		if(ev->type != 0)
 			for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-				fld  = (field *)ev->type->fields.array[pos1];
+				fld  = (field_t *)ev->type->fields.array[pos1];
 				td = fld->type;
 	//      if(td->type != STRING && td->type != SEQUENCE && td->type != ARRAY){
 	//        if(flag == 0) structCount++;  
@@ -788,67 +839,46 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
 
 
 
-    //declare a char pointer if needed : starts at the end of the structs.
-    //if(structCount + hasStrSeq > 1) {
-    //  fprintf(fp,"\t\tchar * ptr = (char*)buff + header_length");
-    //  for(pos1=0;pos1<structCount;pos1++){
-    //    fprintf(fp," + sizeof(struct %s_%s_%d)",ev->name, facName,pos1+1);
-    //  }
-    //  if(structCount + hasStrSeq > 1) fprintf(fp,";\n");
-    //}
+    /* commit */
+		ltt_commit_slot(buf, buff, slot_size);
+		
 
-    // Declare an alias pointer of the struct type to the beginning
-    // of the reserved area, just after the event header.
-    if(ev->type != 0) {
-			unsigned align = max(alignment, td->alignment);
-      if(align > 1) {
-        fprintf(fp,"\t\tptr+=(%u - ((unsigned int)ptr&(%u-1)))&(%u-1);\n",
-              align, align, align);
-      }
-    
-      fprintf(fp, "\t\t__1 = (struct %s_%s_1 *)(ptr);\n",
-          ev->name, facName);
-    }
-    //allocate memory for new struct and initialize it
-    //if(whichTypeFirst == 1){ //struct first
-      //for(pos1=0;pos1<structCount;pos1++){  
-      //  if(pos1==0) fprintf(fp,
-      //      "\tstruct %s_%s_1 * __1 = (struct %s_%s_1 *)buff;\n",
-      //      ev->name, facName,ev->name, facName);
-  //MD disabled      else fprintf(fp,
-  //          "\tstruct %s_%s_%d  __%d;\n",
-  //          ev->name, facName,pos1+1,pos1+1);
-      //}      
-    //}else if(whichTypeFirst == 2){
-     // for(pos1=0;pos1<structCount;pos1++)
-     //   fprintf(fp,"\tstruct %s_%s_%d  __%d;\n",
-     //       ev->name, facName,pos1+1,pos1+1);
-    //}
-    fprintf(fp,"\n");
 
-    if(structCount) fprintf(fp,"\t\t//initialize structs\n");
-    //flag = 0;
-    //structCount = 0;
-		if(ev->type != 0)
-			for(pos1 = 0; pos1 < ev->type->fields.position; pos1++){
-				fld  = (field *)ev->type->fields.array[pos1];
-				td = fld->type;
-				if(td->type != ARRAY && td->type != SEQUENCE && td->type != STRING){
-					//if(flag == 0){
-					//  flag = 1;  
-					//  structCount++;
-					//  if(structCount > 1) fprintf(fp,"\n");
-					//}
-					fprintf(fp, "\t\t__1->%s = %s;\n", fld->name, fld->name );
 
-				//if(structCount == 1 && whichTypeFirst == 1)
-				//  fprintf(fp, "\t__1->%s =  %s;\n",fld->name,fld->name );
-				//else 
-				//  fprintf(fp, "\t__%d.%s =  %s;\n",structCount ,fld->name,fld->name);
-				}
-				//else flag = 0;
-			}
-    if(structCount) fprintf(fp,"\n");
+#if 0
+		fprintf(fp, "\t\tdo {\n");
+		fprintf(fp, "\t\t\told_offset = buf->offset;\n");
+    fprintf(fp, "\t\t\tptr = (char*)buf->data + old_offset;\n");
+
+
+    fprintf(fp, "\t\t\theader_length = ltt_get_event_header_data(trace, "
+																																"channel,\n"
+								"\t\t\t\t\t\t\t\t\t\tptr, &_offset, &delta, &tsc);\n");
+
+    fprintf(fp, "\t\t\tptr += _offset + header_length;\n");
+
+		fprintf(fp, "\t\t\tbuff = relay_reserve(channel->rchan, event_length, "
+												"old_offset);\n");
+		fprintf(fp, "\n");
+ 		fprintf(fp, "\t\t} while(PTR_ERR(buff) == -EAGAIN);\n");
+		fprintf(fp, "\n");
+
+  
+    /* Reserve the channel */
+    //fprintf(fp, "\t\tbuff = relay_reserve(channel->rchan, event_length);\n");
+    fprintf(fp, "\t\tif(buff == NULL) {\n");
+    fprintf(fp, "\t\t\t/* Buffer is full*/\n");
+    fprintf(fp, "\t\t\t/* for debug BUG(); */\n"); // DEBUG!
+    fprintf(fp, "\t\t\tchannel->events_lost[smp_processor_id()]++;\n");
+    fprintf(fp, "\t\t\tbreak;\n");	 /* don't commit a NULL reservation! */
+    fprintf(fp, "\t\t}\n");
+		
+		/* DEBUG */
+    //fprintf(fp, "\t\tif(resret == 1) {\n");
+		//fprintf(fp, "printk(\"f%%lu e\%%u \", ltt_facility_%s_%X, event_%s);",
+		//		facName, checksum, ev->name);
+    //fprintf(fp, "\t\t}\n");
+
     //set ptr to the end of first struct if needed;
     //if(structCount + hasStrSeq > 1){
     //  fprintf(fp,"\n\t\t//set ptr to the end of the first struct\n");
@@ -862,20 +892,15 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
     fprintf(fp, "\t\tltt_write_commit_counter("
 				"channel->rchan->buf[smp_processor_id()],\n"
         "\t\t\t\tbuff);\n");
+#endif //0
 
    /* End of traces iteration */
     fprintf(fp, "\t}\n\n");
 
     fprintf(fp, "\n");
-		// The generated preempt_check_resched is not dangerous because
-		// interrupts are disabled.
-    fprintf(fp, "\tspin_unlock(&ltt_traces.locks[smp_processor_id()]);\n");
 
     fprintf(fp, "unlock:\n");
-		fprintf(fp, "\tbarrier();\n");
 		fprintf(fp, "\tltt_nesting[smp_processor_id()]--;\n");
-    fprintf(fp, "\t/* Re-enable interrupts */\n");
-    fprintf(fp, "\tlocal_irq_restore(_flags);\n");
     fprintf(fp, "\tpreempt_enable_no_resched();\n");
     //fprintf(fp, "\tpreempt_check_resched();\n");
     
@@ -896,8 +921,8 @@ void generateStructFunc(FILE * fp, char * facName, unsigned long checksum){
  *Return Values
  *    char *            : type string
  ****************************************************************************/
-char * getTypeStr(type_descriptor * td){
-  type_descriptor * t ;
+char * getTypeStr(type_descriptor_t * td){
+  type_descriptor_t * t ;
 
   switch(td->type){
     case INT:
@@ -1052,7 +1077,7 @@ void generateCfile(FILE * fp, char * filefacname){
   fprintf(fp, "\tprintk(KERN_INFO \"LTT : ltt-facility-%s init dynamic\\n\");\n", filefacname);
   fprintf(fp, "\n");
   fprintf(fp, "\tLTT_FACILITY_SYMBOL = ltt_facility_dynamic_register(&facility);\n");
-  fprintf(fp, "\tLTT_FACILITY_SYMBOL_CHECKSUM = LTT_FACILITY_SYMBOL;\n");
+  fprintf(fp, "\tLTT_FACILITY_CHECKSUM_SYMBOL = LTT_FACILITY_SYMBOL;\n");
   fprintf(fp, "\n");
   fprintf(fp, "\treturn LTT_FACILITY_SYMBOL;\n");
   fprintf(fp, "}\n");