semantic: 0.282
other: 0.144
device: 0.114
performance: 0.052
socket: 0.052
vnc: 0.051
graphic: 0.047
PID: 0.045
debug: 0.045
files: 0.038
network: 0.037
permissions: 0.037
KVM: 0.028
boot: 0.027
performance: 0.197
debug: 0.132
device: 0.096
network: 0.087
semantic: 0.083
files: 0.078
other: 0.071
PID: 0.060
socket: 0.046
vnc: 0.038
boot: 0.035
permissions: 0.031
KVM: 0.028
graphic: 0.018

PL050 KMIDATA register does not reset

static uint32_t pl050_read(void *opaque, target_phys_addr_t offset){
  ...
   case 2: /* KMIDATA */
        if (s->pending)
            s->last = ps2_read_data(s->dev);
        return s->last;
}

When the receive queue is empty (s->pending is false), is the KMIDATA register supposed to be reset to 0x00? In the current implementation,  the  KMIDATA  does not reverse its value after interrupt is lowered.

On 26 July 2015 at 19:16, T-T Yu <email address hidden> wrote:
> Public bug reported:
>
> static uint32_t pl050_read(void *opaque, target_phys_addr_t offset){
>   ...
>    case 2: /* KMIDATA */
>         if (s->pending)
>             s->last = ps2_read_data(s->dev);
>         return s->last;
> }
>
> When the receive queue is empty (s->pending is false), is the KMIDATA
> register supposed to be reset to 0x00? In the current implementation,
> the  KMIDATA  does not reverse its value after interrupt is lowered.

The documentation for the PL050:
http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ddi0143c/i1014040.html

just says that when KMIDATA is read you get the value in the
receive data register. The implication is that if you read
it multiple times you'll just continue to read the same
value it holds until the keyboard sends further data to be
clocked into the register.

Are you saying that real hardware behaves differently?

thanks
-- PMM


Yes. In our pl050 keyboard controller, the KMIDATA is reset once the receive queue is empty. 

Looking through old bug tickets... is this still an issue with the latest version of QEMU? Or could we close this ticket nowadays?


[Expired for QEMU because there has been no activity for 60 days.]