mirror of
https://github.com/openhwgroup/cvw
synced 2025-02-11 06:05:49 +00:00
Fixed bug with spill support and Instruction DA Page Faults.
This commit is contained in:
parent
15f6871a8d
commit
19ec874641
@ -125,8 +125,8 @@ module ifu (
|
|||||||
if(`C_SUPPORTED) begin : SpillSupport
|
if(`C_SUPPORTED) begin : SpillSupport
|
||||||
|
|
||||||
spillsupport spillsupport(.clk, .reset, .StallF, .PCF, .PCPlusUpperF, .PCNextF, .InstrRawF,
|
spillsupport spillsupport(.clk, .reset, .StallF, .PCF, .PCPlusUpperF, .PCNextF, .InstrRawF,
|
||||||
.IFUCacheBusStallF, .ITLBMissF, .PCNextFSpill, .PCFSpill,
|
.InstrDAPageFaultF, .IFUCacheBusStallF, .ITLBMissF, .PCNextFSpill, .PCFSpill,
|
||||||
.SelNextSpillF, .PostSpillInstrRawF, .CompressedF);
|
.SelNextSpillF, .PostSpillInstrRawF, .CompressedF);
|
||||||
end else begin : NoSpillSupport
|
end else begin : NoSpillSupport
|
||||||
assign PCNextFSpill = PCNextF;
|
assign PCNextFSpill = PCNextF;
|
||||||
assign PCFSpill = PCF;
|
assign PCFSpill = PCF;
|
||||||
@ -160,7 +160,7 @@ module ifu (
|
|||||||
.PMPCFG_ARRAY_REGW, .PMPADDR_ARRAY_REGW);
|
.PMPCFG_ARRAY_REGW, .PMPADDR_ARRAY_REGW);
|
||||||
|
|
||||||
end else begin
|
end else begin
|
||||||
assign {ITLBMissF, InstrAccessFaultF, InstrPageFaultF} = '0;
|
assign {ITLBMissF, InstrAccessFaultF, InstrPageFaultF, InstrDAPageFaultF} = '0;
|
||||||
assign PCPF = PCFExt[`PA_BITS-1:0];
|
assign PCPF = PCFExt[`PA_BITS-1:0];
|
||||||
assign CacheableF = '1;
|
assign CacheableF = '1;
|
||||||
end
|
end
|
||||||
|
@ -42,6 +42,7 @@ module spillsupport (
|
|||||||
input logic [31:0] InstrRawF,
|
input logic [31:0] InstrRawF,
|
||||||
input logic IFUCacheBusStallF,
|
input logic IFUCacheBusStallF,
|
||||||
input logic ITLBMissF,
|
input logic ITLBMissF,
|
||||||
|
input logic InstrDAPageFaultF,
|
||||||
output logic [`XLEN-1:0] PCNextFSpill,
|
output logic [`XLEN-1:0] PCNextFSpill,
|
||||||
output logic [`XLEN-1:0] PCFSpill,
|
output logic [`XLEN-1:0] PCFSpill,
|
||||||
output logic SelNextSpillF,
|
output logic SelNextSpillF,
|
||||||
@ -50,44 +51,41 @@ module spillsupport (
|
|||||||
|
|
||||||
|
|
||||||
localparam integer SPILLTHRESHOLD = (`IMEM == `MEM_CACHE) ? `ICACHE_LINELENINBITS/32 : 1;
|
localparam integer SPILLTHRESHOLD = (`IMEM == `MEM_CACHE) ? `ICACHE_LINELENINBITS/32 : 1;
|
||||||
logic [`XLEN-1:0] PCPlus2F;
|
logic [`XLEN-1:0] PCPlus2F;
|
||||||
logic TakeSpillF;
|
logic TakeSpillF;
|
||||||
logic SpillF;
|
logic SpillF;
|
||||||
logic SelSpillF, SpillSaveF;
|
logic SelSpillF, SpillSaveF;
|
||||||
logic [15:0] SpillDataLine0;
|
logic [15:0] SpillDataLine0;
|
||||||
|
typedef enum logic [1:0] {STATE_READY, STATE_SPILL} statetype;
|
||||||
|
(* mark_debug = "true" *) statetype CurrState, NextState;
|
||||||
|
|
||||||
// *** PLACE ALL THIS IN A MODULE
|
mux2 #(`XLEN) pcplus2mux(.d0({PCF[`XLEN-1:2], 2'b10}), .d1({PCPlusUpperF, 2'b00}),
|
||||||
// this exists only if there are compressed instructions.
|
.s(PCF[1]), .y(PCPlus2F));
|
||||||
// reuse PC+2/4 circuitry to avoid needing a second CPA to add 2
|
mux2 #(`XLEN) pcnextspillmux(.d0(PCNextF), .d1(PCPlus2F), .s(SelNextSpillF),
|
||||||
mux2 #(`XLEN) pcplus2mux(.d0({PCF[`XLEN-1:2], 2'b10}), .d1({PCPlusUpperF, 2'b00}), .s(PCF[1]), .y(PCPlus2F));
|
.y(PCNextFSpill));
|
||||||
mux2 #(`XLEN) pcnextspillmux(.d0(PCNextF), .d1(PCPlus2F), .s(SelNextSpillF), .y(PCNextFSpill));
|
|
||||||
mux2 #(`XLEN) pcspillmux(.d0(PCF), .d1(PCPlus2F), .s(SelSpillF), .y(PCFSpill));
|
mux2 #(`XLEN) pcspillmux(.d0(PCF), .d1(PCPlus2F), .s(SelSpillF), .y(PCFSpill));
|
||||||
|
|
||||||
assign SpillF = &PCF[$clog2(SPILLTHRESHOLD)+1:1];
|
assign SpillF = &PCF[$clog2(SPILLTHRESHOLD)+1:1];
|
||||||
|
assign TakeSpillF = SpillF & ~IFUCacheBusStallF & ~(ITLBMissF | InstrDAPageFaultF);
|
||||||
typedef enum logic [1:0] {STATE_SPILL_READY, STATE_SPILL_SPILL} statetype;
|
|
||||||
(* mark_debug = "true" *) statetype CurrState, NextState;
|
|
||||||
|
|
||||||
always_ff @(posedge clk)
|
always_ff @(posedge clk)
|
||||||
if (reset) CurrState <= #1 STATE_SPILL_READY;
|
if (reset) CurrState <= #1 STATE_READY;
|
||||||
else CurrState <= #1 NextState;
|
else CurrState <= #1 NextState;
|
||||||
|
|
||||||
assign TakeSpillF = SpillF & ~IFUCacheBusStallF & ~ITLBMissF;
|
|
||||||
|
|
||||||
always_comb begin
|
always_comb begin
|
||||||
case (CurrState)
|
case (CurrState)
|
||||||
STATE_SPILL_READY: if (TakeSpillF) NextState = STATE_SPILL_SPILL;
|
STATE_READY: if (TakeSpillF) NextState = STATE_SPILL;
|
||||||
else NextState = STATE_SPILL_READY;
|
else NextState = STATE_READY;
|
||||||
STATE_SPILL_SPILL: if(IFUCacheBusStallF | StallF) NextState = STATE_SPILL_SPILL;
|
STATE_SPILL: if(IFUCacheBusStallF | StallF) NextState = STATE_SPILL;
|
||||||
else NextState = STATE_SPILL_READY;
|
else NextState = STATE_READY;
|
||||||
default: NextState = STATE_SPILL_READY;
|
default: NextState = STATE_READY;
|
||||||
endcase
|
endcase
|
||||||
end
|
end
|
||||||
|
|
||||||
assign SelSpillF = (CurrState == STATE_SPILL_SPILL);
|
assign SelSpillF = (CurrState == STATE_SPILL);
|
||||||
assign SelNextSpillF = (CurrState == STATE_SPILL_READY & TakeSpillF) |
|
assign SelNextSpillF = (CurrState == STATE_READY & TakeSpillF) |
|
||||||
(CurrState == STATE_SPILL_SPILL & IFUCacheBusStallF);
|
(CurrState == STATE_SPILL & IFUCacheBusStallF);
|
||||||
assign SpillSaveF = (CurrState == STATE_SPILL_READY) & TakeSpillF;
|
assign SpillSaveF = (CurrState == STATE_READY) & TakeSpillF;
|
||||||
|
|
||||||
flopenr #(16) SpillInstrReg(.clk(clk),
|
flopenr #(16) SpillInstrReg(.clk(clk),
|
||||||
.en(SpillSaveF),
|
.en(SpillSaveF),
|
||||||
|
Loading…
Reference in New Issue
Block a user