[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 4790b0: iscsi: do not leak acb->buf when comm
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] 4790b0: iscsi: do not leak acb->buf when commands are abor... |
Date: |
Thu, 24 Jan 2013 12:00:12 -0800 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: 4790b03d308f6c7dea7dc6941ddab9867c9530b8
https://github.com/qemu/qemu/commit/4790b03d308f6c7dea7dc6941ddab9867c9530b8
Author: Paolo Bonzini <address@hidden>
Date: 2013-01-24 (Thu, 24 Jan 2013)
Changed paths:
M block/iscsi.c
Log Message:
-----------
iscsi: do not leak acb->buf when commands are aborted
acb->buf is freed in the WRITE(16) callback, but this may not
get called at all when commands are aborted. Add another
free in the ABORT TASK callback, which requires setting acb->buf
to NULL everywhere.
Signed-off-by: Paolo Bonzini <address@hidden>
Commit: 7371d56fb2759f52106c76692440d0c29731ef9c
https://github.com/qemu/qemu/commit/7371d56fb2759f52106c76692440d0c29731ef9c
Author: Peter Lieven <address@hidden>
Date: 2013-01-24 (Thu, 24 Jan 2013)
Changed paths:
M block/iscsi.c
Log Message:
-----------
iscsi: add support for iovectors
This patch adds support for directly passing the iovec
array from QEMUIOVector if libiscsi supports it (1.8.0
or newer).
Signed-off-by: Peter Lieven <address@hidden>
[Preserve the improvements from commit 4cc841b, iscsi: partly
avoid iovec linearization in iscsi_aio_writev, 2012-11-19 - Paolo]
Signed-off-by: Paolo Bonzini <address@hidden>
Commit: b37a2e4576530597dda880387e3f4da52c42b5b5
https://github.com/qemu/qemu/commit/b37a2e4576530597dda880387e3f4da52c42b5b5
Author: Michael Tokarev <address@hidden>
Date: 2013-01-24 (Thu, 24 Jan 2013)
Changed paths:
M hw/serial.c
Log Message:
-----------
Revert "serial: fix retry logic"
This reverts commit 67c5322d7000fd105a926eec44bc1765b7d70bdd:
I'm not sure if the retry logic has ever worked when not using FIFO mode. I
found this while writing a test case although code inspection confirms it is
definitely broken.
The TSR retry logic will never actually happen because it is guarded by an
'if (s->tsr_rety > 0)' but this is the only place that can ever make the
variable greater than zero. That effectively makes the retry logic an 'if
(0)
I believe this is a typo and the intention was >= 0. Once this is fixed
thoug
I see double transmits with my test case. This is because in the non FIFO
case, serial_xmit may get invoked while LSR.THRE is still high because the
character was processed but the retransmit timer was still active.
We can handle this by simply checking for LSR.THRE and returning early.
It's
possible that the FIFO paths also need some attention.
Cc: Stefano Stabellini <address@hidden>
Signed-off-by: Anthony Liguori <address@hidden>
Even if the previous logic was never worked, new logic breaks stuff -
namely,
qemu -enable-kvm -nographic -kernel /boot/vmlinuz-$(uname -r) -append
console=ttyS0 -serial pty
the above command will cause the virtual machine to stuck at startup
using 100% CPU till one connects to the pty and sends any char to it.
Note this is rather typical invocation for various headless virtual
machines by libvirt.
So revert this change for now, till a better solution will be found.
Signed-off-by: Michael Tokarev <address@hidden>
Signed-off-by: Anthony Liguori <address@hidden>
Commit: 11c29918be32be5b00f367c7da9724a5cddbbb0f
https://github.com/qemu/qemu/commit/11c29918be32be5b00f367c7da9724a5cddbbb0f
Author: Anthony Liguori <address@hidden>
Date: 2013-01-24 (Thu, 24 Jan 2013)
Changed paths:
M block/iscsi.c
Log Message:
-----------
Merge remote-tracking branch 'bonzini/scsi-next' into staging
# By Paolo Bonzini (1) and Peter Lieven (1)
# Via Paolo Bonzini
* bonzini/scsi-next:
iscsi: add support for iovectors
iscsi: do not leak acb->buf when commands are aborted
Compare: https://github.com/qemu/qemu/compare/3f668b6c5dc9...11c29918be32
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] 4790b0: iscsi: do not leak acb->buf when commands are abor...,
GitHub <=