Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-26 Thread Marc-André Lureau
Hi On Mon, Oct 26, 2015 at 10:58 AM, Peter Maydell wrote: > This will print a cryptic error message and then not fail the test, > which is not great. Maybe that's ok for the moment in the interests > of not keeping this huge patchset out of tree for too long[*], but > we should look at what glib'

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-26 Thread Peter Maydell
On 26 October 2015 at 09:28, Marc-André Lureau wrote: > Hi Peter, > > On Mon, Oct 19, 2015 at 5:57 PM, Peter Maydell > wrote: >> >> What is happening here is that we are looping infinitely in >> mktempshmem() because shm_open() returns -1 with errno ENOSYS, >> and there's no code in the loop tha

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-26 Thread Marc-André Lureau
Hi Peter, On Mon, Oct 19, 2015 at 5:57 PM, Peter Maydell wrote: > > What is happening here is that we are looping infinitely in > mktempshmem() because shm_open() returns -1 with errno ENOSYS, > and there's no code in the loop that stops the loop on anything > except shm_open succeeding, or even

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-19 Thread Peter Maydell
On 16 October 2015 at 12:26, Peter Maydell wrote: > On 32-bit ARM the test-ivshmem program got stuck somehow (it had been running > for an hour when I killed it). It didn't write anything to the log, > I'm afraid. What is happening here is that we are looping infinitely in mktempshmem() because s

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-19 Thread Peter Maydell
On 14 October 2015 at 15:49, Marc-André Lureau wrote: > The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10: > > Merge remote-tracking branch > 'remotes/armbru/tags/pull-qapi-2015-10-12' into staging (2015-10-13 > 10:42:06 +0100) > > are available in the git repository at

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-17 Thread Peter Maydell
On 16 October 2015 at 15:47, Peter Maydell wrote: > On 16 October 2015 at 12:26, Peter Maydell wrote: >> On 14 October 2015 at 15:49, Marc-André Lureau >> wrote: >>> The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10: >>> >>> Merge remote-tracking branch >>> 'remotes/a

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-16 Thread Peter Maydell
On 16 October 2015 at 12:26, Peter Maydell wrote: > On 14 October 2015 at 15:49, Marc-André Lureau > wrote: >> The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10: >> >> Merge remote-tracking branch >> 'remotes/armbru/tags/pull-qapi-2015-10-12' into staging (2015-10-13 >

Re: [Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-16 Thread Peter Maydell
On 14 October 2015 at 15:49, Marc-André Lureau wrote: > The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10: > > Merge remote-tracking branch > 'remotes/armbru/tags/pull-qapi-2015-10-12' into staging (2015-10-13 > 10:42:06 +0100) > > are available in the git repository at

[Qemu-devel] [PULL v4 00/52] Ivshmem patches

2015-10-14 Thread Marc-André Lureau
The following changes since commit c49d3411faae8ffaab8f7e5db47405a008411c10: Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-10-12' into staging (2015-10-13 10:42:06 +0100) are available in the git repository at: https://github.com/elmarco/qemu tags/ivshmem-pull-request for