authorgravatar for kubkon@jakubkonka.comJakub Konka <kubkon@jakubkonka.com> 2022-09-07 12:46:51+02:00
committergravatar for kubkon@jakubkonka.comJakub Konka <kubkon@jakubkonka.com> 2022-09-07 22:42:59+02:00
log215fce8c51662970d34ae1f4bf1cd043071fea8a
tree3435dba451f0dc69bb419bee76635c9cff4db75e
parent99c2cb72e850ffdfd83abcc941c84a0053f8494e

coff: fix tracking of got and import entries; free relocs in update* fns


1 files changed, 81 insertions(+), 51 deletions(-)

src/link/Coff.zig+81-51
......@@ -63,11 +63,13 @@ globals_free_list: std.ArrayListUnmanaged(u32) = .{},
6363strtab: StringTable(.strtab) = .{},
6464strtab_offset: ?u32 = null,
6565
66got_entries: std.AutoArrayHashMapUnmanaged(SymbolWithLoc, u32) = .{},
66got_entries: std.ArrayListUnmanaged(Entry) = .{},
6767got_entries_free_list: std.ArrayListUnmanaged(u32) = .{},
68got_entries_table: std.AutoHashMapUnmanaged(SymbolWithLoc, u32) = .{},
6869
69imports_table: std.AutoArrayHashMapUnmanaged(SymbolWithLoc, u32) = .{},
70imports_table_free_list: std.ArrayListUnmanaged(u32) = .{},
70imports: std.ArrayListUnmanaged(Entry) = .{},
71imports_free_list: std.ArrayListUnmanaged(u32) = .{},
72imports_table: std.AutoHashMapUnmanaged(SymbolWithLoc, u32) = .{},
7173
7274/// Virtual address of the entry point procedure relative to image base.
7375entry_addr: ?u32 = null,
......@@ -115,6 +117,12 @@ relocs: RelocTable = .{},
115117/// this will be a table indexed by index into the list of Atoms.
116118base_relocs: BaseRelocationTable = .{},
117119
120const Entry = struct {
121 target: SymbolWithLoc,
122 // Index into the synthetic symbol table (i.e., file == null).
123 sym_index: u32,
124};
125
118126pub const Reloc = struct {
119127 @"type": enum {
120128 got,
......@@ -309,8 +317,10 @@ pub fn deinit(self: *Coff) void {
309317 self.strtab.deinit(gpa);
310318 self.got_entries.deinit(gpa);
311319 self.got_entries_free_list.deinit(gpa);
320 self.got_entries_table.deinit(gpa);
321 self.imports.deinit(gpa);
322 self.imports_free_list.deinit(gpa);
312323 self.imports_table.deinit(gpa);
313 self.imports_table_free_list.deinit(gpa);
314324 self.decls.deinit(gpa);
315325 self.atom_by_index_table.deinit(gpa);
316326
......@@ -684,42 +694,44 @@ fn allocateGlobal(self: *Coff) !u32 {
684694pub fn allocateGotEntry(self: *Coff, target: SymbolWithLoc) !u32 {
685695 const gpa = self.base.allocator;
686696 try self.got_entries.ensureUnusedCapacity(gpa, 1);
697
687698 const index: u32 = blk: {
688699 if (self.got_entries_free_list.popOrNull()) |index| {
689700 log.debug(" (reusing GOT entry index {d})", .{index});
690 if (self.got_entries.getIndex(target)) |existing| {
691 assert(existing == index);
692 }
693701 break :blk index;
694702 } else {
695 log.debug(" (allocating GOT entry at index {d})", .{self.got_entries.keys().len});
696 const index = @intCast(u32, self.got_entries.keys().len);
697 self.got_entries.putAssumeCapacityNoClobber(target, 0);
703 log.debug(" (allocating GOT entry at index {d})", .{self.got_entries.items.len});
704 const index = @intCast(u32, self.got_entries.items.len);
705 _ = self.got_entries.addOneAssumeCapacity();
698706 break :blk index;
699707 }
700708 };
701 self.got_entries.keys()[index] = target;
709
710 self.got_entries.items[index] = .{ .target = target, .sym_index = 0 };
711 try self.got_entries_table.putNoClobber(gpa, target, index);
712
702713 return index;
703714}
704715
705716pub fn allocateImportEntry(self: *Coff, target: SymbolWithLoc) !u32 {
706717 const gpa = self.base.allocator;
707 try self.imports_table.ensureUnusedCapacity(gpa, 1);
718 try self.imports.ensureUnusedCapacity(gpa, 1);
719
708720 const index: u32 = blk: {
709 if (self.imports_table_free_list.popOrNull()) |index| {
721 if (self.imports_free_list.popOrNull()) |index| {
710722 log.debug(" (reusing import entry index {d})", .{index});
711 if (self.imports_table.getIndex(target)) |existing| {
712 assert(existing == index);
713 }
714723 break :blk index;
715724 } else {
716 log.debug(" (allocating import entry at index {d})", .{self.imports_table.keys().len});
717 const index = @intCast(u32, self.imports_table.keys().len);
718 self.imports_table.putAssumeCapacityNoClobber(target, 0);
725 log.debug(" (allocating import entry at index {d})", .{self.imports.items.len});
726 const index = @intCast(u32, self.imports.items.len);
727 _ = self.imports.addOneAssumeCapacity();
719728 break :blk index;
720729 }
721730 };
722 self.imports_table.keys()[index] = target;
731
732 self.imports.items[index] = .{ .target = target, .sym_index = 0 };
733 try self.imports_table.putNoClobber(gpa, target, index);
734
723735 return index;
724736}
725737
......@@ -734,7 +746,6 @@ fn createGotAtom(self: *Coff, target: SymbolWithLoc) !*Atom {
734746
735747 try self.managed_atoms.append(gpa, atom);
736748 try self.atom_by_index_table.putNoClobber(gpa, atom.sym_index, atom);
737 self.got_entries.getPtr(target).?.* = atom.sym_index;
738749
739750 const sym = atom.getSymbolPtr(self);
740751 sym.section_number = @intToEnum(coff.SectionNumber, self.got_section_index.? + 1);
......@@ -762,7 +773,7 @@ fn createGotAtom(self: *Coff, target: SymbolWithLoc) !*Atom {
762773 return atom;
763774}
764775
765fn createImportAtom(self: *Coff, target: SymbolWithLoc) !*Atom {
776fn createImportAtom(self: *Coff) !*Atom {
766777 const gpa = self.base.allocator;
767778 const atom = try gpa.create(Atom);
768779 errdefer gpa.destroy(atom);
......@@ -773,7 +784,6 @@ fn createImportAtom(self: *Coff, target: SymbolWithLoc) !*Atom {
773784
774785 try self.managed_atoms.append(gpa, atom);
775786 try self.atom_by_index_table.putNoClobber(gpa, atom.sym_index, atom);
776 self.imports_table.getPtr(target).?.* = atom.sym_index;
777787
778788 const sym = atom.getSymbolPtr(self);
779789 sym.section_number = @intToEnum(coff.SectionNumber, self.idata_section_index.? + 1);
......@@ -804,7 +814,7 @@ fn writeAtom(self: *Coff, atom: *Atom, code: []const u8) !void {
804814 const sym = atom.getSymbol(self);
805815 const section = self.sections.get(@enumToInt(sym.section_number) - 1);
806816 const file_offset = section.header.pointer_to_raw_data + sym.value - section.header.virtual_address;
807 log.debug("writing atom for symbol {s} at file offset 0x{x}", .{ atom.getName(self), file_offset });
817 log.debug("writing atom for symbol {s} at file offset 0x{x} to 0x{x}", .{ atom.getName(self), file_offset, file_offset + code.len });
808818 try self.base.file.?.pwriteAll(code, file_offset);
809819 try self.resolveRelocs(atom);
810820}
......@@ -860,11 +870,12 @@ fn resolveRelocs(self: *Coff, atom: *Atom) !void {
860870 const target_vaddr = target_atom.getSymbol(self).value;
861871 const target_vaddr_with_addend = target_vaddr + reloc.addend;
862872
863 log.debug(" ({x}: [() => 0x{x} ({s})) ({s})", .{
873 log.debug(" ({x}: [() => 0x{x} ({s})) ({s}) (in file at 0x{x})", .{
864874 source_sym.value + reloc.offset,
865875 target_vaddr_with_addend,
866876 self.getSymbolName(reloc.target),
867877 @tagName(reloc.@"type"),
878 file_offset + reloc.offset,
868879 });
869880
870881 reloc.dirty = false;
......@@ -901,8 +912,7 @@ fn freeAtom(self: *Coff, atom: *Atom) void {
901912 log.debug("freeAtom {*}", .{atom});
902913
903914 // Remove any relocs and base relocs associated with this Atom
904 _ = self.relocs.remove(atom);
905 _ = self.base_relocs.remove(atom);
915 self.freeRelocationsForAtom(atom);
906916
907917 const sym = atom.getSymbol(self);
908918 const sect_id = @enumToInt(sym.section_number) - 1;
......@@ -966,11 +976,14 @@ pub fn updateFunc(self: *Coff, module: *Module, func: *Module.Fn, air: Air, live
966976 const tracy = trace(@src());
967977 defer tracy.end();
968978
979 const decl_index = func.owner_decl;
980 const decl = module.declPtr(decl_index);
981 self.freeUnnamedConsts(decl_index);
982 self.freeRelocationsForAtom(&decl.link.coff);
983
969984 var code_buffer = std.ArrayList(u8).init(self.base.allocator);
970985 defer code_buffer.deinit();
971986
972 const decl_index = func.owner_decl;
973 const decl = module.declPtr(decl_index);
974987 const res = try codegen.generateFunction(
975988 &self.base,
976989 decl.srcLoc(),
......@@ -1082,6 +1095,8 @@ pub fn updateDecl(self: *Coff, module: *Module, decl_index: Module.Decl.Index) !
10821095 }
10831096 }
10841097
1098 self.freeRelocationsForAtom(&decl.link.coff);
1099
10851100 var code_buffer = std.ArrayList(u8).init(self.base.allocator);
10861101 defer code_buffer.deinit();
10871102
......@@ -1190,8 +1205,9 @@ fn updateDeclCode(self: *Coff, decl_index: Module.Decl.Index, code: []const u8,
11901205 sym.value = vaddr;
11911206
11921207 const got_target = SymbolWithLoc{ .sym_index = atom.sym_index, .file = null };
1193 _ = try self.allocateGotEntry(got_target);
1208 const got_index = try self.allocateGotEntry(got_target);
11941209 const got_atom = try self.createGotAtom(got_target);
1210 self.got_entries.items[got_index].sym_index = got_atom.sym_index;
11951211 try self.writePtrWidthAtom(got_atom);
11961212 }
11971213
......@@ -1199,6 +1215,11 @@ fn updateDeclCode(self: *Coff, decl_index: Module.Decl.Index, code: []const u8,
11991215 try self.writeAtom(atom, code);
12001216}
12011217
1218fn freeRelocationsForAtom(self: *Coff, atom: *Atom) void {
1219 _ = self.relocs.remove(atom);
1220 _ = self.base_relocs.remove(atom);
1221}
1222
12021223fn freeUnnamedConsts(self: *Coff, decl_index: Module.Decl.Index) void {
12031224 const gpa = self.base.allocator;
12041225 const unnamed_consts = self.unnamed_const_atoms.getPtr(decl_index) orelse return;
......@@ -1237,14 +1258,20 @@ pub fn freeDecl(self: *Coff, decl_index: Module.Decl.Index) void {
12371258
12381259 // Try freeing GOT atom if this decl had one
12391260 const got_target = SymbolWithLoc{ .sym_index = sym_index, .file = null };
1240 if (self.got_entries.getIndex(got_target)) |got_index| {
1261 if (self.got_entries_table.get(got_target)) |got_index| {
12411262 self.got_entries_free_list.append(gpa, @intCast(u32, got_index)) catch {};
1242 self.got_entries.values()[got_index] = 0;
1263 self.got_entries.items[got_index] = .{
1264 .target = .{ .sym_index = 0, .file = null },
1265 .sym_index = 0,
1266 };
1267 _ = self.got_entries_table.remove(got_target);
1268
12431269 log.debug(" adding GOT index {d} to free list (target local@{d})", .{ got_index, sym_index });
12441270 }
12451271
12461272 self.locals.items[sym_index].section_number = .UNDEFINED;
12471273 _ = self.atom_by_index_table.remove(sym_index);
1274 log.debug(" adding local symbol index {d} to free list", .{sym_index});
12481275 decl.link.coff.sym_index = 0;
12491276 }
12501277}
......@@ -1453,8 +1480,9 @@ pub fn flushModule(self: *Coff, comp: *Compilation, prog_node: *std.Progress.Nod
14531480 const global = self.globals.items[entry.key];
14541481 if (self.imports_table.contains(global)) continue;
14551482
1456 _ = try self.allocateImportEntry(global);
1457 const import_atom = try self.createImportAtom(global);
1483 const import_index = try self.allocateImportEntry(global);
1484 const import_atom = try self.createImportAtom();
1485 self.imports.items[import_index].sym_index = import_atom.sym_index;
14581486 try self.writePtrWidthAtom(import_atom);
14591487 }
14601488
......@@ -1668,8 +1696,8 @@ fn writeImportTable(self: *Coff) !void {
16681696 defer names_table.deinit();
16691697
16701698 // TODO: check if import is still valid
1671 for (self.imports_table.keys()) |target| {
1672 const target_name = self.getSymbolName(target);
1699 for (self.imports.items) |entry| {
1700 const target_name = self.getSymbolName(entry.target);
16731701 const start = names_table.items.len;
16741702 mem.writeIntLittle(u16, try names_table.addManyAsArray(2), 0); // TODO: currently, hint is set to 0 as we haven't yet parsed any DLL
16751703 try names_table.appendSlice(target_name);
......@@ -2013,19 +2041,19 @@ pub fn getEntryPoint(self: Coff) ?SymbolWithLoc {
20132041 return self.globals.items[global_index];
20142042}
20152043
2016/// Returns pointer-to-symbol described by `sym_with_loc` descriptor.
2044/// Returns pointer-to-symbol described by `sym_loc` descriptor.
20172045pub fn getSymbolPtr(self: *Coff, sym_loc: SymbolWithLoc) *coff.Symbol {
20182046 assert(sym_loc.file == null); // TODO linking object files
20192047 return &self.locals.items[sym_loc.sym_index];
20202048}
20212049
2022/// Returns symbol described by `sym_with_loc` descriptor.
2050/// Returns symbol described by `sym_loc` descriptor.
20232051pub fn getSymbol(self: *const Coff, sym_loc: SymbolWithLoc) *const coff.Symbol {
20242052 assert(sym_loc.file == null); // TODO linking object files
20252053 return &self.locals.items[sym_loc.sym_index];
20262054}
20272055
2028/// Returns name of the symbol described by `sym_with_loc` descriptor.
2056/// Returns name of the symbol described by `sym_loc` descriptor.
20292057pub fn getSymbolName(self: *const Coff, sym_loc: SymbolWithLoc) []const u8 {
20302058 assert(sym_loc.file == null); // TODO linking object files
20312059 const sym = self.getSymbol(sym_loc);
......@@ -2033,25 +2061,27 @@ pub fn getSymbolName(self: *const Coff, sym_loc: SymbolWithLoc) []const u8 {
20332061 return self.strtab.get(offset).?;
20342062}
20352063
2036/// Returns atom if there is an atom referenced by the symbol described by `sym_with_loc` descriptor.
2064/// Returns atom if there is an atom referenced by the symbol described by `sym_loc` descriptor.
20372065/// Returns null on failure.
20382066pub fn getAtomForSymbol(self: *Coff, sym_loc: SymbolWithLoc) ?*Atom {
20392067 assert(sym_loc.file == null); // TODO linking with object files
20402068 return self.atom_by_index_table.get(sym_loc.sym_index);
20412069}
20422070
2043/// Returns GOT atom that references `sym_with_loc` if one exists.
2071/// Returns GOT atom that references `sym_loc` if one exists.
20442072/// Returns null otherwise.
20452073pub fn getGotAtomForSymbol(self: *Coff, sym_loc: SymbolWithLoc) ?*Atom {
2046 const got_index = self.got_entries.get(sym_loc) orelse return null;
2047 return self.atom_by_index_table.get(got_index);
2074 const got_index = self.got_entries_table.get(sym_loc) orelse return null;
2075 const got_entry = self.got_entries.items[got_index];
2076 return self.getAtomForSymbol(.{ .sym_index = got_entry.sym_index, .file = null });
20482077}
20492078
2050/// Returns import atom that references `sym_with_loc` if one exists.
2079/// Returns import atom that references `sym_loc` if one exists.
20512080/// Returns null otherwise.
20522081pub fn getImportAtomForSymbol(self: *Coff, sym_loc: SymbolWithLoc) ?*Atom {
20532082 const imports_index = self.imports_table.get(sym_loc) orelse return null;
2054 return self.atom_by_index_table.get(imports_index);
2083 const imports_entry = self.imports.items[imports_index];
2084 return self.getAtomForSymbol(.{ .sym_index = imports_entry.sym_index, .file = null });
20552085}
20562086
20572087fn setSectionName(self: *Coff, header: *coff.SectionHeader, name: []const u8) !void {
......@@ -2141,21 +2171,21 @@ fn logSymtab(self: *Coff) void {
21412171 }
21422172
21432173 log.debug("GOT entries:", .{});
2144 for (self.got_entries.keys()) |target, i| {
2145 const got_sym = self.getSymbol(.{ .sym_index = self.got_entries.values()[i], .file = null });
2146 const target_sym = self.getSymbol(target);
2174 for (self.got_entries.items) |entry, i| {
2175 const got_sym = self.getSymbol(.{ .sym_index = entry.sym_index, .file = null });
2176 const target_sym = self.getSymbol(entry.target);
21472177 if (target_sym.section_number == .UNDEFINED) {
21482178 log.debug(" {d}@{x} => import('{s}')", .{
21492179 i,
21502180 got_sym.value,
2151 self.getSymbolName(target),
2181 self.getSymbolName(entry.target),
21522182 });
21532183 } else {
21542184 log.debug(" {d}@{x} => local(%{d}) in object({?d}) {s}", .{
21552185 i,
21562186 got_sym.value,
2157 target.sym_index,
2158 target.file,
2187 entry.target.sym_index,
2188 entry.target.file,
21592189 logSymAttributes(target_sym, &buf),
21602190 });
21612191 }