mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-21 22:01:31 +02:00
Code Issues Projects Releases Wiki Activity

BUILD: cpuset: do not use const on the source of CPU_AND/CPU_ASSIGN

Browse Source 
The man page indicates that CPU_AND() and CPU_ASSIGN() take a variable,
not a const on the source, even though it doesn't make much sense. But
with older libcs, this triggers a build warning:

  src/cpuset.c: In function 'ha_cpuset_and':
  src/cpuset.c:53: warning: initialization discards qualifiers from pointer target type
  src/cpuset.c: In function 'ha_cpuset_assign':
  src/cpuset.c:101: warning: initialization discards qualifiers from pointer target type

Better stick stricter to the documented API as this is really harmless
here. There's no need to backport it (unless build issues are reported,
which is quite unlikely).
This commit is contained in:
Willy Tarreau 2022-01-28 09:10:52 +01:00
parent 8da23393a1
commit a65b4933ba
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
include/haproxy/cpuset.h
View File

@ -21,7 +21,7 @@ int ha_cpuset_clr(struct hap_cpuset *set, int cpu);
/* Bitwise and equivalent operation between <src> and <dst> stored in <dst>.
*/
void ha_cpuset_and(struct hap_cpuset *dst, const struct hap_cpuset *src);
void ha_cpuset_and(struct hap_cpuset *dst, struct hap_cpuset *src);
/* Returns the count of set index in <set>.
*/
@ -35,7 +35,7 @@ int ha_cpuset_ffs(const struct hap_cpuset *set);
/* Copy <src> set into <dst>.
*/
void ha_cpuset_assign(struct hap_cpuset *dst, const struct hap_cpuset *src);
void ha_cpuset_assign(struct hap_cpuset *dst, struct hap_cpuset *src);
/* Returns the biggest index plus one usable on the platform.
*/

4
src/cpuset.c
View File

@ -47,7 +47,7 @@ int ha_cpuset_clr(struct hap_cpuset *set, int cpu)
#endif
}
void ha_cpuset_and(struct hap_cpuset *dst, const struct hap_cpuset *src)
void ha_cpuset_and(struct hap_cpuset *dst, struct hap_cpuset *src)
{
#if defined(CPUSET_USE_CPUSET)
CPU_AND(&dst->cpuset, &dst->cpuset, &src->cpuset);
@ -94,7 +94,7 @@ int ha_cpuset_ffs(const struct hap_cpuset *set)
#endif
}
void ha_cpuset_assign(struct hap_cpuset *dst, const struct hap_cpuset *src)
void ha_cpuset_assign(struct hap_cpuset *dst, struct hap_cpuset *src)
{
#if defined(CPUSET_USE_CPUSET)
CPU_ZERO(&dst->cpuset);