January 2024 - Dev - lists.ceph.io

by Yuri Weinstein

Details of this release are summarized here: https://tracker.ceph.com/issues/64151#note-1 Seeking approvals/reviews for: rados - Radek, Laura, Travis, Ernesto, Adam King rgw - Casey fs - Venky rbd - Ilya krbd - in progress upgrade/nautilus-x (pacific) - Casey PTL (regweed tests failed) upgrade/octopus-x (pacific) - Casey PTL (regweed tests failed) upgrade/pacific-x (quincy) - in progress upgrade/pacific-p2p - Ilya PTL (maybe rbd related?) ceph-volume - Guillaume TIA YuriW

2 months

12
27
0 0

arrow submodule version

by Casey Bodley

hey Gal and Eric, in today's standup, we discussed the version of our apache arrow submodule. it's currently pinned at 6.0.1, which was tagged in nov. 2021. the centos9 builds are using the system package libarrow-devel-9.0.0. arrow's upstream recently tagged an 11.0.0 release as far as i know, there still aren't any system packages for ubuntu, so we're likely to be stuck with the submodule for quite a while. how do guys want to handle these updates? is it worth trying to update before the reef release?

2 months, 2 weeks

4
7
0 0

New milestones for reef and quincy

by Yuri Weinstein

Hello Dev Leads - just a friendly reminder that we've defined new milestones for reef and quincy releases: 18.2.2 and 17.2.8 Please assign the PRs that are to be tested and merged as part of those milestones and add the "needs-qa" label. TIA

2 months, 4 weeks

5
6
0 0

1/25/2024 perf meeting is canceled

by Mark Nelson

Hi Folks, I wanted to send out a reminder that the performance meeting is canceled this week as I'm out on travel. See you next week! Thanks, Mark -- Best Regards, Mark Nelson Head of R&D (USA) Clyso GmbH p: +49 89 21552391 12 a: Loristraße 8 | 80335 München | Germany w: https://clyso.com | e: mark.nelson(a)clyso.com We are hiring: https://www.clyso.com/jobs/

3 months

1
0
0 0

CLT meeting notes January 24th 2024

by Adam King

- Build/package PRs- who to best review these? - Example: https://github.com/ceph/ceph/pull/55218 - Idea: create a GitHub team specifically for these types of PRs https://github.com/orgs/ceph/teams - Laura will try to organize people for the group - Pacific 16.2.15 status - Handful of PRs left in 16.2.15 tag https://github.com/ceph/ceph/pulls?q=is%3Apr+is%3Aopen+milestone%3Av16.2.15 that still need to be tested and merged - Yuri will begin testing RC after that

3 months

1
0
0 0

1/18/2023 perf meeting is on!

by Mark Nelson

Hi Folks, Meeting starts in 5 minutes! Today we'll be covering a couple of PRs, but otherwise have no specific agenda set. I suspect it'll be a pretty quick meeting. Please feel free to add a topic if there's anything folks would like to discuss. Since I've been having so many issues with jitsi in the past month, I'm going to switch the meeting this week over to google meet this week: https://meet.google.com/cbq-qyur-cke Thanks, Mark Etherpad: https://pad.ceph.com/p/performance_weekly Meeting URL: https://meet.google.com/cbq-qyur-cke Mark -- Best Regards, Mark Nelson Head of R&D (USA) Clyso GmbH p: +49 89 21552391 12 a: Loristraße 8 | 80335 München | Germany w: https://clyso.com | e: mark.nelson(a)clyso.com We are hiring: https://www.clyso.com/jobs/

3 months, 1 week

1
0
0 0

Join us for the User + Dev Monthly Meetup - January 18th!

by Laura Flores

