]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
Revert "MEDIUM: freq-ctr: use explicit-size types for freq-ctr struct"
authorAurelien DARRAGON <adarragon@haproxy.com>
Fri, 24 Oct 2025 07:09:59 +0000 (09:09 +0200)
committerAurelien DARRAGON <adarragon@haproxy.com>
Fri, 24 Oct 2025 07:35:29 +0000 (09:35 +0200)
This reverts commit 4693ee0ff7a5fa4a12ff69b1a33adca142e781ac.
As discussed in GH #3168, this works but it is not the proper way to fix
the issue. See following commits.

include/haproxy/freq_ctr-t.h

index 4eed37aaba7c7de7096c8edef5d7976a25a37104..d5f1a89129dfb411dd1cc491ab6f55098a0e45db 100644 (file)
  * period has to be known by the user. The period is measured in ticks and
  * must be at least 2 ticks long. This form is slightly more CPU intensive for
  * reads than the per-second form as it involves a divide.
- *
- * /!\ any change performed here will impact shm-stats-file mapping because the
- * struct is embedded in shm_stats_file_object struct, so proceed with caution
- * and change shm stats file version if needed. Also, fixed width integer types
- * should be used (for portability)
  */
 struct freq_ctr {
-       FIXED_SIZE(4, unsigned int, curr_tick); /* start date of current period (wrapping ticks) */
-       FIXED_SIZE(4, unsigned int, curr_ctr);  /* cumulated value for current period */
-       FIXED_SIZE(4, unsigned int, prev_ctr);  /* value for last period */
+       unsigned int curr_tick; /* start date of current period (wrapping ticks) */
+       unsigned int curr_ctr; /* cumulated value for current period */
+       unsigned int prev_ctr; /* value for last period */
 };
 
 #endif /* _HAPROXY_FREQ_CTR_T_H */