From: SourceForge.net <no...@so...> - 2010-08-26 20:17:25
|
Bugs item #3053794, was opened at 2010-08-26 16:17 Message generated for change (Tracker Item Submitted) made by rgrunber You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Nobody/Anonymous (nobody) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-08-27 14:47:18
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Nobody/Anonymous (nobody) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-08-27 14:57:51
|
Bugs item #3053794, was opened at 2010-08-26 16:17 Message generated for change (Comment added) made by rgrunber You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Nobody/Anonymous (nobody) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 10:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 10:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-09-05 22:14:04
|
Bugs item #3053794, was opened at 2010-08-26 16:17 Message generated for change (Comment added) made by rgrunber You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Nobody/Anonymous (nobody) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 18:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 10:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 10:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-09-07 22:38:29
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) >Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-09-08 15:54:20
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-09-30 02:07:30
|
Bugs item #3053794, was opened at 2010-08-26 16:17 Message generated for change (Comment added) made by rgrunber You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 22:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 11:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 18:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 18:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 10:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 10:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-09-30 15:22:06
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 10:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 21:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-10-01 03:23:38
|
Bugs item #3053794, was opened at 2010-08-26 16:17 Message generated for change (Comment added) made by rgrunber You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Roland Grunberg (rgrunber) Date: 2010-09-30 23:23 Message: Minor change to patch. (bump up schema version to 1.1) ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 11:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 22:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 11:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 18:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 18:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 10:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 10:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-10-01 16:05:31
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-10-01 11:05 Message: Updated patch looks fine. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-30 22:23 Message: Minor change to patch. (bump up schema version to 1.1) ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 10:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 21:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2010-10-15 18:19:42
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2010-10-15 13:19 Message: Roland, I'd like to close out this bug and get your patch committed. But as I said in my September 30 comment to this bug . . . "your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours." Can you send your review comments to that patch to the oprofile-list please? Thanks. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-10-01 11:05 Message: Updated patch looks fine. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-30 22:23 Message: Minor change to patch. (bump up schema version to 1.1) ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 10:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 21:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2011-01-23 16:12:20
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open >Resolution: Fixed Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2011-01-23 10:12 Message: The patch for this bug was committed to oprofile CVS on Oct 15, 2010. I neglected to update this bugzilla with that information until now. Changed "Resolution" field to "Fixed". ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-10-15 13:19 Message: Roland, I'd like to close out this bug and get your patch committed. But as I said in my September 30 comment to this bug . . . "your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours." Can you send your review comments to that patch to the oprofile-list please? Thanks. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-10-01 11:05 Message: Updated patch looks fine. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-30 22:23 Message: Minor change to patch. (bump up schema version to 1.1) ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 10:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 21:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |
From: SourceForge.net <no...@so...> - 2011-08-12 14:11:07
|
Bugs item #3053794, was opened at 2010-08-26 15:17 Message generated for change (Settings changed) made by ssuthiku You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed Resolution: Fixed Priority: 5 Private: No Submitted By: Roland Grunberg (rgrunber) Assigned to: Maynard Johnson (maynardj) Summary: ophelp -X doesn't display event's unit mask type Initial Comment: oprofile-0.9.6-6.fc13.i686 It would be nice if "ophelp -X" displayed the unit mask type (mandatory, exclusive, bitmask) as an attribute in the <unit_masks> tag. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2011-01-23 10:12 Message: The patch for this bug was committed to oprofile CVS on Oct 15, 2010. I neglected to update this bugzilla with that information until now. Changed "Resolution" field to "Fixed". ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-10-15 13:19 Message: Roland, I'd like to close out this bug and get your patch committed. But as I said in my September 30 comment to this bug . . . "your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours." Can you send your review comments to that patch to the oprofile-list please? Thanks. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-10-01 11:05 Message: Updated patch looks fine. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-30 22:23 Message: Minor change to patch. (bump up schema version to 1.1) ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-30 10:22 Message: Roland, your patch looks good, with one caveat. I think, by rights, we should bump the schema version to "1.1" since the new attribute being added is required. As you know, your patch is dependent on the patch that I posted to the list (mentioned below) to fix other unrelated schema issues with ophelp.xsd. I've not been able to get Richard Purdie's attention to review that patch. But as a user of ophelp's XML output, your review would be more than satisfactory. So please find that original posting of mine (Sept 8, subject "[Patch] Fix ophelp schema validation problems and xml generation error") and give it a thumbs up. Please also indicate that you are a consumer of ophelp XML output and mention that this patch is a pre-req to a patch you submitted for this bug. Once that's done, I'll commit both that patch and yours. Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-29 21:07 Message: Patch is posted, and applied on top of the one mentioned. xml file included as well. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-08 10:54 Message: Roland, I just posted a patch to the oprofile-list that fixes some problems in ophelp.xsd and the generated XML output. Please pull the latest oprofile CVS, apply that patch, and then respin your patch, addressing my aforementioned review comments. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-09-07 17:38 Message: Thanks for the patch submission. A few comments: 1. The patch gets a compile error when applied to current CVS (minor one line change needed). 2. I suggest changing the new unit mask attribute name from "type" to something else (e.g., "category"), since "type" is a reserved word in xml schema language. It may be OK to do, but it could tend to be confusing to some. 3. The doc/ophelp.xsd should be updated to include the new optional attribute. Regarding comment #3, your patch motivated me to check our current ophelp schema doc and the ophelp XML instances our code creates using an XML validator, and unfortunately, I found some problems. I'll be posting a patch to fix those problems, which will include fixes to doc/ophelp.xsd, so please wait until that patch is committed to our CVS before you re-spin your patch to address my above comments. Oh, and when you do submit a new patch, could you also attach an example XML instance that I can run through the XML validator (the one I was using can be found at http://tools.decisionsoft.com/schemaValidate/). Thanks. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-09-05 17:14 Message: Resubmitted patch as attachment to conform with contribution guidelines. ---------------------------------------------------------------------- Comment By: Roland Grunberg (rgrunber) Date: 2010-08-27 09:57 Message: Hi, the patch is was the attached file "unit_mask_type.patch". It's small enough that I could have probably posted it here, so sorry for the confusion. (hope this shows up correctly) diff -ru oprofile-0.9.6/libop/op_xml_events.c oprofile-0.9.6-new/libop/op_xml_events.c --- oprofile-0.9.6/libop/op_xml_events.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_events.c 2010-08-26 14:30:26.000000000 -0400 @@ -80,6 +80,19 @@ open_xml_element(HELP_UNIT_MASKS, 1, buffer, MAX_BUFFER); init_xml_int_attr(HELP_DEFAULT_MASK, event->unit->default_mask, buffer, MAX_BUFFER); + char um_type[10]; + switch (event->unit->unit_type_mask){ + case utm_bitmask: + strncpy(um_type, "bitmask", sizeof(um_type)); + break; + case utm_exclusive: + strncpy(um_type, "exclusive", sizeof(um_type)); + break; + case utm_mandatory: + strncpy(um_type, "mandatory", sizeof(um_type)); + break; + } + init_xml_str_attr(HELP_UNIT_MASKS_TYPE, um_type, buffer, MAX_BUFFER); close_xml_element(NONE, 1, buffer, MAX_BUFFER); for (i = 0; i < event->unit->num; i++) { open_xml_element(HELP_UNIT_MASK, 1, buffer, MAX_BUFFER); diff -ru oprofile-0.9.6/libop/op_xml_out.c oprofile-0.9.6-new/libop/op_xml_out.c --- oprofile-0.9.6/libop/op_xml_out.c 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.c 2010-08-26 12:21:23.000000000 -0400 @@ -80,6 +80,7 @@ "ext", "unit_masks", "default", + "type", "unit_mask", "mask", "desc" diff -ru oprofile-0.9.6/libop/op_xml_out.h oprofile-0.9.6-new/libop/op_xml_out.h --- oprofile-0.9.6/libop/op_xml_out.h 2009-11-24 10:25:17.000000000 -0500 +++ oprofile-0.9.6-new/libop/op_xml_out.h 2010-08-26 12:19:53.000000000 -0400 @@ -54,6 +54,7 @@ HELP_EXT, HELP_UNIT_MASKS, HELP_DEFAULT_MASK, + HELP_UNIT_MASKS_TYPE, HELP_UNIT_MASK, HELP_UNIT_MASK_VALUE, HELP_UNIT_MASK_DESC ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2010-08-27 09:47 Message: Can you submit a patch? I'd be happy to review it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3053794&group_id=16191 |