Ceph : Issueshttps://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2017-08-24T13:15:51ZCeph
Redmine Ceph - Bug #21094 (Closed): warning on rawhide group_pool uninitializedhttps://tracker.ceph.com/issues/210942017-08-24T13:15:51ZAmit Kumar
<p>In file included from /root/ceph/src/cls/rbd/cls_rbd_types.cc:5:0:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h: In member function ‘void cls::rbd::SnapshotNamespaceOnDisk::decode(ceph::buffer::list::iterator&)’:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘*((void*)(& temp)+8).cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />[ 20%] Building CXX object src/cls/CMakeFiles/cephd_cls_base.dir/rbd/cls_rbd_client.cc.o</p> Ceph - Bug #21093 (Closed): compilation warnings on rawhide from /root/ceph/src/zstd/lib https://tracker.ceph.com/issues/210932017-08-24T12:52:01ZAmit Kumar
<p>-- Build files have been written to: /root/ceph/build/src/compressor/zstd/libzstd<br />[ 3%] Building C object src/lua/CMakeFiles/liblua.dir/src/lapi.c.o<br />[ 4%] Performing forcebuild step for 'zstd_ext'<br />[ 4%] Performing build step for 'zstd_ext'<br />Scanning dependencies of target libzstd_static<br />[ 6%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/common/entropy_common.c.o<br />[ 4%] Building C object src/lua/CMakeFiles/liblua.dir/src/lcode.c.o<br />[ 13%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/common/zstd_common.c.o<br />[ 20%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/common/error_private.c.o<br />[ 26%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/common/xxhash.c.o<br />[ 33%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/common/fse_decompress.c.o<br />In file included from /root/ceph/src/zstd/lib/common/fse_decompress.c:63:0:<br />/root/ceph/src/zstd/lib/common/bitstream.h: In function ‘BIT_initDStream’:<br />/root/ceph/src/zstd/lib/common/bitstream.h:277:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 7: bitD->bitContainer <ins>= (size_t)(((const BYTE*)(srcBuffer))[6]) << (sizeof(bitD->bitContainer)*8 - 16);<br /> ~<sub>~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/common/bitstream.h:278:13: note: here<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/common/bitstream.h:278:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/common/bitstream.h:279:13: note: here<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/common/bitstream.h:279:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/common/bitstream.h:280:13: note: here<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/common/bitstream.h:280:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/common/bitstream.h:281:13: note: here<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/common/bitstream.h:281:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/common/bitstream.h:282:13: note: here<br /> case 2: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[1]) << 8;<br /> ^<sub>~~<br />[ 40%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/compress/fse_compress.c.o<br />[ 46%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/compress/huf_compress.c.o<br />/root/ceph/src/zstd/lib/compress/huf_compress.c: In function ‘HUF_compress1X_usingCTable’:<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:425:8: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> if (sizeof((stream)->bitContainer)*8 < HUF_TABLELOG_MAX*4+7) HUF_FLUSHBITS(stream)<br /> ^<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:446:18: note: in expansion of macro ‘HUF_FLUSHBITS_2’<br /> HUF_FLUSHBITS_2(&bitC);<br /> ^~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:447:9: note: here<br /> case 2 : HUF_encodeSymbol(&bitC, ip[n</ins> 1], CTable);<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:422:8: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> if (sizeof((stream)->bitContainer)*8 < HUF_TABLELOG_MAX*2+7) HUF_FLUSHBITS(stream)<br /> ^<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:448:18: note: in expansion of macro ‘HUF_FLUSHBITS_1’<br /> HUF_FLUSHBITS_1(&bitC);<br /> ^~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/compress/huf_compress.c:449:9: note: here<br /> case 1 : HUF_encodeSymbol(&bitC, ip[n+ 0], CTable);<br /> ^<sub>~~<br />[ 53%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/compress/zstd_compress.c.o<br />/root/ceph/src/zstd/lib/compress/zstd_compress.c: In function ‘ZSTD_compressStream_generic’:<br />/root/ceph/src/zstd/lib/compress/zstd_compress.c:3045:28: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> zcs->stage = zcss_flush; /* pass-through to flush stage <strong>/<br /> ~~~~~~~~~~~^~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/compress/zstd_compress.c:3048:9: note: here<br /> case zcss_flush:<br /> ^<sub>~~<br />[ 4%] Building C object src/lua/CMakeFiles/liblua.dir/src/lctype.c.o<br />[ 4%] Building C object src/lua/CMakeFiles/liblua.dir/src/ldebug.c.o<br />[ 4%] Building C object src/lua/CMakeFiles/liblua.dir/src/ldo.c.o<br />[ 60%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/decompress/huf_decompress.c.o<br />In file included from /root/ceph/src/zstd/lib/decompress/huf_decompress.c:55:0:<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h: In function ‘BIT_initDStream’:<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:277:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 7: bitD->bitContainer += (size_t)(((const BYTE</strong>)(srcBuffer))[6]) << (sizeof(bitD->bitContainer)*8 - 16);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:278:13: note: here<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:278:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:279:13: note: here<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:279:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:280:13: note: here<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:280:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:281:13: note: here<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:281:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:282:13: note: here<br /> case 2: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[1]) << 8;<br /> ^<sub>~~<br />[ 4%] Building C object src/lua/CMakeFiles/liblua.dir/src/ldump.c.o<br />[ 66%] Building C object lib/CMakeFiles/libzstd_static.dir/root/ceph/src/zstd/lib/decompress/zstd_decompress.c.o<br />In file included from /root/ceph/src/zstd/build/cmake/lib/../../../lib/common/fse.h:278:0,<br /> from /root/ceph/src/zstd/lib/decompress/zstd_decompress.c:49:<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h: In function ‘BIT_initDStream’:<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:277:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 7: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[6]) << (sizeof(bitD->bitContainer)*8 - 16);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:278:13: note: here<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:278:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 6: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[5]) << (sizeof(bitD->bitContainer)*8 - 24);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:279:13: note: here<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:279:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 5: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[4]) << (sizeof(bitD->bitContainer)*8 - 32);<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:280:13: note: here<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:280:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 4: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[3]) << 24;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:281:13: note: here<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ^<sub>~~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:281:40: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> case 3: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[2]) << 16;<br /> ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/build/cmake/lib/../../../lib/common/bitstream.h:282:13: note: here<br /> case 2: bitD->bitContainer += (size_t)(((const BYTE*)(srcBuffer))[1]) << 8;<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c: In function ‘ZSTD_decompressContinue’:<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:1531:24: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> dctx->expected = 0; /* not necessary to copy more */<br /> ~~~~~~~~~~~~~~~^</sub>~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:1533:5: note: here<br /> case ZSTDds_decodeFrameHeader:<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c: In function ‘ZSTD_decompressStream’:<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2063:24: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> zds->stage = zdss_read;<br /> ~~~~~~~~~~~^~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2066:9: note: here<br /> case zdss_read:<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2086:28: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> zds->stage = zdss_load;<br /> ~~~~~~~~~~~^~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2090:9: note: here<br /> case zdss_load:<br /> ^<sub>~~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2109:32: warning: this statement may fall through [-Wimplicit-fallthrough=]<br /> zds->stage = zdss_flush;<br /> ~~~~~~~~~~~^~~~~~~~~~</sub>~<br />/root/ceph/src/zstd/lib/decompress/zstd_decompress.c:2113:9: note: here<br /> case zdss_flush:</p> Ceph - Cleanup #21075 (New): Creating type alias for complex type definitionshttps://tracker.ceph.com/issues/210752017-08-23T09:48:11ZAmit Kumar
<p>Example:<br />using pairlist_t = std::vector<std::pair<std::string, int>>;</p>
<p>Everywhere you need to use that type can get cumbersome.</p> Ceph - Cleanup #21044 (Rejected): Replacement of assert() with static_assert() whereever applicablehttps://tracker.ceph.com/issues/210442017-08-21T12:42:16ZAmit Kumar
<p>In some conditions if we can do compile time checks,<br />its better to use static_assert() in place of assert()</p> Ceph - Bug #20839 (Closed): /root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.c...https://tracker.ceph.com/issues/208392017-07-28T06:22:38ZAmit Kumar
<p>[ 15%] Building CXX object src/cls/CMakeFiles/cls_rbd.dir/rbd/cls_rbd_types.cc.o<br />In file included from /root/ceph/src/cls/rbd/cls_rbd_types.cc:5:0:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h: In member function ‘void cls::rbd::SnapshotNamespaceOnDisk::decode(ceph::buffer::list::iterator&)’:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘*((void*)(& temp)+8).cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {</p> Ceph - Bug #20838 (Rejected): warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’...https://tracker.ceph.com/issues/208382017-07-28T06:19:20ZAmit Kumar
<ol>
<li>pwd<br />/root/ceph/build</li>
<li>make -j4<br />[ 10%] Building CXX object src/tools/rbd_mirror/CMakeFiles/rbd_mirror_types.dir/instance_watcher/Types.cc.o<br />In file included from /root/ceph/src/cls/rbd/cls_rbd_types.cc:5:0:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h: In member function ‘void cls::rbd::SnapshotNamespaceOnDisk::decode(ceph::buffer::list::iterator&)’:<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘<anonymous>.cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {<br /> ^<sub>~~~~~~~~~~~~~~~~~~</sub>~<br />/root/ceph/src/cls/rbd/cls_rbd_types.h:248:8: warning: ‘*((void*)(& temp)+8).cls::rbd::GroupSnapshotNamespace::group_pool’ may be used uninitialized in this function [-Wmaybe-uninitialized]<br /> struct GroupSnapshotNamespace {</li>
</ol> Ceph - Cleanup #20811 (Resolved): warning: ‘<<’ in boolean context, did you mean ‘<’ ? [-Wint-in-...https://tracker.ceph.com/issues/208112017-07-28T06:15:02ZAmit Kumar
<ol>
<li>pwd<br />/root/ceph/build</li>
<li>make -j4<br />/root/ceph/src/osd/PGLog.h:29:43: warning: ‘<<’ in boolean context, did you mean ‘<’ ? [-Wint-in-bool-context]<br /> #define PGLOG_INDEXED_DUPS (1 << 3)<br /> ~<sub>~^~~</sub>~<br />/root/ceph/src/osd/PGLog.h:29:43: note: in definition of macro ‘PGLOG_INDEXED_DUPS’<br /> #define PGLOG_INDEXED_DUPS (1 << 3)</li>
</ol> Ceph - Cleanup #20698 (New): Constants in code should be replaced with MACROhttps://tracker.ceph.com/issues/206982017-07-20T11:43:59ZAmit Kumar
<p>I found huge number of constants in code.<br />In future if we want to change the comparison statement, we need to go to every file and change it.<br />Macro at common place may be common/config.h would be good.</p>
<p>Doing this for module ceph/src/os</p> Ceph - Bug #20697 (Closed): dpdk:Correction of comparison statement ceph/src/dpdk/app/test-pmd/cm...https://tracker.ceph.com/issues/206972017-07-20T09:36:00ZAmit Kumar
<p>code is wrongly written. In place of comparison, initialization was done inside if statement.<br />cmd_config_rxtx_queue_parsed(void *parsed_result,<br /> <i>attribute</i>((unused)) struct cmdline *cl,<br /> <i>attribute</i>((unused)) void *data)</p>
<p>if (port_is_started(res->portid) = 1) {<br />}<br />should be replaced with</p>
<p>if (port_is_started(res->portid) == 1) {<br />}</p> Ceph - Fix #20695 (New): http://www.irchelp.org/irchelp/new2irc.html [File not found]https://tracker.ceph.com/issues/206952017-07-20T06:38:02ZAmit Kumar
<p>Link provided on <a class="external" href="http://docs.ceph.com/docs/master/dev/#introduction">http://docs.ceph.com/docs/master/dev/#introduction</a><br />under IRC Section:<br />Internet Relay Chat.<<<click<br /><a class="external" href="http://www.irchelp.org/">http://www.irchelp.org/</a> <br />Get Started <<<< click Give File not found</p> Ceph - Bug #20662 (Closed): ceph version creates tracebackhttps://tracker.ceph.com/issues/206622017-07-18T09:53:18ZAmit Kumar
<ol>
<li>ceph version<br />Traceback (most recent call last):<br /> File "/usr/local/bin/ceph", line 106, in <module><br /> import rados<br />ImportError: No module named rados
#</li>
</ol> Ceph - Bug #19294 (Can't reproduce): Not able to build source code from Steps mentioned in READMEhttps://tracker.ceph.com/issues/192942017-03-17T10:31:10ZAmit Kumar
<p>Steps Followed:
# git clone <a class="external" href="https://github.com/amitkumar50/ceph.git">https://github.com/amitkumar50/ceph.git</a>
# cd ceph/
# ./install-deps.sh
# ./do_cmake.sh
# cd build
# make<br />make: *** No targets specified and no makefile found. Stop.
#</p>