[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/4] serial: only resample THR interrupt on r
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/4] serial: only resample THR interrupt on rising edge of IER.THRI |
Date: |
Mon, 15 Dec 2014 17:10:46 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 15/12/2014 17:05, Dr. David Alan Gilbert wrote:
> > + if ((changed & UART_IER_MSI) && s->poll_msl >= 0) {
> > if (s->ier & UART_IER_MSI) {
> > s->poll_msl = 1;
> > serial_update_msl(s);
>
> This MSI stuff, this change is just intended to do the same as you're
> doing for THRI and making it change based? The commit message and title
> doens't mention it.
It has no change. If MSI = 1, there is already a timer ticking every
100th of a second to poll the modem status, so it's useless to go
through serial_udpate_msl/timer_del/...
It's just because I have the changed bits at hand now.
BTW, looks like the poll_msl subsection after all was not necessary. It
can be gleaned by ANDing "does CHR_IOCTL_SERIAL_GET_TIOCM work?" (which
is immutable state depending on the command-line configuration" with
IER.MSI.
Paolo
- [Qemu-devel] [PATCH v3 0/4] serial fixes, including 2.2->2.1 migration, Paolo Bonzini, 2014/12/12
- [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Paolo Bonzini, 2014/12/12
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Dr. David Alan Gilbert, 2014/12/15
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Paolo Bonzini, 2014/12/15
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Dr. David Alan Gilbert, 2014/12/15
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Paolo Bonzini, 2014/12/15
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Dr. David Alan Gilbert, 2014/12/15
- Re: [Qemu-devel] [PATCH v3 1/4] serial: reset thri_pending on IER writes with THRI=0, Paolo Bonzini, 2014/12/15
[Qemu-devel] [PATCH v3 4/4] serial: only resample THR interrupt on rising edge of IER.THRI, Paolo Bonzini, 2014/12/12
[Qemu-devel] [PATCH v3 3/4] serial: update LSR on enabling/disabling FIFOs, Paolo Bonzini, 2014/12/12
[Qemu-devel] [PATCH v3 2/4] serial: clean up THRE/TEMT handling, Paolo Bonzini, 2014/12/12