Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Richard W.M. Jones
On Fri, Apr 01, 2016 at 08:10:48PM +0100, Richard W.M. Jones wrote: > On Fri, Apr 01, 2016 at 03:04:15PM -0400, Kevin O'Connor wrote: > > Otherwise, it doesn't make > > sense that disabling CONFIG_BOOTORDER=n would change the boot time. > > Could it be explained by it avoiding slow access to qemu

Re: [Qemu-devel] [Nbd] [PATCH] doc/proto.md: restore formatting

2016-04-01 Thread Eric Blake
On 04/01/2016 07:30 AM, Eric Blake wrote: > On 04/01/2016 04:34 AM, Alex Bligh wrote: >> Restore formatting and correct name of 'length' >> >> Signed-off-by: Alex Bligh >> --- >> doc/proto.md | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > Reviewed-by: Eric Blake I've folded t

[Qemu-devel] [PATCH] doc: Cleanups to structured reads

2016-04-01 Thread Eric Blake
A couple of typos, odd formatting, and missing words made it into the structured read spec, and several potential ambiguous situations were worth rewording for clarity. Signed-off-by: Eric Blake Signed-off-by: Alex Bligh --- doc/proto.md | 112 +++

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Kevin O'Connor
On Fri, Apr 01, 2016 at 08:15:29PM +0100, Richard W.M. Jones wrote: > On Fri, Apr 01, 2016 at 08:10:48PM +0100, Richard W.M. Jones wrote: > > On Fri, Apr 01, 2016 at 03:04:15PM -0400, Kevin O'Connor wrote: > > > Otherwise, it doesn't make > > > sense that disabling CONFIG_BOOTORDER=n would change t

Re: [Qemu-devel] [PATCH] doc: Cleanups to structured reads

2016-04-01 Thread Alex Bligh
On 1 Apr 2016, at 20:39, Eric Blake wrote: > A couple of typos, odd formatting, and missing words made it into > the structured read spec, and several potential ambiguous situations > were worth rewording for clarity. > > Signed-off-by: Eric Blake > Signed-off-by: Alex Bligh Reviewed-by: Ale

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Kevin O'Connor
On Fri, Apr 01, 2016 at 07:41:31PM +0100, Richard W.M. Jones wrote: > On Fri, Apr 01, 2016 at 11:35:40AM -0400, Kevin O'Connor wrote: > > > +# general stuff > > > +CONFIG_QEMU=y > > > +CONFIG_ROM_SIZE=128 > > > > Why force a size of 128K - I would think 64K would be fine. > > Agreed. Setting thi

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Richard W.M. Jones
On Fri, Apr 01, 2016 at 03:44:14PM -0400, Kevin O'Connor wrote: [...] I ran all the tests again, but this time I ran the test program 3 times (so 30 passes for each setting). As you can see from the results below the test is not very stable, so that could easily have accounted for the variation I

Re: [Qemu-devel] [Nbd] [PATCH v2 1/1] NBD proto: add WRITE_ZEROES extension

2016-04-01 Thread Eric Blake
On 04/01/2016 02:37 AM, Wouter Verhelst wrote: > Hi, > > Thanks, applied. > > On Thu, Mar 31, 2016 at 04:02:05PM +0300, Denis V. Lunev wrote: >> From: Pavel Borzenkov >> >> There exist some cases when a client knows that the data it is going to >> write is all zeroes. Such cases include mirrorin

Re: [Qemu-devel] [PATCH 01/18 for-2.6] tests: append i386 tests

2016-04-01 Thread Eric Blake
On 04/01/2016 05:16 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Do not overwrite x86-64 tests, re-enable vhost-user-test. > > Signed-off-by: Marc-André Lureau > --- > tests/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This one should go in 2.6. Re

Re: [Qemu-devel] [PATCH 10/18] vhost-user: add slave-fd support

2016-04-01 Thread Eric Blake
On 04/01/2016 05:16 AM, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > Learn to give a socket to the slave to let him make requests to the > master. > > Signed-off-by: Marc-André Lureau > --- > docs/specs/vhost-user.txt | 23 > hw/virtio/vhost-user.c| 69

Re: [Qemu-devel] [PATCH v1 3/5] include/qemu/atomic.h: default to __atomic functions

2016-04-01 Thread Pranith Kumar
Hi Alex, I have one question inline below. Alex Bennée writes: > The __atomic primitives have been available since GCC 4.7 and provide > a richer interface for describing memory ordering requirements. As a > bonus by using the primitives instead of hand-rolled functions we can > use tools such

Re: [Qemu-devel] [PATCH] tcg/mips: Fix type of tcg_target_reg_alloc_order[]

