Re: kvm-84 kernel panic

2009-03-05 Thread Ryan Harper
* Johannes Baumann johannes.baum...@gmx.de [2009-03-05 16:08]: Ryan Harper schrieb: * Johannes Baumann johannes.baum...@gmx.de [2009-03-05 15:52]: update: its not only happening with smp guest, but with non-smp guest. mcelog is empty. the commandline for a debian lenny im using is: /usr

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread Ryan Harper
-- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry...@us.ibm.com -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-02 Thread Ryan Harper
and spawning those on a target system. I do still like the idea of having a client-side test that can just run on a developer/user's system to produce results without having to configure all of the autotest server-side bits. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp

[PATCH] lsi: add ISTAT1 register read

2009-02-27 Thread Ryan Harper
SLES10 SP2 installer complains when probing a scsi disk and exits qemu when failing to read one of the registers. lsi_scsi: error: readb 0x15 -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry...@us.ibm.com diffstat output: lsi53c895a.c |2 ++ 1 files

[PATCH 0/2] LSI53C895A and scsi-disk fixes

2009-01-23 Thread Ryan Harper
, Windows XP 64, 2k3 and didn't see any regressions. Signed-off-by: Ryan Harper ry...@us.ibm.com -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] lsi,scsi-disk: check for reentrance via tag matching

2009-01-23 Thread Ryan Harper
allows SLES10 SP2 installer to complete and guest function fully. Signed-off-by: Ryan Harper ry...@us.ibm.com diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 912d7d5..34f50df 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -668,6 +668,7 @@ static void lsi_do_command(LSIState *s

[PATCH 1/2] lsi: add ISTAT1 register read

2009-01-23 Thread Ryan Harper
SLES10 SP2 installer complains with: lsi_scsi: error: readb 0x15 Signed-off-by: Ryan Harper ry...@us.ibm.com diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 9f50dcb..912d7d5 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -1368,6 +1368,8 @@ static uint8_t lsi_reg_readb(LSIState *s

Re: [PATCH 2/2] lsi,scsi-disk: check for reentrance via tag matching

2009-01-23 Thread Ryan Harper
something that the OS driver handles well. I suppose once we figure out how to stop executing scripts after sending command completion in a way that the OS drivers understand, moving that to a bottom half will work fine. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry

Re: Still LSI scsi problems with Windows guest and kvm 82?

2009-01-13 Thread Ryan Harper
length, command f0 lsi_scsi: error: Reselect with pending DMA scsi-disk: Bad buffer tag 0x0 Is this a known problem? Can you reproduce with the kvm release rather than debian packages? I've not seen this string of errors before. -- Ryan Harper Software Engineer; Linux Technology

Re: qcow2 corruption?

2009-01-09 Thread Ryan Harper
surprising to see. If you can't recreate now, I think we're done. =) Sorry for the bother, guys. np, thanks for testing. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry...@us.ibm.com -- To unsubscribe from this list: send the line unsubscribe kvm in the body

Re: BUG() with SCSI-interfaced disk images

2009-01-08 Thread Ryan Harper
* John Morrissey j...@horde.net [2009-01-07 20:16]: On Wed, Jan 07, 2009 at 04:34:50PM -0600, Ryan Harper wrote: * John Morrissey j...@horde.net [2009-01-07 15:59]: On Fri, Dec 26, 2008 at 04:00:28PM -0500, John Morrissey wrote: I'm encountering a kernel BUG() in guests using SCSI

Re: Win2K3 SCSI errors

2009-01-08 Thread Ryan Harper
reworking of the lsi_queue so we don't remove tasks from the queue until we've signaled to the driver we've completed the task. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry...@us.ibm.com -- To unsubscribe from this list: send the line unsubscribe kvm in the body

Re: qcow2 corruption?

2009-01-08 Thread Ryan Harper
are using ide and getting corruption, try again but with creating a disk image with the raw format: qemu-img create -f raw imagename size That should help track down where the corruption is coming from. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry

Re: Poor write- and overall performance

2008-12-19 Thread Ryan Harper
working on fixing this; stay tuned. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ry...@us.ibm.com -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: lsi_scsi: error: Bad Status move errors with kvm-79

2008-12-08 Thread Ryan Harper
* Anssi Kolehmainen [EMAIL PROTECTED] [2008-12-04 10:11]: On Thu, Dec 04, 2008 at 09:26:35AM -0600, Ryan Harper wrote: * Anssi Kolehmainen [EMAIL PROTECTED] [2008-12-04 08:51]: Hi, I have kvm environment with linux-2.6.28-rc7 x86_64 (Xeon), kvm-79 host and bunch of Win2K3 guests

[PATCH 1/4] LSI53C895A: Rename dmbs register to dbms

2008-12-08 Thread Ryan Harper
Register name should be: Dynamic Block Move Selector (dbms) according to page 215 of the LSI 53C895A Technical Manual[1]. 1. http://www.lsi.com/DistributionSystem/AssetDocument/files/docs/techdocs/storage_stand_prod/SCSIControllers/lsi53c895a_tech_manual.pdf Signed-off-by: Ryan Harper [EMAIL

[PATCH 3/4] LSI53C895A: Implement TARGET RESET message

2008-12-08 Thread Ryan Harper
-by: Ryan Harper [EMAIL PROTECTED] diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index b36c08c..ac8c5a5 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -804,6 +804,10 @@ static void lsi_do_msgout(LSIState *s) goto bad; } break; +case 0x0c

