Browse Source

Revert "Fix race condition while building EH frame header."

This reverts commit 7cdd7d57e6.
gdb-7.7-branch
Cary Coutant 13 years ago
parent
commit
cafdd5697b
  1. 14
      gold/ChangeLog
  2. 6
      gold/ehframe.cc
  3. 7
      gold/ehframe.h

14
gold/ChangeLog

@ -1,3 +1,17 @@
2013-11-14 Cary Coutant <ccoutant@google.com>
Revert patch -- this did not fix the problem, and there is
no race there.
2013-11-14 Cary Coutant <ccoutant@google.com>
PR gold/14860
* ehframe.cc (Eh_frame_hdr::Eh_frame_hdr): Initialize lock_.
(Ehframe_hdr::set_final_data_size): Allocate a Lock.
* ehframe.h (Eh_frame_hdr::record_fde): Hold the lock while
updating fde_offsets_.
(Eh_frame_hdr::lock_): New data member.
2013-11-14 Cary Coutant <ccoutant@google.com> 2013-11-14 Cary Coutant <ccoutant@google.com>
* dwp.cc (Dwo_file_entry): New type. * dwp.cc (Dwo_file_entry): New type.

6
gold/ehframe.cc

@ -84,8 +84,7 @@ Eh_frame_hdr::Eh_frame_hdr(Output_section* eh_frame_section,
eh_frame_section_(eh_frame_section), eh_frame_section_(eh_frame_section),
eh_frame_data_(eh_frame_data), eh_frame_data_(eh_frame_data),
fde_offsets_(), fde_offsets_(),
any_unrecognized_eh_frame_sections_(false), any_unrecognized_eh_frame_sections_(false)
lock_(NULL)
{ {
} }
@ -103,9 +102,6 @@ Eh_frame_hdr::set_final_data_size()
this->fde_offsets_.reserve(fde_count); this->fde_offsets_.reserve(fde_count);
} }
this->set_data_size(data_size); this->set_data_size(data_size);
// We need a lock for updating the fde_offsets_ vector while writing
// the FDEs.
this->lock_ = new Lock();
} }
// Write the data to the file. // Write the data to the file.

7
gold/ehframe.h

@ -60,10 +60,7 @@ class Eh_frame_hdr : public Output_section_data
record_fde(section_offset_type fde_offset, unsigned char fde_encoding) record_fde(section_offset_type fde_offset, unsigned char fde_encoding)
{ {
if (!this->any_unrecognized_eh_frame_sections_) if (!this->any_unrecognized_eh_frame_sections_)
{ this->fde_offsets_.push_back(std::make_pair(fde_offset, fde_encoding));
Hold_lock(*this->lock_);
this->fde_offsets_.push_back(std::make_pair(fde_offset, fde_encoding));
}
} }
protected: protected:
@ -160,8 +157,6 @@ class Eh_frame_hdr : public Output_section_data
// Whether we found any .eh_frame sections which we could not // Whether we found any .eh_frame sections which we could not
// process. // process.
bool any_unrecognized_eh_frame_sections_; bool any_unrecognized_eh_frame_sections_;
// Lock held while updating fde_offsets_.
Lock* lock_;
}; };
// This class holds an FDE. // This class holds an FDE.

Loading…
Cancel
Save