Hi Ceph users and developers, You are invited to join us at the User + Dev meeting this week Thursday, January 18th at 10:00 AM Eastern Time! See below for more meeting details. The focus topic, "Ceph Feature Request from the DKIST Data Center: Add a service backed by tape that is analogous to AWS Glacier", will be presented by Joel Davidow, a Ceph operator from the National Solar Observatory. In his talk, he will propose a feature request to add support for tape as a storage class with lifecycle management for object storage. Feel free to add questions or additional topics under the "Open Discussion" section on the agenda: https://pad.ceph.com/p/ceph-user-dev-monthly-minutes If you have an idea for a focus topic you'd like to present at a future meeting, you are welcome to submit it to this Google Form: https://docs.google.com/forms/d/e/1FAIpQLSdboBhxVoBZoaHm8xSmeBoemuXoV_rmh4v… Any Ceph user or developer is eligible to submit! Thanks, Laura Flores Meeting link: https://meet.jit.si/ceph-user-dev-monthly Time Conversions: UTC: Thursday, January 18, 15:00 UTC Mountain View, CA, US: Thursday, January 18, 7:00 PST Phoenix, AZ, US: Thursday, January 18, 8:00 MST Denver, CO, US: Thursday, January 18, 8:00 MST Huntsville, AL, US: Thursday, January 18, 9:00 CST Raleigh, NC, US: Thursday, January 18, 10:00 EST London, England: Thursday, January 18, 15:00 GMT Paris, France: Thursday, January 18, 16:00 CET Helsinki, Finland: Thursday, January 18, 17:00 EET Tel Aviv, Israel: Thursday, January 18, 17:00 IST Pune, India: Thursday, January 18, 20:30 IST Brisbane, Australia: Friday, January 19, 1:00 AEST Singapore, Asia: Thursday, January 18, 23:00 +08 Auckland, New Zealand: Friday, January 19, 4:00 NZDT -- Laura Flores She/Her/Hers Software Engineer, Ceph Storage <https://ceph.io> Chicago, IL lflores(a)ibm.com | lflores(a)redhat.com <lflores(a)redhat.com> M: +17087388804

3 months, 1 week

1
2
0 0

Re: Make rgw use tcmalloc/jemalloc

by Yixin Jin

Hi Casey, I have always thought the official ceph radosgw build links with tcmalloc. Your early response confirmed that and our older servers (running v15) also confirmed that. However, today, I actually download radosgw Debian packages (17.2.7, 17.2.6 and 17.2.5) from download.ceph.com <http://download.ceph.com/>. Unpack them and then find radosgw isn’t linked with tcmalloc. ldd radosgw shows no trace of it. I even checked 15.1.0 and 15.2.9. None of them have radosgw linked with tcmalloc. Am I getting the packages from the wrong source? Thanks, Yixin PS. I downloaded packages for ubuntu focal release.

3 months, 1 week

2
1
0 0

Fwd: New Defects reported by Coverity Scan for ceph

by Kaleb Keithley

