Project

General

Profile

Actions

Bug #72077

open

ECTransaction.cc:616:25: warning: comparison of integer expressions of different signedness

Added by Casey Bodley 8 months ago. Updated 7 months ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Backport:
tentacle
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Pull request ID:
Tags (freeform):
backport_processed
Fixed In:
v20.3.0-2314-gbc264090a2
Released In:
Upkeep Timestamp:
2025-08-13T07:11:38+00:00

Description

compile warning with gcc 15.1.1:

[482/563] Building CXX object src/osd/CMakeFiles/osd.dir/ECTransaction.cc.o
ceph/src/osd/ECTransaction.cc: In constructor ‘ECTransaction::Generate::Generate(PGTransaction&, ceph::ErasureCodeInterfaceRef&, pg_t&, const ECUtil::stripe_info_t&, const std::map<hobject_t, ECUtil::shard_extent_map_t>&, std::map<hobject_t, ECUtil::shard_extent_map_t>*, shard_id_map<ceph::os::Transaction>&, const OSDMapRef&, const hobject_t&, PGTransaction::ObjectOperation&, ECTransaction::WritePlanObj&, DoutPrefixProvider*, pg_log_entry_t*)’:
ceph/src/osd/ECTransaction.cc:616:25: warning: comparison of integer expressions of different signedness: ‘int’ and ‘__gnu_cxx::__alloc_traits<std::allocator<unsigned int>, unsigned int>::value_type’ {aka ‘unsigned int’} [-Wsign-compare]
  616 |     if (t.get_num_ops() > old_transaction_counts[int(shard)] &&


Related issues 1 (1 open0 closed)

Copied to RADOS - Backport #72561: tentacle: ECTransaction.cc:616:25: warning: comparison of integer expressions of different signednessIn ProgressAlex AinscowActions
Actions #1

Updated by Radoslaw Zarzynski 8 months ago

  • Assignee set to Alex Ainscow
Actions #2

Updated by Radoslaw Zarzynski 8 months ago

Bump up.

Actions #3

Updated by Radoslaw Zarzynski 8 months ago

Bump up.

Actions #4

Updated by Laura Flores 8 months ago

@Alex Ainscow any update on this one?

Actions #5

Updated by Alex Ainscow 8 months ago · Edited

@Laura Flores This will be fixed in EC fixpack 2: https://github.com/ceph/ceph/pull/64501. This has completed test and I to merge soon.

Actions #6

Updated by Alex Ainscow 7 months ago

  • Status changed from New to Pending Backport
Actions #7

Updated by Alex Ainscow 7 months ago

  • Status changed from Pending Backport to New
Actions #8

Updated by Alex Ainscow 7 months ago

  • Status changed from New to Pending Backport

Sneakily using this tracker to generate a backport tracker

Actions #9

Updated by Alex Ainscow 7 months ago

  • Backport set to tentacle
  • Pull request ID set to 64501
Actions #10

Updated by Upkeep Bot 7 months ago

  • Copied to Backport #72561: tentacle: ECTransaction.cc:616:25: warning: comparison of integer expressions of different signedness added
Actions #11

Updated by Upkeep Bot 7 months ago

  • Tags (freeform) set to backport_processed
Actions #12

Updated by Upkeep Bot 7 months ago

  • Merge Commit set to bc264090a20327a16574bf5892b495097a3c7f3d
  • Fixed In set to v20.3.0-2314-gbc264090a2
  • Upkeep Timestamp set to 2025-08-13T07:11:38+00:00
Actions

Also available in: Atom PDF