On 28 Mar 2012, at 15:26, Alon Levy wrote:
> Signed-off-by: Alon Levy
> ---
> scripts/tracetool |6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/scripts/tracetool b/scripts/tracetool
> index 65bd0a1..e7cebf3 100755
> --- a/scripts/tracetool
> +++ b/scripts/tracet
On 27/03/2012 12:31, Andreas Färber wrote:
Am 27.03.2012 09:23, schrieb Stefan Hajnoczi:
On Sat, Mar 24, 2012 at 04:26:27PM +, Lee Essen wrote:
libsocket and libxnet are required for base network functionality
used in os_dep.c, qemu-socket.c, qga/commands-posix.c and cutils.c
Signed-off
On 27/03/2012 08:29, Stefan Hajnoczi wrote:
On Sat, Mar 24, 2012 at 04:26:28PM +, Lee Essen wrote:
sigbus_reraise is used by the kvm_wait_io_event function and is
needed on both Linux and Solaris. This patch adds CONFIG_SOLARIS
to the current CONFIG_LINUX only ifdef.
Signed-off-by: Lee
Solaris requires a -m32/-m64 flag to determin if building for
32bit or 64bit environments. This patch add a CONFIG_DTRACE_FLAGS
variable to configure and then makes use of that in the relevant
Makefiles and rules.mak
Signed-off-by: Lee Essen
---
Makefile.objs |2 +-
configure |8
Solaris requires that dtrace is supplied with all objects when
generating the dtrace object file. This patch updates the build
for each of the utility binaries and the main target generation.
Signed-off-by: Lee Essen
---
Makefile| 37
On 26/03/2012 11:14, Stefan Hajnoczi wrote:
On Fri, Mar 23, 2012 at 2:11 PM, Lee Essen wrote:
On 23 Mar 2012, at 08:08, Stefan Hajnoczi wrote:
On Thu, Mar 22, 2012 at 05:00:53PM +, Lee Essen wrote:
On 22/03/2012 16:28, Stefan Hajnoczi wrote:
On Wed, Mar 21, 2012 at 1:01 PM, Andreas
libsocket and libxnet are required for base network functionality
used in os_dep.c, qemu-socket.c, qga/commands-posix.c and cutils.c
Signed-off-by: Lee Essen
---
configure |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/configure b/configure
index 8b4e3c1..152adaa
Solaris does not support the O_ASYNC option to open, this patch
adds the same functionality through the I_SETSIG ioctl.
Signed-off-by: Lee Essen
---
qga/channel-posix.c | 16
1 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/qga/channel-posix.c b/qga/channel
Dynticks was limited to linux. This patch adds Solaris support
and ensures a CLOCK_HIGHRES clock is used which is the optimal
setup for Solaris systems.
Signed-off-by: Lee Essen
---
qemu-timer.c | 14 +-
1 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/qemu-timer.c b
sigbus_reraise is used by the kvm_wait_io_event function and is
needed on both Linux and Solaris. This patch adds CONFIG_SOLARIS
to the current CONFIG_LINUX only ifdef.
Signed-off-by: Lee Essen
---
cpus.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/cpus.c b
On 24 Mar 2012, at 15:44, Andreas Färber wrote:
> Am 24.03.2012 15:51, schrieb Lee Essen:
>> On 24 Mar 2012, at 12:56, Blue Swirl wrote:
>>
>>> On Sat, Mar 17, 2012 at 08:04, Lee Essen wrote:
>>>> (third email attempt, apologies if you get duplicates)
&
On 24 Mar 2012, at 12:56, Blue Swirl wrote:
> On Sat, Mar 17, 2012 at 08:04, Lee Essen wrote:
>> (third email attempt, apologies if you get duplicates)
>
> The patch does not apply:
> git am:
> Applying: Basic Illumos support
> fatal: corrupt patch at line 19
>
On 23 Mar 2012, at 08:08, Stefan Hajnoczi wrote:
> On Thu, Mar 22, 2012 at 05:00:53PM +0000, Lee Essen wrote:
>> On 22/03/2012 16:28, Stefan Hajnoczi wrote:
>>> On Wed, Mar 21, 2012 at 1:01 PM, Andreas Färber wrote:
>>>> Hi,
>>>>
>>>> A
On 23 Mar 2012, at 10:18, Alon Levy wrote:
> On Fri, Mar 23, 2012 at 08:10:36AM +0000, Lee Essen wrote:
>> Hi,
>>
>> I think I've found a bug with the way that spice uses qemu_name.
>>
>> qemu_name is a char *, that's only set to if "-
Hi,
I think I've found a bug with the way that spice uses qemu_name.
qemu_name is a char *, that's only set to if "-name" is given (and then the arg
is strdup'd), otherwise it's not set properly.
In ui/spice_core.c spice_server_set_name() is called with qemu_name, which if
not set causes a co
On 22/03/2012 16:28, Stefan Hajnoczi wrote:
On Wed, Mar 21, 2012 at 1:01 PM, Andreas Färber wrote:
Hi,
Am 21.03.2012 11:45, schrieb Lee Essen:
I've been trying to find a sensible way to solve the Solaris/Illumos
dtrace requirement to pass all the objs to the dtrace command so tha
On 22/03/2012 08:28, Stefan Hajnoczi wrote:
On Wed, Mar 21, 2012 at 05:08:15PM +, Lee Essen wrote:
On Solaris/Illumos dtrace will remove the "const" qualifier from any
arguments when producing the header file, this results in hundreds
of compile-time warnings.
I wonder why D
On Solaris/Illumos dtrace will remove the "const" qualifier from any
arguments when producing the header file, this results in hundreds
of compile-time warnings.
I have put together a patch to tracetool to cast any const argument
appropriately to remove the warnings, but I don't know how it beha
On 21/03/2012 13:01, Andreas Färber wrote:
Hi,
Am 21.03.2012 11:45, schrieb Lee Essen:
I've been trying to find a sensible way to solve the Solaris/Illumos
dtrace requirement to pass all the objs to the dtrace command so that
the resultant object file contains all the symbols need
Hi,
I've been trying to find a sensible way to solve the Solaris/Illumos
dtrace requirement to pass all the objs to the dtrace command so that
the resultant object file contains all the symbols needed to properly
link the relevant binary.
The easiest way to do this is just prior to linking t
On 20/03/2012 16:59, Stefan Hajnoczi wrote:
On Mon, Mar 19, 2012 at 1:35 PM, Lee Essen wrote:
On 19 Mar 2012, at 12:32, Andreas Färber wrote:
Am 19.03.2012 13:05, schrieb Lee Essen:
In my original (way-too-long) patch for Illumos I included fixes for
both of the above, the approach I took
If there are "disabled" entries in the trace-events file then
linetod_nop() is called if the backend is dtrace, it's currently
not present. Also equivalent fix for stap.
Signed-off-by: Lee Essen
--
diff --git a/scripts/tracetool b/scripts/tracetool
index 65bd0a1..a279c2c 10075
If there are "disabled" entries in the trace-events file then
linetod_nop() is called if the backend is dtrace, it's currently
not present.
Signed-off-by: Lee Essen
--
diff --git a/scripts/tracetool b/scripts/tracetool
index 65bd0a1..d5e5591 100755
--- a/scripts/tracetoo
On 19 Mar 2012, at 12:32, Andreas Färber wrote:
> Am 19.03.2012 13:05, schrieb Lee Essen:
>> On 19 Mar 2012, at 11:59, Stefan Hajnoczi wrote:
>>
>>> I'm going to spend some time today reviewing recent tracing patches.
>>> I'd prefer to move to a Py
On 19 Mar 2012, at 11:59, Stefan Hajnoczi wrote:
> On Fri, Mar 16, 2012 at 12:29 PM, Lee Essen wrote:
>> Signed-off-by: Lee Essen
>>
>> ---
>>
>> scripts/tracetool |2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> I'm going
e are lots of __linux__ || __sun__ constructs, I wanted
to make sure I didn't alter the linux behaviour and this seemed the safest
way to do it.
Signed-off-by: Lee Essen
---
configure|1 +
cpus.c |4 +++-
qemu-timer.c | 14 +-
qga/chann
On 16 Mar 2012, at 11:56, Paolo Bonzini wrote:
> Il 16/03/2012 10:23, Lee Essen ha scritto:
>> +while (mlock(base, (nbytes = step * ps)) == -1) {
>> +if (errno != EAGAIN) {
>> +return -1;
>> +}
>> +
>> +if (
On 16 Mar 2012, at 16:28, Paolo Bonzini wrote:
> Il 16/03/2012 14:46, Lee Essen ha scritto:
>> It does indeed … this came from the original Joyent code, I must
>> admit I did wonder whether gcc would optimise it away. I did consider
>> adding something to stop gcc complaini
On 16 Mar 2012, at 13:14, Jan Kiszka wrote:
> On 2012-03-16 10:23, Lee Essen wrote:
>> +#ifdef __sun__
>> +#include
>> +#else
>> #include
>> #include
>> +#endif
>
> As Paolo already said, this should somehow be centralised.
Yep, fair point. I
On 16 Mar 2012, at 12:44, Andreas Färber wrote:
> Am 16.03.2012 13:29, schrieb Lee Essen:
>> Signed-off-by: Lee Essen
>>
>> ---
>>
>> scripts/tracetool |2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/scr
Signed-off-by: Lee Essen
---
scripts/tracetool |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/scripts/tracetool b/scripts/tracetool
index 65bd0a1..2e43d05 100755
--- a/scripts/tracetool
+++ b/scripts/tracetool
@@ -123,7 +123,7 @@ get_argc()
# Get the format string
On 16 Mar 2012, at 12:20, Lee Essen wrote:
>
> On 16 Mar 2012, at 12:14, Andreas Färber wrote:
>
>> Am 16.03.2012 13:02, schrieb Lee Essen:
>>>
>>> target_list=""
>>>
>>> @@ -442,6 +443,7 @@ SunOS)
>>> # have to selec
On 16 Mar 2012, at 12:14, Andreas Färber wrote:
> Am 16.03.2012 13:02, schrieb Lee Essen:
>>
>> target_list=""
>>
>> @@ -442,6 +443,7 @@ SunOS)
>> # have to select again, because `uname -m` returns i86pc
>> # even on an x86_64 box.
>>
Use the SHELL macro (set in configure) to ensure tracetool and hxtool are
correctly called.
Signed-off-by: Lee Essen
---
Makefile.objs |6 +++---
Makefile.target |6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/Makefile.objs b/Makefile.objs
index 226b01d
Adds support to configure for controlling which shell to use, defaults to "sh"
as before
but adds "bash" for Solaris/Illumos builds. Plus ensures that tracetool is
called with a
shell.
Signed-off-by: Lee Essen
--
configure |7 +--
1 files changed, 5 insert
On 15/03/2012 13:12, Andreas Färber wrote:
Something went wrong with your reply... While it contains a quote it
does not reference the mail you replied to, that way messages get lost
among the constant chatter on qemu-devel.
Hmmm ... sorry -- I'm having to cut and paste emails between mail clien
Resend -- not sure what happened to my first mail...
Hi,
I've been working on getting the mainline qemu tree working on SmartOS
(illumos), the current qemu support seems to be a branch of qemu-kvm and we've
been having some problems getting spice working, so moving to the main qemu
tree seemed
Hi,
I've been working on getting the mainline qemu tree working on SmartOS
(illumos), the current qemu support seems to be a branch of qemu-kvm and we've
been having some problems getting spice working, so moving to the main qemu
tree seemed sensible.
Anyway, couple of questions…
1. Are you h
On 15/03/2012 11:56, Stefan Hajnoczi wrote:
Anyway, couple of questions…
1. Are you happy to accept patches that enable this?
- There are a few build related things that don't work … assumption of sh
being bash, missing -64 argument to dtrace etc.
Sounds good, but have you checked the Joyen
Resend from different email -- not sure what happened to my first 2 attempts!
Hi,
I've been working on getting the mainline qemu tree working on SmartOS (illumos), the current qemu support seems to be a branch of qemu-kvm and we've been having some problems getting spice working, so moving to the
40 matches
Mail list logo