[PATCH 2/4] Add 64-bit Block Move support (Direct Table Indirect)

2008-12-08 Thread Ryan Harper
SYM_CONF_DMA_ADDRESSING_MODE=2 need a quirk fixup in Patch 4 to function properly. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index a2162ba..b36c08c 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -338,6 +338,20 @@ static int lsi_dma_40bit(LSIState

[PATCH 4/4] LSI53C895A: Don't reset scratch C-R on soft reset

2008-12-08 Thread Ryan Harper
. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index b36c08c..542d5a0 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -276,7 +276,11 @@ static void lsi_soft_reset(LSIState *s) s-dnad = 0; s-dbc = 0; s-temp = 0; -memset(s-scratch, 0

Re: [Qemu-devel] [PATCH 3/4] LSI53C895A: Implement TARGET RESET message

2008-12-08 Thread Ryan Harper
* Paul Brook [EMAIL PROTECTED] [2008-12-08 12:39]: On Monday 08 December 2008, Ryan Harper wrote: Linux and Windows send a TARGET RESET message to the device when it fails to respond as it expects. For example, when it tries to select LUN1, which we don't support. This patch is needed

Re: [Qemu-devel] [PATCH 3/4] LSI53C895A: Implement TARGET RESET message

2008-12-08 Thread Ryan Harper
on the first 2 patches. The last two patches are to address a rather uncommon mode of the linux driver so, if the first two patches are acceptable, I'd like to see those go in. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx [EMAIL PROTECTED] -- To unsubscribe from

Re: lsi_scsi: error: Bad Status move errors with kvm-79

2008-12-04 Thread Ryan Harper
/downloadable copies of Bae or Oracle that I can use to recreate? -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx [EMAIL PROTECTED] -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: qcow2 slowdown in kvm-78,79

2008-11-19 Thread Ryan Harper
, specifically, what if type? ide, scsi, virtio? -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx [EMAIL PROTECTED] -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org

[PATCH] Add 40-bit DMA support to LSI scsi emulation

2008-11-12 Thread Ryan Harper
This patch fixes Linux machines configured with 4G of ram and using a SCSI device. I'm working on 64-bit DMA support as well which should hopefully fix 64-bit Windows using SCSI devices as well. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx (512) 838-9253 T

[PATCH 0/2] Add qemu-test for automated testing

2008-10-15 Thread Ryan Harper
This patch provides a relatively simple method for exercising various features of qemu by interacting with the guest via serial and qemu via the monitor that individual developers can use to validate that their changes haven't broken feature/function. Such tests need to be very accessable and

[PATCH 1/2] Add qemu-test for automated testing

2008-10-15 Thread Ryan Harper
features against a set of images. Patch 2/2 provides Makefile integration into qemu to execute qemu-test against the current build of the tree. This can be done without the Makefile integration, but simplifies the execution. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/tests/qemu

Re: [Qemu-devel] [PATCH 0/4] Improve emulated scsi write performance

2008-10-13 Thread Ryan Harper
* Ryan Harper [EMAIL PROTECTED] [2008-10-05 18:08]: Any comments/review for patches 1-3? It seems that patch4 is something that we need to work out before committing, but I'd like to see pathes 1-3 go in since they provide a performance boost in and of themselves. Any movement here

Re: [Qemu-devel] [PATCH 4/4] Reallocate dma buffers in read/write path if needed

2008-10-04 Thread Ryan Harper
ulimits if this is really a concern. Are we any more concerned that by splitting the requests into many smaller requests that we're wasting cpu, pegging the processor to 100% in some cases? -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx (512) 838-9253 T/L: 678-9253

Re: [Qemu-devel] [5323] Implement an fd pool to get real AIO with posix-aio

2008-09-26 Thread Ryan Harper
=on posix-aio w/ patch |165.0 | 70.90 | 7.09 new results --+--+--+-- kvm:cache=off posix-aio fd_pool[16]| 78.2 | 58.24 | 15.43 kvm:cache=off posix-aio fd_pool[64]|129.0 | 71.62 | 9.11 -- Ryan Harper

Re: [Qemu-devel] [5323] Implement an fd pool to get real AIO with posix-aio

2008-09-26 Thread Ryan Harper
* Anthony Liguori [EMAIL PROTECTED] [2008-09-26 13:37]: Ryan Harper wrote: * Anthony Liguori [EMAIL PROTECTED] [2008-09-26 11:03]: kvm: cache=on posix-aio w/o patch |127.0 | 115.78 | 9.19 kvm: cache=on posix-aio w/ patch |126.0 | 67.35 | 9.30 It looks like

Re: [Qemu-devel] Re: [PATCH 1/3] Only call aio flush handler if set

2008-09-23 Thread Ryan Harper
* Anthony Liguori [EMAIL PROTECTED] [2008-09-22 21:49]: Ryan Harper wrote: If the aio handler doesn't register an io_flush handler, we'd SEGV; fix that by only calling the flush handler if set. BTW, aio handlers *should* register an io_flush routine. Signed-off-by: Ryan Harper [EMAIL

Re: [Qemu-devel] Re: [PATCH 2/3] Move aio implementation out of raw block driver

2008-09-23 Thread Ryan Harper
* Anthony Liguori [EMAIL PROTECTED] [2008-09-22 21:52]: Ryan Harper wrote: +//#define DEBUG_BLOCK_AIO +#if defined(DEBUG_BLOCK_AIO) +#define BLPRINTF(formatCstr, args...) do { fprintf(stderr, formatCstr, ##args); fflush(stderr); } while (0) This is GCC syntax. You should use C99

Re: [Qemu-devel] Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-23 Thread Ryan Harper
patch set isn't useful, but I would like to get performance to be better for the case that we're going through the page cache. I can run the test, but it is orthogonal to the patchset which is focused on using O_DIRECT and linux-aio. -- Ryan Harper Software Engineer; Linux Technology Center IBM

[PATCH 2/3] Move aio implementation out of raw block driver

2008-09-22 Thread Ryan Harper
. The changes pave the way for other aio implementations, namely linux aio. Each block device will init the proper aio driver depending on how the device is opened. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/Makefile b/Makefile index de6393e..18477ba 100644 --- a/Makefile +++ b/Makefile

[PATCH 1/3] Only call aio flush handler if set

2008-09-22 Thread Ryan Harper
If the aio handler doesn't register an io_flush handler, we'd SEGV; fix that by only calling the flush handler if set. BTW, aio handlers *should* register an io_flush routine. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/aio.c b/aio.c index 687e4be..2bb3ed4 100644 --- a/aio.c +++ b

[PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-22 Thread Ryan Harper
The patchset adds additional AIO driver abstraction to the block raw driver to support multiple aio implementations for each device. The first patch pulls the posix aio implementation out of the block-raw device using a generic call to the newly created AIO Driver structure. The posix aio

Re: [PATCH 2/3] Move aio implementation out of raw block driver

2008-09-22 Thread Ryan Harper
* Ryan Harper [EMAIL PROTECTED] [2008-09-22 18:18]: This patch moves the existing posix aio implementation out of block-raw-posix.c into aio-posix.c. Added in a per-block device aio driver abstraction. Block-raw-posix invokes the aio driver methods, .submit, .flush, and .cancel as needed

[PATCH 3/3] Add linux aio implementation for raw block devices

2008-09-22 Thread Ryan Harper
* Ryan Harper [EMAIL PROTECTED] [2008-09-22 18:18]: This patch adds a linux aio raw block driver implementation. If a raw block device is opened with cached=off (O_DIRECT) then we can utilize linux aio to submit io to/from the block device. Utilizing linux aio allows for multiple outstanding

Re: [Qemu-devel] [PATCH 1/1][RESEND] v2: Fix text console size/resize when using curses

2008-09-15 Thread Ryan Harper
* andrzej zaborowski [EMAIL PROTECTED] [2008-09-15 11:51]: Hi, Hello, 2008/9/4 Ryan Harper [EMAIL PROTECTED]: v2: drop initial size adjustment, not needed. Resize events fail to ensure that both the text console and curses display areas are the same size; this causes broken output

[PATCH 0/1][RESEND] v2: Fix monitor console in curses mode

2008-09-04 Thread Ryan Harper
) (qemu) (qemu) and you can resize your terminal window and the monitor and serial text consoles adjust properly as well. Signed-off-by: Ryan Harper [EMAIL PROTECTED] -- To unsubscribe from this list: send the line unsubscribe

[PATCH 0/2] v2: Fix monitor console in curses mode

2008-08-25 Thread Ryan Harper
) (qemu) and you can resize your terminal window and the monitor and serial text consoles adjust properly as well. The second patch is a trivial typo fix in the comments in console.c Comments welcome since I'm new to both curses and qemu console code. Signed-off-by: Ryan

[PATCH 2/2] Fix typo in console.c comment

2008-08-25 Thread Ryan Harper
Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/console.c b/console.c index 89bdc52..c7a5a80 100644 --- a/console.c +++ b/console.c @@ -172,7 +172,7 @@ void vga_hw_screen_dump(const char *filename) previous_active_console = active_console; active_console = consoles[0

[PATCH 0/2] Fix monitor console in curses mode

2008-08-22 Thread Ryan Harper
code. Signed-off-by: Ryan Harper [EMAIL PROTECTED] -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 1/2] Fix text console size/resize when using curses

2008-08-22 Thread Ryan Harper
for text consoles on refresh; also force a resize event whenever we invalidate the text console. Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/console.c b/console.c index 1c94980..89bdc52 100644 --- a/console.c +++ b/console.c @@ -608,6 +608,9 @@ static void console_refresh(TextConsole

[PATCH 2/2] Fix typo in console.c comment

2008-08-22 Thread Ryan Harper
Signed-off-by: Ryan Harper [EMAIL PROTECTED] diff --git a/console.c b/console.c index 89bdc52..c7a5a80 100644 --- a/console.c +++ b/console.c @@ -172,7 +172,7 @@ void vga_hw_screen_dump(const char *filename) previous_active_console = active_console; active_console = consoles[0

Re: [Qemu-devel] [PATCH 1/2] Fix text console size/resize when using curses

2008-08-22 Thread Ryan Harper
* andrzej zaborowski [EMAIL PROTECTED] [2008-08-22 17:40]: 2008/8/22 Ryan Harper [EMAIL PROTECTED]: The current console output is broken in curses mode size the default starting size might not match the launching terminal size. Change initial size to be based on the terminal. Also

[PATCH 0/1] integrate qemu-test into kvm-userspace

2008-07-25 Thread Ryan Harper
This patch provides a relatively simple method for exercising various features of qemu/kvm by interacting with the guest via serial and qemu via the monitor that individual developers can use to validate that their changes having broken fundamental feature/function. Such tests need to be very

Re: [ANNOUNCE] kvm-autotest

2008-07-16 Thread Ryan Harper
the image as needed, or you'll want some sort of shared storage mounted on each host that you want to run kvm tests upon. And obviously, shared storage is need as soon as we put migration into the mix. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx (512) 838-9253 T

Re: [PATCH] Add HPET support to BIOS

2008-07-10 Thread Ryan Harper
* Elizabeth Kon [EMAIL PROTECTED] [2008-07-09 23:07]: This patch, written by Ryan Harper, adds HPET support to BIOS. Based on hpet changes to Xen's hvm firmware/rombios. Signed-off-by: Beth Kon [EMAIL PROTECTED] Signed-off-by: Ryan Harper [EMAIL PROTECTED] -- Ryan Harper Software Engineer

Re: [PATCH] Add HPET support to BIOS

2008-07-10 Thread Ryan Harper
* Sebastian Herbszt [EMAIL PROTECTED] [2008-07-10 10:46]: Hey Sebastian, Thanks for the review, Beth Kon wrote: This patch, written by Ryan Harper, adds HPET support to BIOS. Signed-off-by: Beth Kon [EMAIL PROTECTED] diff --git a/bios/Makefile b/bios/Makefile index 48022ea..3e73fb5

<    1   2