2016-04-01 Thread Richard Henderson
On 04/01/2016 07:49 AM, James Hogan wrote: The MIPS TCG backend is the only one to have tcg_target_reg_alloc_order[] elements of type TCGReg rather than int. This resulted in commit 91478cefaaf2 ("tcg: Allocate indirect_base temporaries in a different order") breaking the build on MIPS since the

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Richard W.M. Jones
On Fri, Apr 01, 2016 at 04:05:46PM -0400, Kevin O'Connor wrote: > On Fri, Apr 01, 2016 at 07:41:31PM +0100, Richard W.M. Jones wrote: > > On Fri, Apr 01, 2016 at 11:35:40AM -0400, Kevin O'Connor wrote: > > > > +# general stuff > > > > +CONFIG_QEMU=y > > > > +CONFIG_ROM_SIZE=128 > > > > > > Why for

Re: [Qemu-devel] [PATCH for-2.6] target-i386: KVM loves Hyper-V!

2016-04-01 Thread KY Srinivasan
> -Original Message- > From: Paolo Bonzini [mailto:paolo.bonz...@gmail.com] On Behalf Of Paolo > Bonzini > Sent: Friday, April 1, 2016 1:18 AM > To: qemu-devel@nongnu.org > Cc: Andreas Färber ; Alex Williamson > ; Denis V. Lunev ; > Eduardo Habkost ; Roman Kagan > ; KY Srinivasan > Subje

Re: [Qemu-devel] [PATCH 2/3] ide: restart atapi dma by re-evaluating command packet

2016-04-01 Thread John Snow
On 04/01/2016 10:32 AM, Denis V. Lunev wrote: > From: Pavel Butsykin > > ide_atapi_dma_restart() used to just complete the DMA with an error, > under the assumption that there isn't enough information to restart it. > > However, as the contents of the ->io_buffer is preserved, it looks safe to

Re: [Qemu-devel] [Qemu-block] [PATCH] block: Move away from anglo-centricity

2016-04-01 Thread John Snow
On 04/01/2016 10:50 AM, Max Reitz wrote: > In its organizational structure, the qemu project is rather > decentralized: Many different maintainers manage their own more or less > secluded subsystems. However, regarding languages, it is still rather > anglo-centric. > > This issue has been brough

[Qemu-devel] [PATCH] doc: Flip bit sense for allowing trim during WRITE_ZEROES

2016-04-01 Thread Eric Blake
Rather than requiring allocation by default and allowing trims only on request during WRITE_ZEROES, it seems like a better default is to allow server optimizations by default and require full allocation by specific request. Since WRITE_ZEROES is experimental and has not yet been implemented, we ca

Re: [Qemu-devel] [PATCH] tcg/mips: Fix type of tcg_target_reg_alloc_order[]

2016-04-01 Thread Aurelien Jarno
On 2016-04-01 15:49, James Hogan wrote: > The MIPS TCG backend is the only one to have > tcg_target_reg_alloc_order[] elements of type TCGReg rather than int. > This resulted in commit 91478cefaaf2 ("tcg: Allocate indirect_base > temporaries in a different order") breaking the build on MIPS since t

Re: [Qemu-devel] [PATCH] doc: Flip bit sense for allowing trim during WRITE_ZEROES

2016-04-01 Thread Alex Bligh
On 1 Apr 2016, at 22:29, Eric Blake wrote: > Rather than requiring allocation by default and allowing trims > only on request during WRITE_ZEROES, it seems like a better > default is to allow server optimizations by default and require > full allocation by specific request. Since WRITE_ZEROES i

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Kevin O'Connor
On Fri, Apr 01, 2016 at 09:46:05PM +0100, Richard W.M. Jones wrote: > On Fri, Apr 01, 2016 at 04:05:46PM -0400, Kevin O'Connor wrote: > > On Fri, Apr 01, 2016 at 07:41:31PM +0100, Richard W.M. Jones wrote: > > > On Fri, Apr 01, 2016 at 11:35:40AM -0400, Kevin O'Connor wrote: > > > > > +# general st

Re: [Qemu-devel] [PATCH 3/3] ide: really restart pending and in-flight atapi dma

2016-04-01 Thread John Snow
On 04/01/2016 10:32 AM, Denis V. Lunev wrote: > From: Pavel Butsykin > > Restart of ATAPI DMA used to be unreachable, because the request to do > so wasn't indicated in bus->error_status due to the lack of spare bits, and > ide_restart_bh() would return early doing nothing. > > This patch make

Re: [Qemu-devel] Why is SeaBIOS used with -kernel?

2016-04-01 Thread Paolo Bonzini
On 01/04/2016 22:05, Kevin O'Connor wrote: > > I looked closer at your setup and it appears the SeaBIOS virtio-scsi > driver is very slow because it does a full search of all 256 possible > scsi targets. This full scan takes a lot of time. I put together a > quick patch (see below) to stop the

<    1   2   3