-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3623/
-----------------------------------------------------------

(Updated Sept. 19, 2016, 10:31 p.m.)


Review request for Default and Andreas Sandberg.


Changes
-------

Removed useless conditional check as pointed out by Andreas.


Repository: gem5


Description
-------

Improve PL011 console interactivity


Diffs (updated)
-----

  src/dev/arm/pl011.cc d726d0cea027 

Diff: http://reviews.gem5.org/r/3623/diff/


Testing
-------

We are operating in "register mode" (not FIFO) which means if we clear an 
interrupt and have more data available, we need to re-raise the interrupt 
again.  Add two more cases where this is needed.

With this the interactivity on FreeBSD went to usable.  Before this after a 
certain event, it could take up to 15(?) additional charatcers to get 
previously pasted command lines echoed and executed.

I have a possible report about similar behaviour from people at ARM on Linux 
(uncofirmed) and I am putting the patch up so they can test.


Thanks,

Bjoern A. Zeeb

_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to