Next Article in Journal
Research on UAV Safety Distance Warning Technology Based on Electric Field Ring Rate of Change
Next Article in Special Issue
Memory-Tree Based Design of Optical Character Recognition in FPGA
Previous Article in Journal
A Novel Color Image Encryption Algorithm Using Coupled Map Lattice with Polymorphic Mapping
Previous Article in Special Issue
Scalable Hardware Efficient Architecture for Parallel FIR Filters with Symmetric Coefficients
 
 
Article
Peer-Review Record

A Deterministic Branch Prediction Technique for a Real-Time Embedded Processor Based on PicoBlaze Architecture

Electronics 2022, 11(21), 3438; https://doi.org/10.3390/electronics11213438
by Ehsan Ali and Wanchalerm Pora *
Reviewer 1:
Reviewer 2: Anonymous
Reviewer 3: Anonymous
Electronics 2022, 11(21), 3438; https://doi.org/10.3390/electronics11213438
Submission received: 27 September 2022 / Revised: 15 October 2022 / Accepted: 18 October 2022 / Published: 24 October 2022
(This article belongs to the Special Issue FPGAs Based Hardware Design)

Round 1

Reviewer 1 Report

Please,find the review in the attached file

Comments for author File: Comments.pdf

Author Response

The response is included in attached MS Word file.

Author Response File: Author Response.pdf

Reviewer 2 Report

The author presented a novel method for branch prediction for central processing units in the article. The proposed technique helps to avoid the possible problems caused by the conditional jump instructions. Overall, the paper is interesting, and the results look good. However, there are a few comments as follows:

1. In Figure 11, the author presents the diagram for the modified Zipi8. However, there is no ALU component in this. It is better to have a full diagram as Figure 4 with highlighting the differences.

2. Figure 13 shows the Stack module architecture. However, the caption for this is the same as Figure 12. Please, clarify this.

3. CPI value in Equation (4) is wrong, it should be “1” in the case of using DAP-Zipi8 architecture.

4. Quality of Figure 1 should be improved.

5. Caption and content of the same table (Tables 1 and 5) need to be in the same page.

Author Response

The response is included in attached MS Word file.

Author Response File: Author Response.pdf

Reviewer 3 Report

Major remarks

    + Figure 5 is difficult to understand, I suggest to highlight only the relevant parts of the schematic and to remove the circuitry that is not directly referenced in the text, to help readability;
    + To me it is not completely clear why the authors chose the Xilinx Picoblaze to investigate their contributions, instead of an opensource core. Maybe I can suggest to detail better this aspect in the introductory or background sections;
    
    
    + The experimental section that shows the behaviour of the proposed system in a real context is limited. What I suggest is to consider a much broader experimental campaign, in order to better appreciate the benefits obtained. Currently only four benchmarks are considered. I also recommend the use more representative benchmarks in the context of embedded/real-time domain. For example embenches (https://www.embench.org/) could be more adequate;
    
    + The final goal of the proposed work is to modify the design of a core in order to make its behavior predictable. However this aspect is not addressed and validated in the experimental section. To this end, a type of experiment that I recommend to take into consideration is to quantify the variability on the execution time of a benchmark, considering the standard Picoblaze, compared to the modified version;
    
    + Section 7 which introduces the design changes in order to obtain a predictable CPI = 1 is perhaps too long and unbalanced compared to the rest of the paper. For this reason it is rather complex to follow that section in all its parts. I suggest to slightly reduce this part, eventually moving part of this in the appendix and use the space obtained for further experiments.
    
Minor remarks

    + "no benchmarking program is required because they both cores.." --> "no benchmarking program is required because both cores..";
    
    + The instruction "ADD 1 s0", I think that should be "ADD s1, s0";
    
    + In the "definitions" section at page 2, the authors say that for the implementation of a processor-based design, in addition to FPGA and ASIC devices, it is possible to use MCUs. However, it is not entirely clear what is meant in this case, since MCUs are fixed hardware devices, that can not be re-configured like FPGAs.

Author Response

The response is included in attached MS Word file.

Author Response File: Author Response.pdf

Round 2

Reviewer 1 Report

All my comments are taken into account. The paper is corrected in a proper way. I think that now the artcile can be accepted in the current form.

Back to TopTop