On Mon, 8 Dec 2014, Ken Dreyer wrote:
> We should get this reported to Ceph upstream.
>
> Ceph is bundling an old copy of rocksdb (ugh), and I wonder if this is
> fixed in a later version of rocksdb.
>
> By the way, my hunch is that the reason that Ceph bundles rocksdb is
> because it was not packaged in the distros at the time that Ceph wanted
> to use it.
Exactly.
> Is it now packaged in Ubuntu?
Nope!
Let's pull rocksdb up to a newer version and see if that helps?
sage
>
> --
> You received this bug notification because you are subscribed to ceph in
> Ubuntu.
> Matching subscriptions: ceph
> https://bugs.launchpad.net/bugs/1400299
>
> Title:
> rocksdb ftbfs on ppc64el with -03
>
> Status in ceph package in Ubuntu:
> New
> Status in gcc-4.9 package in Ubuntu:
> New
>
> Bug description:
> The rocksdb library in the ceph source tree fails to build with the
> default -03 optimizer flag on ppc64el.
>
> (vivid-ppc64el)jamespage@kelsey01:~/build-area/ceph-0.87/src/rocksdb$ g++ -DHAVE_CONFIG_H -I. -D_FORTIFY_SOURCE=2 -Wall -Werror -std=c++11 -fPIC -I. -I./include -DOS_LINUX -DROCKSDB_PLATFORM_POSIX -fno-builtin-memcmp -DNDEBUG -DHAVE_JEMALLOC -Woverloaded-virtual -DROCKSDB_ATOMIC_PRESENT -DROCKSDB_FALLOCATE_PRESENT -DROCKSDB_ATOMIC_PRESENT -DROCKSDB_FALLOCATE_PRESENT -DSNAPPY -DZLIB -DBZIP2 -g -O3 -fstack-protector-strong -Wformat -Werror=format-security -std=c++11 -c table/plain_table_reader.cc -fPIE -o table/librocksdb_la-plain_table_reader.o
> In file included from ./table/plain_table_reader.h:14:0,
> from table/plain_table_reader.cc:6:
> ./db/dbformat.h: In member function 'virtual void rocksdb::PlainTableIterator::Next()':
> ./db/dbformat.h:266:67: error: 'parsed_key.rocksdb::ParsedInternalKey::sequence' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> EncodeFixed64(key_ + usize, PackSequenceAndType(s, value_type));
> ^
> table/plain_table_reader.cc:737:23: note: 'parsed_key.rocksdb::ParsedInternalKey::sequence' was declared here
> ParsedInternalKey parsed_key;
> ^
> In file included from ./table/plain_table_reader.h:14:0,
> from table/plain_table_reader.cc:6:
> ./db/dbformat.h:266:67: error: '*((void*)& parsed_key +24)' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> EncodeFixed64(key_ + usize, PackSequenceAndType(s, value_type));
> ^
> table/plain_table_reader.cc:737:23: note: '*((void*)& parsed_key +24)' was declared here
> ParsedInternalKey parsed_key;
> ^
> cc1plus: all warnings being treated as errors
>
> Works OK with -02
>
> (vivid-ppc64el)jamespage@kelsey01:~/build-area/ceph-0.87/src/rocksdb$
> g++ -DHAVE_CONFIG_H -I. -D_FORTIFY_SOURCE=2 -Wall -Werror -std=c++11
> -fPIC -I. -I./include -DOS_LINUX -DROCKSDB_PLATFORM_POSIX -fno-
> builtin-memcmp -DNDEBUG -DHAVE_JEMALLOC -Woverloaded-virtual
> -DROCKSDB_ATOMIC_PRESENT -DROCKSDB_FALLOCATE_PRESENT
> -DROCKSDB_ATOMIC_PRESENT -DROCKSDB_FALLOCATE_PRESENT -DSNAPPY -DZLIB
> -DBZIP2 -g -O2 -fstack-protector-strong -Wformat -Werror=format-
> security -std=c++11 -c table/plain_table_reader.cc -fPIE -o table
> /librocksdb_la-plain_table_reader.o
>
> Full log:
>
> https://launchpadlibrarian.net/191913939/buildlog_ubuntu-vivid-
> ppc64el.ceph_0.87-0ubuntu1_FAILEDTOBUILD.txt.gz
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu/+source/ceph/+bug/1400299/+subscriptions
>
>
On Mon, 8 Dec 2014, Ken Dreyer wrote:
> We should get this reported to Ceph upstream.
>
> Ceph is bundling an old copy of rocksdb (ugh), and I wonder if this is
> fixed in a later version of rocksdb.
>
> By the way, my hunch is that the reason that Ceph bundles rocksdb is
> because it was not packaged in the distros at the time that Ceph wanted
> to use it.
Exactly.
> Is it now packaged in Ubuntu?
Nope!
Let's pull rocksdb up to a newer version and see if that helps?
sage
> /bugs.launchpad .net/bugs/ 1400299 ppc64el) jamespage@ kelsey01: ~/build- area/ceph- 0.87/src/ rocksdb$ g++ -DHAVE_CONFIG_H -I. -D_FORTIFY_SOURCE=2 -Wall -Werror -std=c++11 -fPIC -I. -I./include -DOS_LINUX -DROCKSDB_ PLATFORM_ POSIX -fno-builtin-memcmp -DNDEBUG -DHAVE_JEMALLOC -Woverloaded- virtual -DROCKSDB_ ATOMIC_ PRESENT -DROCKSDB_ FALLOCATE_ PRESENT -DROCKSDB_ ATOMIC_ PRESENT -DROCKSDB_ FALLOCATE_ PRESENT -DSNAPPY -DZLIB -DBZIP2 -g -O3 -fstack- protector- strong -Wformat -Werror= format- security -std=c++11 -c table/plain_ table_reader. cc -fPIE -o table/librocksd b_la-plain_ table_reader. o plain_table_ reader. h:14:0, table_reader. cc:6: :PlainTableIter ator::Next( )': h:266:67: error: 'parsed_ key.rocksdb: :ParsedInternal Key::sequence' may be used uninitialized in this function [-Werror= maybe-uninitial ized] Type(s, value_type)); table_reader. cc:737: 23: note: 'parsed_ key.rocksdb: :ParsedInternal Key::sequence' was declared here plain_table_ reader. h:14:0, table_reader. cc:6: h:266:67: error: '*((void*)& parsed_key +24)' may be used uninitialized in this function [-Werror= maybe-uninitial ized] Type(s, value_type)); table_reader. cc:737: 23: note: '*((void*)& parsed_key +24)' was declared here ppc64el) jamespage@ kelsey01: ~/build- area/ceph- 0.87/src/ rocksdb$ PLATFORM_ POSIX -fno- virtual ATOMIC_ PRESENT -DROCKSDB_ FALLOCATE_ PRESENT ATOMIC_ PRESENT -DROCKSDB_ FALLOCATE_ PRESENT -DSNAPPY -DZLIB protector- strong -Wformat -Werror=format- table_reader. cc -fPIE -o table la-plain_ table_reader. o /launchpadlibra rian.net/ 191913939/ buildlog_ ubuntu- vivid- ceph_0. 87-0ubuntu1_ FAILEDTOBUILD. txt.gz /bugs.launchpad .net/ubuntu/ +source/ ceph/+bug/ 1400299/ +subscriptions
> --
> You received this bug notification because you are subscribed to ceph in
> Ubuntu.
> Matching subscriptions: ceph
> https:/
>
> Title:
> rocksdb ftbfs on ppc64el with -03
>
> Status in ceph package in Ubuntu:
> New
> Status in gcc-4.9 package in Ubuntu:
> New
>
> Bug description:
> The rocksdb library in the ceph source tree fails to build with the
> default -03 optimizer flag on ppc64el.
>
> (vivid-
> In file included from ./table/
> from table/plain_
> ./db/dbformat.h: In member function 'virtual void rocksdb:
> ./db/dbformat.
> EncodeFixed64(key_ + usize, PackSequenceAnd
> ^
> table/plain_
> ParsedInternalKey parsed_key;
> ^
> In file included from ./table/
> from table/plain_
> ./db/dbformat.
> EncodeFixed64(key_ + usize, PackSequenceAnd
> ^
> table/plain_
> ParsedInternalKey parsed_key;
> ^
> cc1plus: all warnings being treated as errors
>
> Works OK with -02
>
> (vivid-
> g++ -DHAVE_CONFIG_H -I. -D_FORTIFY_SOURCE=2 -Wall -Werror -std=c++11
> -fPIC -I. -I./include -DOS_LINUX -DROCKSDB_
> builtin-memcmp -DNDEBUG -DHAVE_JEMALLOC -Woverloaded-
> -DROCKSDB_
> -DROCKSDB_
> -DBZIP2 -g -O2 -fstack-
> security -std=c++11 -c table/plain_
> /librocksdb_
>
> Full log:
>
> https:/
> ppc64el.
>
> To manage notifications about this bug go to:
> https:/
>
>