---------- Forwarded message --------- From: <scan-admin(a)coverity.com> Date: Fri, Jan 12, 2024 at 4:15 PM Subject: New Defects reported by Coverity Scan for ceph To: <kkeithle(a)redhat.com> Hi, Please find the latest report on new defect(s) introduced to ceph found with Coverity Scan. 552 new defect(s) introduced to ceph found with Coverity Scan. 300 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 20 of 552 defect(s) ** CID 1585278: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1219 in std::__equal_aux<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() ________________________________________________________________________________________________________ *** CID 1585278: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1219 in std::__equal_aux<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() 1213 template<typename _II1, typename _II2> 1214 _GLIBCXX20_CONSTEXPR 1215 inline bool 1216 __equal_aux(_II1 __first1, _II1 __last1, _II2 __first2) 1217 { 1218 return std::__equal_aux1(std::__niter_base(__first1), >>> CID 1585278: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "__last1" is passed-by-value as parameter to "__niter_base" when it could be moved instead. 1219 std::__niter_base(__last1), 1220 std::__niter_base(__first2)); 1221 } 1222 1223 template<typename _II1, typename _Seq1, typename _Cat1, typename _II2> 1224 bool ** CID 1585277: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_rados.cc: 9898 in RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, unsigned long, unsigned long, optional_yield)() ________________________________________________________________________________________________________ *** CID 1585277: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_rados.cc: 9898 in RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, unsigned long, unsigned long, optional_yield)() 9892 int RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *dpp, const string& oid, const string& user, const string& bucket, 9893 uint64_t start_epoch, uint64_t end_epoch, optional_yield y) 9894 { 9895 rgw_raw_obj obj(svc.zone->get_zone_params().usage_log_pool, oid); 9896 9897 rgw_rados_ref ref; >>> CID 1585277: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "obj" is passed-by-value as parameter to "get_raw_obj_ref" when it could be moved instead. 9898 int r = get_raw_obj_ref(dpp, obj, &ref); 9899 if (r < 0) { 9900 return r; 9901 } 9902 9903 r = cls_rgw_usage_log_trim_repeat(dpp, ref, user, bucket, start_epoch, end_epoch, y); ** CID 1585276: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/test/librbd/test_internal.cc: 1789 in librbd::TestInternal_SparsifyClone_Test::TestBody()() ________________________________________________________________________________________________________ *** CID 1585276: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/test/librbd/test_internal.cc: 1789 in librbd::TestInternal_SparsifyClone_Test::TestBody()() 1783 1784 bufferptr read_ptr(bl.length()); 1785 bufferlist read_bl; 1786 read_bl.push_back(read_ptr); 1787 1788 librbd::io::ReadResult read_result{&read_bl}; >>> CID 1585276: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "read_result" is passed-by-value as parameter to "read" when it could be moved instead. 1789 ASSERT_EQ((ssize_t)read_bl.length(), 1790 api::Io<>::read(*ictx, (1 << ictx->order) * 10, read_bl.length(), 1791 librbd::io::ReadResult{read_result}, 0)); 1792 ASSERT_TRUE(bl.contents_equal(read_bl)); 1793 1794 std::string oid = ictx->get_object_name(0); ** CID 1585275: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_bucket_sync.cc: 584 in RGWBucketSyncFlowManager::init(const DoutPrefixProvider *, const rgw_sync_policy_info &)::[lambda(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) (instance 1)]::operator ()(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) const() ________________________________________________________________________________________________________ *** CID 1585275: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_bucket_sync.cc: 584 in RGWBucketSyncFlowManager::init(const DoutPrefixProvider *, const rgw_sync_policy_info &)::[lambda(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) (instance 1)]::operator ()(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) const() 578 const rgw_zone_id& dest_zone, 579 std::optional<rgw_bucket> dest_bucket) { 580 if (!parent) { 581 return true; 582 } 583 return parent->allowed_data_flow(source_zone, >>> CID 1585275: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "source_bucket" is passed-by-value as parameter to "allowed_data_flow" when it could be moved instead. 584 source_bucket, 585 dest_zone, 586 dest_bucket, 587 false); /* just check that it's not disabled */ 588 }); 589 } ** CID 1585274: (ORDER_REVERSAL) ________________________________________________________________________________________________________ *** CID 1585274: (ORDER_REVERSAL) /src/static/src/github/ceph/src/client/Client.cc: 12306 in Client::statfs(const char *, statvfs *, const UserPerm &)() 12300 // respect that one instead. 12301 ceph_assert(root != nullptr); 12302 InodeRef quota_root = root->quota.is_enabled(QUOTA_MAX_BYTES) ? root : get_quota_root(root.get(), perms, QUOTA_MAX_BYTES); 12303 12304 // get_quota_root should always give us something if client quotas are 12305 // enabled >>> CID 1585274: (ORDER_REVERSAL) >>> Calling "get_val" acquires lock "ceph::common::ConfigProxy.lock" while holding lock "std::unique_lock<std::mutex>._M_device" (count: 2 / 5). 12306 ceph_assert(cct->_conf.get_val<bool>("client_quota") == false || quota_root != nullptr); 12307 12308 /* If bytes quota is set on a directory and conf option "client quota df" 12309 * is also set, available space = quota limit - used space. Else, 12310 * available space = total space - used space. */ 12311 if (quota_root && cct->_conf->client_quota_df && quota_root->quota.max_bytes) { /src/static/src/github/ceph/src/client/Client.cc: 12302 in Client::statfs(const char *, statvfs *, const UserPerm &)() 12296 stbuf->f_namemax = NAME_MAX; 12297 12298 // Usually quota_root will == root_ancestor, but if the mount root has no 12299 // quota but we can see a parent of it that does have a quota, we'll 12300 // respect that one instead. 12301 ceph_assert(root != nullptr); >>> CID 1585274: (ORDER_REVERSAL) >>> Calling "get_quota_root" acquires lock "ceph::common::ConfigProxy.lock" while holding lock "std::unique_lock<std::mutex>._M_device" (count: 2 / 5). 12302 InodeRef quota_root = root->quota.is_enabled(QUOTA_MAX_BYTES) ? root : get_quota_root(root.get(), perms, QUOTA_MAX_BYTES); 12303 12304 // get_quota_root should always give us something if client quotas are 12305 // enabled 12306 ceph_assert(cct->_conf.get_val<bool>("client_quota") == false || quota_root != nullptr); 12307 ** CID 1585273: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sync.h: 294 in RGWMetaSyncStatusManager::read_master_log_shards_next(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::map<int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::less<int>, std::allocator<std::pair<const int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>>>, std::map<int, rgw_mdlog_shard_data, std::less<int>, std::allocator<std::pair<const int, rgw_mdlog_shard_data>>>*)() ________________________________________________________________________________________________________ *** CID 1585273: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sync.h: 294 in RGWMetaSyncStatusManager::read_master_log_shards_next(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::map<int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::less<int>, std::allocator<std::pair<const int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>>>, std::map<int, rgw_mdlog_shard_data, std::less<int>, std::allocator<std::pair<const int, rgw_mdlog_shard_data>>>*)() 288 return master_log.read_log_info(dpp, log_info); 289 } 290 int read_master_log_shards_info(const DoutPrefixProvider *dpp, const std::string& master_period, std::map<int, RGWMetadataLogInfo> *shards_info) { 291 return master_log.read_master_log_shards_info(dpp, master_period, shards_info); 292 } 293 int read_master_log_shards_next(const DoutPrefixProvider *dpp, const std::string& period, std::map<int, std::string> shard_markers, std::map<int, rgw_mdlog_shard_data> *result) { >>> CID 1585273: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "shard_markers" is passed-by-value as parameter to "read_master_log_shards_next" when it could be moved instead. 294 return master_log.read_master_log_shards_next(dpp, period, shard_markers, result); 295 } 296 297 int run(const DoutPrefixProvider *dpp, optional_yield y) { return master_log.run_sync(dpp, y); } 298 299 ** CID 1585272: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sal_rados.cc: 536 in rgw::sal::RadosBucket::read_stats_async(const DoutPrefixProvider *, const rgw::bucket_index_layout_generation &, int, boost::intrusive_ptr<rgw::sal::ReadStatsCB>)() ________________________________________________________________________________________________________ *** CID 1585272: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sal_rados.cc: 536 in rgw::sal::RadosBucket::read_stats_async(const DoutPrefixProvider *, const rgw::bucket_index_layout_generation &, int, boost::intrusive_ptr<rgw::sal::ReadStatsCB>)() 530 } 531 532 int RadosBucket::read_stats_async(const DoutPrefixProvider *dpp, 533 const bucket_index_layout_generation& idx_layout, 534 int shard_id, boost::intrusive_ptr<ReadStatsCB> ctx) 535 { >>> CID 1585272: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "ctx" is passed-by-value as parameter to "get_bucket_stats_async" when it could be moved instead. 536 return store->getRados()->get_bucket_stats_async(dpp, get_info(), idx_layout, shard_id, ctx); 537 } 538 539 int RadosBucket::sync_user_stats(const DoutPrefixProvider *dpp, optional_yield y, 540 RGWBucketEnt* ent) 541 { ** CID 1585271: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/MirrorDaemonServiceInfo.cc: 104 in rbd::MirrorDaemonServiceInfo::get_mirror_service_dump()() ________________________________________________________________________________________________________ *** CID 1585271: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/MirrorDaemonServiceInfo.cc: 104 in rbd::MirrorDaemonServiceInfo::get_mirror_service_dump()() 98 int MirrorDaemonServiceInfo::get_mirror_service_dump() { 99 librados::Rados rados(m_io_ctx); 100 std::string cmd = R"({"prefix": "service dump", "format": "json"})"; 101 bufferlist in_bl; 102 bufferlist out_bl; 103 >>> CID 1585271: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "cmd" is passed-by-value as parameter to "mon_command" when it could be moved instead. 104 int r = rados.mon_command(cmd, in_bl, &out_bl, nullptr); 105 if (r < 0) { 106 std::cerr << "rbd: failed to query services: " << cpp_strerror(r) 107 << std::endl; 108 return r; 109 } ** CID 1585270: Uninitialized variables (USE_AFTER_MOVE) /src/static/src/github/ceph/src/include/neorados/RADOS.hpp: 1697 in neorados::RADOS::enumerate_objects<boost::asio::redirect_error_t<boost::asio::use_awaitable_t<boost::asio::any_io_executor>>>(neorados::IOContext, neorados::Cursor, neorados::Cursor, unsigned int, ceph::buffer::v15_2_0::list, T1 &&)() ________________________________________________________________________________________________________ *** CID 1585270: Uninitialized variables (USE_AFTER_MOVE) /src/static/src/github/ceph/src/include/neorados/RADOS.hpp: 1697 in neorados::RADOS::enumerate_objects<boost::asio::redirect_error_t<boost::asio::use_awaitable_t<boost::asio::any_io_executor>>>(neorados::IOContext, neorados::Cursor, neorados::Cursor, unsigned int, ceph::buffer::v15_2_0::list, T1 &&)() 1691 using EnumerateComp = boost::asio::any_completion_handler<EnumerateSig>; 1692 template<boost::asio::completion_token_for<EnumerateSig> CompletionToken> 1693 auto enumerate_objects(IOContext ioc, Cursor begin, 1694 Cursor end, std::uint32_t max, 1695 ceph::buffer::list filter, 1696 CompletionToken&& token) { >>> CID 1585270: Uninitialized variables (USE_AFTER_MOVE) >>> "token" is used after it has been already moved. 1697 auto consigned = boost::asio::consign( 1698 std::forward<CompletionToken>(token), boost::asio::make_work_guard( 1699 boost::asio::get_associated_executor(token, get_executor()))); 1700 return boost::asio::async_initiate<decltype(consigned), EnumerateSig>( 1701 [ioc = std::move(ioc), begin = std::move(begin), end = std::move(end), 1702 max, filter = std::move(filter), this](auto&& handler) mutable { ** CID 1585269: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/action/Perf.cc: 125 in rbd::action::perf::<unnamed>::query_iostats(librados::v14_2_0::Rados &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, rbd::action::perf::<unnamed>::StatDescriptor, std::vector<rbd::action::perf::<unnamed>::ImageStat, std::allocator<rbd::action::perf::<unnamed>::ImageStat>> *, std::basic_ostream<char, std::char_traits<char>>&)() ________________________________________________________________________________________________________ *** CID 1585269: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/action/Perf.cc: 125 in rbd::action::perf::<unnamed>::query_iostats(librados::v14_2_0::Rados &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, rbd::action::perf::<unnamed>::StatDescriptor, std::vector<rbd::action::perf::<unnamed>::ImageStat, std::allocator<rbd::action::perf::<unnamed>::ImageStat>> *, std::basic_ostream<char, std::char_traits<char>>&)() 119 "format": "json" 120 }")"; 121 122 bufferlist in_bl; 123 bufferlist out_bl; 124 std::string outs; >>> CID 1585269: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "cmd" is passed-by-value as parameter to "mgr_command" when it could be moved instead. 125 int r = rados.mgr_command(cmd, in_bl, &out_bl, &outs); 126 if (r == -EOPNOTSUPP) { 127 err_os << "rbd: 'rbd_support' mgr module is not enabled." 128 << std::endl << std::endl 129 << "Use 'ceph mgr module enable rbd_support' to enable." 130 << std::endl; ** CID 1585268: Error handling issues (UNCAUGHT_EXCEPT) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.h: 195 in Scrub::IntervalChanged::~IntervalChanged()() ________________________________________________________________________________________________________ *** CID 1585268: Error handling issues (UNCAUGHT_EXCEPT) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.h: 195 in Scrub::IntervalChanged::~IntervalChanged()() 189 * This event is distinct from FullReset because replicas are always responsible 190 * for releasing any interval specific state (including but certainly not limited to 191 * scrub reservations) upon interval change, without coordination from the 192 * Primary. This event notifies the ScrubMachine that it can forget about 193 * such remote state. 194 */ >>> CID 1585268: Error handling issues (UNCAUGHT_EXCEPT) >>> An exception of type "boost::container::length_error_t" is thrown but the exception specification "/*implicit*/noexcept" doesn't allow it to be thrown. This will result in a call to terminate(). 195 MEV(IntervalChanged) 196 197 /** 198 * stops the scrubbing session, and resets the scrubber. 199 * For a replica - aborts the handling of the current request. 200 * In both cases - a transition to the peering mode quiescent state (i.e. ** CID 1585267: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.cc: 180 in Scrub::Session::Session(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)() ________________________________________________________________________________________________________ *** CID 1585267: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.cc: 180 in Scrub::Session::Session(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)() 174 scrbr->clear_pgscrub_state(); 175 } 176 177 // ----------------------- Session ----------------------------------------- 178 179 Session::Session(my_context ctx) >>> CID 1585267: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "ctx" is passed-by-value as parameter to "boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::state(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)" when it could be moved instead. 180 : my_base(ctx) 181 , NamedSimply(context<ScrubMachine>().m_scrbr, "PrimaryActive/Session") 182 { 183 dout(10) << "-- state -->> PrimaryActive/Session" << dendl; 184 DECLARE_LOCALS; // 'scrbr' & 'pg_id' aliases 185 ** CID 1585266: Memory - illegal accesses (WRAPPER_ESCAPE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_gc.cc: 231 in RGWGC::remove(int, const std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>> &, librados::v14_2_0::AioCompletion **, optional_yield)() ________________________________________________________________________________________________________ *** CID 1585266: Memory - illegal accesses (WRAPPER_ESCAPE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_gc.cc: 231 in RGWGC::remove(int, const std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>> &, librados::v14_2_0::AioCompletion **, optional_yield)() 225 ObjectWriteOperation op; 226 cls_rgw_gc_remove(op, tags); 227 228 aio_completion_ptr c{librados::Rados::aio_create_completion(nullptr, nullptr)}; 229 int ret = store->gc_aio_operate(obj_names[index], c.get(), &op); 230 if (ret >= 0) { >>> CID 1585266: Memory - illegal accesses (WRAPPER_ESCAPE) >>> The internal representation of local "c" escapes into "*pc", but is destroyed when it exits scope. 231 *pc = c.get(); 232 c.release(); 233 } 234 return ret; 235 } 236 ** CID 1585265: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_data_sync.cc: 1940 in RGWDataIncSyncShardCR::RGWDataIncSyncShardCR(RGWDataSyncCtx *, const rgw_pool &, unsigned int, rgw_data_sync_marker &, std::shared_ptr<RGWSyncTraceNode>, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const rgw_raw_obj &, boost::intrusive_ptr<const RGWContinuousLeaseCR>, const rgw_data_sync_status &, RGWObjVersionTracker &, const boost::intrusive_ptr<rgw::bucket_sync::Cache> &, std::mutex &, boost::container::flat_set<rgw_data_notify_entry, std::less<rgw_data_notify_entry>, void> &)() ________________________________________________________________________________________________________ *** CID 1585265: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_data_sync.cc: 1940 in RGWDataIncSyncShardCR::RGWDataIncSyncShardCR(RGWDataSyncCtx *, const rgw_pool &, unsigned int, rgw_data_sync_marker &, std::shared_ptr<RGWSyncTraceNode>, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const rgw_raw_obj &, boost::intrusive_ptr<const RGWContinuousLeaseCR>, const rgw_data_sync_status &, RGWObjVersionTracker &, const boost::intrusive_ptr<rgw::bucket_sync::Cache> &, std::mutex &, boost::container::flat_set<rgw_data_notify_entry, std::less<rgw_data_notify_entry>, void> &)() 1934 const string& status_oid, const rgw_raw_obj& error_repo, 1935 boost::intrusive_ptr<const RGWContinuousLeaseCR> lease_cr, 1936 const rgw_data_sync_status& sync_status, RGWObjVersionTracker& objv, 1937 const boost::intrusive_ptr<rgw::bucket_sync::Cache>& bucket_shard_cache, 1938 ceph::mutex& inc_lock, 1939 bc::flat_set<rgw_data_notify_entry>& modified_shards) >>> CID 1585265: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "tn" is passed-by-value as parameter to "RGWDataBaseSyncShardCR::RGWDataBaseSyncShardCR(RGWDataSyncCtx *, rgw_pool const &, uint32_t, rgw_data_sync_marker &, RGWSyncTraceNodeRef, std::string const &, rgw_raw_obj const &, boost::intrusive_ptr<RGWContinuousLeaseCR const>, rgw_data_sync_status const &, RGWObjVersionTracker &, boost::intrusive_ptr<rgw::bucket_sync::Cache> const &)" when it could be moved instead. 1940 : RGWDataBaseSyncShardCR(sc, pool, shard_id, sync_marker, tn, 1941 status_oid, error_repo, std::move(lease_cr), 1942 sync_status, objv, bucket_shard_cache), 1943 inc_lock(inc_lock), modified_shards(modified_shards) {} 1944 1945 int operate(const DoutPrefixProvider *dpp) override { ** CID 1585264: Uninitialized variables (MISSING_RETURN) /src/static/src/github/ceph/src/test/neorados/read_operations.cc: 268 in NeoRadosReadOps_Checksum_Test::CoTestBody()() ________________________________________________________________________________________________________ *** CID 1585264: Uninitialized variables (MISSING_RETURN) /src/static/src/github/ceph/src/test/neorados/read_operations.cc: 268 in NeoRadosReadOps_Checksum_Test::CoTestBody()() 262 co_await execute(oid, ReadOp{}.checksum(hash_alg::crc32c, seed, 0, 263 data.length(), 4, &crc)); 264 EXPECT_EQ(crc32c(seed, data.substr(0, 4)), crc[0]); 265 EXPECT_EQ(crc32c(seed, data.substr(4, 4)), crc[1]); 266 } 267 >>> CID 1585264: Uninitialized variables (MISSING_RETURN) >>> Arriving at the end of a function without returning a value. 268 co_return; 269 } 270 271 CORO_TEST_F(NeoRadosReadOps, RWOrderedRead, ReadOpTest) { 272 buffer::list bl; 273 sys::error_code ec; ** CID 1585263: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/json/impl/parser.ipp: 56 in boost::json::parser::reset(boost::json::storage_ptr)() ________________________________________________________________________________________________________ *** CID 1585263: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/json/impl/parser.ipp: 56 in boost::json::parser::reset(boost::json::storage_ptr)() 50 51 void 52 parser:: 53 reset(storage_ptr sp) noexcept 54 { 55 p_.reset(); >>> CID 1585263: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "sp" is passed-by-value as parameter to "reset" when it could be moved instead. 56 p_.handler().st.reset(sp); 57 } 58 59 std::size_t 60 parser:: 61 write_some( ** CID 1585262: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/intrusive/hashtable.hpp: 1172 in boost::intrusive::bucket_plus_vtraits<boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, boost::intrusive::bucket_traits_impl<boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> *, unsigned long>, (bool)0>::priv_erase_from_single_bucket<boost::intrusive::node_cast_adaptor<boost::intrusive::detail::node_disposer<boost::intrusive::detail::null_disposer, boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, (boost::intrusive::algo_types)3>, boost::intrusive::slist_node<void *> *, boost::intrusive::slist_node<void *> *>>(boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> &, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, T1, boost::move_detail::bool_<(bool)0>)() ________________________________________________________________________________________________________ *** CID 1585262: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/intrusive/hashtable.hpp: 1172 in boost::intrusive::bucket_plus_vtraits<boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, boost::intrusive::bucket_traits_impl<boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> *, unsigned long>, (bool)0>::priv_erase_from_single_bucket<boost::intrusive::node_cast_adaptor<boost::intrusive::detail::node_disposer<boost::intrusive::detail::null_disposer, boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, (boost::intrusive::algo_types)3>, boost::intrusive::slist_node<void *> *, boost::intrusive::slist_node<void *> *>>(boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> &, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, T1, boost::move_detail::bool_<(bool)0>)() 1166 } 1167 1168 template<class NodeDisposer> 1169 static std::size_t priv_erase_from_single_bucket 1170 (bucket_type &, siterator sbefore_first, siterator slast, NodeDisposer node_disposer, detail::false_) //optimize multikey 1171 { >>> CID 1585262: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "node_disposer" is passed-by-value as parameter to "unlink_after_and_dispose" when it could be moved instead. 1172 return slist_node_algorithms::unlink_after_and_dispose(sbefore_first.pointed_node(), slast.pointed_node(), node_disposer); 1173 } 1174 1175 template<class NodeDisposer> 1176 static void priv_erase_node(bucket_type &b, siterator i, NodeDisposer node_disposer, detail::true_) //optimize multikey 1177 { ** CID 1585261: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/rgw_sal_filter.cc: 722 in rgw::sal::FilterBucket::get_multipart_upload(const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::optional<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>, ACLOwner, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<(long)1, (long)1000000000>>>)() ________________________________________________________________________________________________________ *** CID 1585261: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/rgw_sal_filter.cc: 722 in rgw::sal::FilterBucket::get_multipart_upload(const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::optional<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>, ACLOwner, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<(long)1, (long)1000000000>>>)() 716 std::unique_ptr<MultipartUpload> FilterBucket::get_multipart_upload( 717 const std::string& oid, 718 std::optional<std::string> upload_id, 719 ACLOwner owner, ceph::real_time mtime) 720 { 721 std::unique_ptr<MultipartUpload> nmu = >>> CID 1585261: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "upload_id" is passed-by-value as parameter to "get_multipart_upload" when it could be moved instead. 722 next->get_multipart_upload(oid, upload_id, owner, mtime); 723 724 return std::make_unique<FilterMultipartUpload>(std::move(nmu), this); 725 } 726 727 int FilterBucket::list_multiparts(const DoutPrefixProvider *dpp, ** CID 1585260: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/dbstore/sqlite/sqliteDB.h: 355 in SQLPutObjectData::SQLPutObjectData(sqlite3 **, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, ceph::common::CephContext *)() ________________________________________________________________________________________________________ *** CID 1585260: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/dbstore/sqlite/sqliteDB.h: 355 in SQLPutObjectData::SQLPutObjectData(sqlite3 **, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, ceph::common::CephContext *)() 349 private: 350 sqlite3 **sdb = NULL; 351 sqlite3_stmt *stmt = NULL; // Prepared statement 352 353 public: 354 SQLPutObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {} >>> CID 1585260: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "db_name" is passed-by-value as parameter to "SQLiteDB::SQLiteDB(sqlite3 *, std::string, ceph::common::CephContext *)" when it could be moved instead. 355 SQLPutObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {} 356 357 ~SQLPutObjectData() { 358 if (stmt) 359 sqlite3_finalize(stmt); 360 } ** CID 1585259: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1555 in std::equal<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() ________________________________________________________________________________________________________ *** CID 1585259: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1555 in std::equal<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() 1549 __glibcxx_function_requires(_InputIteratorConcept<_II2>) 1550 __glibcxx_function_requires(_EqualOpConcept< 1551 typename iterator_traits<_II1>::value_type, 1552 typename iterator_traits<_II2>::value_type>) 1553 __glibcxx_requires_can_increment_range(__first1, __last1, __first2); 1554 >>> CID 1585259: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) >>> "__last1" is passed-by-value as parameter to "__equal_aux" when it could be moved instead. 1555 return std::__equal_aux(__first1, __last1, __first2); 1556 } 1557 1558 /** 1559 * @brief Tests a range for element-wise equality. 1560 * @ingroup non_mutating_algorithms ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P… To manage Coverity Scan email notifications for "kkeithle(a)redhat.com", click https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P… -- Kaleb

3 months, 2 weeks

1
0
0 0

Fwd: Coverity Scan: Analysis completed for ceph

by Kaleb Keithley

Coverity scans are up and running again. https://scan.coverity.com/projects/ceph?tab=overview ---------- Forwarded message --------- From: <scan-admin(a)coverity.com> Date: Fri, Jan 12, 2024 at 4:13 PM Subject: Coverity Scan: Analysis completed for ceph To: <kkeithle(a)redhat.com> Your request for analysis of ceph has been completed successfully. The results are available at https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P… Build ID: 583541 Analysis Summary: New defects found: 1622 Defects eliminated: 300 If you have difficulty understanding any defects, email us at scan-admin(a)coverity.com, or post your question to StackOverflow at https://u15810271.ct.sendgrid.net/ls/click?upn=CTPegkVN6peWFCMEieYYmPWIi1E4… -- Kaleb

3 months, 2 weeks

1
0
0 0

2024

2023

2022

2021

2020

2019

Dev January 2024