From: SourceForge.net <no...@so...> - 2013-01-04 22:46:56
|
Bugs item #3599530, was opened at 2013-01-04 14:04 Message generated for change (Settings changed) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&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: Maynard Johnson (maynardj) Assigned to: Nobody/Anonymous (nobody) >Summary: Unit mask bitmasks containing non-unique values should fail Initial Comment: When named unit mask support was added by Andi Kleen (Intel) in the 0.9.7 timeframe, it became possible for multiple unit masks to have the same hex value for a given event. The (only) way to disambiguate these duplicated unit mask values is to specify them by name. It was pointed out during patch reviews that OR'ing together such unit masks (for bitmask types of unit masks) was problematic. Andi asserted that bitmasks containing any of the duplicated values simly would not be supported. Unfortunately, we dropped the ball and did not document this restriction, and neither did we put any checks into the code to prevent users from doing something like the following: On i386/sandybridge, the int_misc event has the following possible unit mask values that can be put into a bitmask: Unit masks (default 0x40) ---------- 0x40: rat_stall_cycles Cycles Resource Allocation Table (RAT) external stall is sent to Instruction Decode Queue (IDQ) for this thread. 0x03: recovery_cycles Number of cycles waiting to be recover after Nuke due to all other cases except JEClear. (extra: cmask=1) 0x03: recovery_stalls_count Edge applied to recovery_cycles, thus counts occurrences. (extra: edge cmask=1) The event specification of int_misc:2000000:43 will be incorrectly accepted as valid even though i it is clearly ambiguous. Such unit mask bitmask specifications should be detected, and we should exit gracefully with an informative error message. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&group_id=16191 |
From: SourceForge.net <no...@so...> - 2013-01-08 18:36:28
|
Bugs item #3599530, was opened at 2013-01-04 14:04 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&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: Maynard Johnson (maynardj) Assigned to: Nobody/Anonymous (nobody) Summary: Unit mask bitmasks containing non-unique values should fail Initial Comment: When named unit mask support was added by Andi Kleen (Intel) in the 0.9.7 timeframe, it became possible for multiple unit masks to have the same hex value for a given event. The (only) way to disambiguate these duplicated unit mask values is to specify them by name. It was pointed out during patch reviews that OR'ing together such unit masks (for bitmask types of unit masks) was problematic. Andi asserted that bitmasks containing any of the duplicated values simly would not be supported. Unfortunately, we dropped the ball and did not document this restriction, and neither did we put any checks into the code to prevent users from doing something like the following: On i386/sandybridge, the int_misc event has the following possible unit mask values that can be put into a bitmask: Unit masks (default 0x40) ---------- 0x40: rat_stall_cycles Cycles Resource Allocation Table (RAT) external stall is sent to Instruction Decode Queue (IDQ) for this thread. 0x03: recovery_cycles Number of cycles waiting to be recover after Nuke due to all other cases except JEClear. (extra: cmask=1) 0x03: recovery_stalls_count Edge applied to recovery_cycles, thus counts occurrences. (extra: edge cmask=1) The event specification of int_misc:2000000:43 will be incorrectly accepted as valid even though i it is clearly ambiguous. Such unit mask bitmask specifications should be detected, and we should exit gracefully with an informative error message. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2013-01-08 10:36 Message: A patch was posted for review on Jan 4. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&group_id=16191 |
From: SourceForge.net <no...@so...> - 2013-01-29 21:20:21
|
Bugs item #3599530, was opened at 2013-01-04 14:04 Message generated for change (Comment added) made by maynardj You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&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: Maynard Johnson (maynardj) Assigned to: Nobody/Anonymous (nobody) Summary: Unit mask bitmasks containing non-unique values should fail Initial Comment: When named unit mask support was added by Andi Kleen (Intel) in the 0.9.7 timeframe, it became possible for multiple unit masks to have the same hex value for a given event. The (only) way to disambiguate these duplicated unit mask values is to specify them by name. It was pointed out during patch reviews that OR'ing together such unit masks (for bitmask types of unit masks) was problematic. Andi asserted that bitmasks containing any of the duplicated values simly would not be supported. Unfortunately, we dropped the ball and did not document this restriction, and neither did we put any checks into the code to prevent users from doing something like the following: On i386/sandybridge, the int_misc event has the following possible unit mask values that can be put into a bitmask: Unit masks (default 0x40) ---------- 0x40: rat_stall_cycles Cycles Resource Allocation Table (RAT) external stall is sent to Instruction Decode Queue (IDQ) for this thread. 0x03: recovery_cycles Number of cycles waiting to be recover after Nuke due to all other cases except JEClear. (extra: cmask=1) 0x03: recovery_stalls_count Edge applied to recovery_cycles, thus counts occurrences. (extra: edge cmask=1) The event specification of int_misc:2000000:43 will be incorrectly accepted as valid even though i it is clearly ambiguous. Such unit mask bitmask specifications should be detected, and we should exit gracefully with an informative error message. ---------------------------------------------------------------------- >Comment By: Maynard Johnson (maynardj) Date: 2013-01-29 13:20 Message: The patch was committed on Jan 23. ---------------------------------------------------------------------- Comment By: Maynard Johnson (maynardj) Date: 2013-01-08 10:36 Message: A patch was posted for review on Jan 4. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=116191&aid=3599530&group_id=16191 |