qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
@ 2012-07-25 15:29 Peter Maydell
  2012-07-28 10:20 ` Jan Kiszka
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Peter Maydell @ 2012-07-25 15:29 UTC (permalink / raw)
  To: qemu-devel; +Cc: Jan Kiszka, Rusty Russell, patches

Add asm-generic/kvm_para.h to the set of non-architecture specific
KVM kernel headers we copy into QEMU. This header may be included
by an architecture's kvm_para.h header.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 scripts/update-linux-headers.sh |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
index 9d2a4bc..a639c5b 100755
--- a/scripts/update-linux-headers.sh
+++ b/scripts/update-linux-headers.sh
@@ -46,6 +46,11 @@ mkdir -p "$output/linux-headers/linux"
 for header in kvm.h kvm_para.h vhost.h virtio_config.h virtio_ring.h; do
     cp "$tmpdir/include/linux/$header" "$output/linux-headers/linux"
 done
+rm -rf "$output/linux-headers/asm-generic"
+mkdir -p "$output/linux-headers/asm-generic"
+for header in kvm_para.h; do
+    cp "$tmpdir/include/asm-generic/$header" "$output/linux-headers/asm-generic"
+done
 if [ -L "$linux/source" ]; then
     cp "$linux/source/COPYING" "$output/linux-headers"
 else
-- 
1.7.5.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-07-25 15:29 [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h Peter Maydell
@ 2012-07-28 10:20 ` Jan Kiszka
  2012-08-08 12:34 ` Peter Maydell
  2012-08-14 22:23 ` Marcelo Tosatti
  2 siblings, 0 replies; 7+ messages in thread
From: Jan Kiszka @ 2012-07-28 10:20 UTC (permalink / raw)
  To: Peter Maydell; +Cc: qemu-devel, Rusty Russell, patches

[-- Attachment #1: Type: text/plain, Size: 1205 bytes --]

On 2012-07-25 17:29, Peter Maydell wrote:
> Add asm-generic/kvm_para.h to the set of non-architecture specific
> KVM kernel headers we copy into QEMU. This header may be included
> by an architecture's kvm_para.h header.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  scripts/update-linux-headers.sh |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
> index 9d2a4bc..a639c5b 100755
> --- a/scripts/update-linux-headers.sh
> +++ b/scripts/update-linux-headers.sh
> @@ -46,6 +46,11 @@ mkdir -p "$output/linux-headers/linux"
>  for header in kvm.h kvm_para.h vhost.h virtio_config.h virtio_ring.h; do
>      cp "$tmpdir/include/linux/$header" "$output/linux-headers/linux"
>  done
> +rm -rf "$output/linux-headers/asm-generic"
> +mkdir -p "$output/linux-headers/asm-generic"
> +for header in kvm_para.h; do
> +    cp "$tmpdir/include/asm-generic/$header" "$output/linux-headers/asm-generic"
> +done
>  if [ -L "$linux/source" ]; then
>      cp "$linux/source/COPYING" "$output/linux-headers"
>  else
> 

Reviewed-by: Jan Kiszka <jan.kiszka@siemens.com>



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-07-25 15:29 [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h Peter Maydell
  2012-07-28 10:20 ` Jan Kiszka
@ 2012-08-08 12:34 ` Peter Maydell
  2012-08-13  9:24   ` Peter Maydell
  2012-08-14 22:23 ` Marcelo Tosatti
  2 siblings, 1 reply; 7+ messages in thread
From: Peter Maydell @ 2012-08-08 12:34 UTC (permalink / raw)
  To: qemu-devel; +Cc: Jan Kiszka, Rusty Russell, patches

Ping?

patchwork url: http://patchwork.ozlabs.org/patch/173202/

-- PMM

On 25 July 2012 16:29, Peter Maydell <peter.maydell@linaro.org> wrote:
> Add asm-generic/kvm_para.h to the set of non-architecture specific
> KVM kernel headers we copy into QEMU. This header may be included
> by an architecture's kvm_para.h header.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  scripts/update-linux-headers.sh |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
> index 9d2a4bc..a639c5b 100755
> --- a/scripts/update-linux-headers.sh
> +++ b/scripts/update-linux-headers.sh
> @@ -46,6 +46,11 @@ mkdir -p "$output/linux-headers/linux"
>  for header in kvm.h kvm_para.h vhost.h virtio_config.h virtio_ring.h; do
>      cp "$tmpdir/include/linux/$header" "$output/linux-headers/linux"
>  done
> +rm -rf "$output/linux-headers/asm-generic"
> +mkdir -p "$output/linux-headers/asm-generic"
> +for header in kvm_para.h; do
> +    cp "$tmpdir/include/asm-generic/$header" "$output/linux-headers/asm-generic"
> +done
>  if [ -L "$linux/source" ]; then
>      cp "$linux/source/COPYING" "$output/linux-headers"
>  else
> --
> 1.7.5.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-08-08 12:34 ` Peter Maydell
@ 2012-08-13  9:24   ` Peter Maydell
  2012-08-13 14:24     ` Anthony Liguori
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Maydell @ 2012-08-13  9:24 UTC (permalink / raw)
  To: qemu-devel; +Cc: Anthony Liguori, Jan Kiszka, Rusty Russell, patches

Ping^2 ?

On 8 August 2012 13:34, Peter Maydell <peter.maydell@linaro.org> wrote:
> Ping?
>
> patchwork url: http://patchwork.ozlabs.org/patch/173202/
>
> -- PMM
>
> On 25 July 2012 16:29, Peter Maydell <peter.maydell@linaro.org> wrote:
>> Add asm-generic/kvm_para.h to the set of non-architecture specific
>> KVM kernel headers we copy into QEMU. This header may be included
>> by an architecture's kvm_para.h header.
>>
>> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
>> ---
>>  scripts/update-linux-headers.sh |    5 +++++
>>  1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
>> index 9d2a4bc..a639c5b 100755
>> --- a/scripts/update-linux-headers.sh
>> +++ b/scripts/update-linux-headers.sh
>> @@ -46,6 +46,11 @@ mkdir -p "$output/linux-headers/linux"
>>  for header in kvm.h kvm_para.h vhost.h virtio_config.h virtio_ring.h; do
>>      cp "$tmpdir/include/linux/$header" "$output/linux-headers/linux"
>>  done
>> +rm -rf "$output/linux-headers/asm-generic"
>> +mkdir -p "$output/linux-headers/asm-generic"
>> +for header in kvm_para.h; do
>> +    cp "$tmpdir/include/asm-generic/$header" "$output/linux-headers/asm-generic"
>> +done
>>  if [ -L "$linux/source" ]; then
>>      cp "$linux/source/COPYING" "$output/linux-headers"
>>  else
>> --
>> 1.7.5.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-08-13  9:24   ` Peter Maydell
@ 2012-08-13 14:24     ` Anthony Liguori
  2012-08-13 16:05       ` Peter Maydell
  0 siblings, 1 reply; 7+ messages in thread
From: Anthony Liguori @ 2012-08-13 14:24 UTC (permalink / raw)
  To: Peter Maydell, qemu-devel
  Cc: Marcelo Tosatti, Jan Kiszka, Avi Kivity, Rusty Russell, patches

Peter Maydell <peter.maydell@linaro.org> writes:

> Ping^2 ?

In a previous thread, we all agreed that all changes to linux headers
would come in through uq/master to ensure that we didn't have a repeat
scenario of depending on a header that didn't make it to Avi's tree
unchanged.

Avi/Marcelo, can you guys pick this up through uq/master?

Regards,

Anthony Liguori

>
> On 8 August 2012 13:34, Peter Maydell <peter.maydell@linaro.org> wrote:
>> Ping?
>>
>> patchwork url: http://patchwork.ozlabs.org/patch/173202/
>>
>> -- PMM
>>
>> On 25 July 2012 16:29, Peter Maydell <peter.maydell@linaro.org> wrote:
>>> Add asm-generic/kvm_para.h to the set of non-architecture specific
>>> KVM kernel headers we copy into QEMU. This header may be included
>>> by an architecture's kvm_para.h header.
>>>
>>> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
>>> ---
>>>  scripts/update-linux-headers.sh |    5 +++++
>>>  1 files changed, 5 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
>>> index 9d2a4bc..a639c5b 100755
>>> --- a/scripts/update-linux-headers.sh
>>> +++ b/scripts/update-linux-headers.sh
>>> @@ -46,6 +46,11 @@ mkdir -p "$output/linux-headers/linux"
>>>  for header in kvm.h kvm_para.h vhost.h virtio_config.h virtio_ring.h; do
>>>      cp "$tmpdir/include/linux/$header" "$output/linux-headers/linux"
>>>  done
>>> +rm -rf "$output/linux-headers/asm-generic"
>>> +mkdir -p "$output/linux-headers/asm-generic"
>>> +for header in kvm_para.h; do
>>> +    cp "$tmpdir/include/asm-generic/$header" "$output/linux-headers/asm-generic"
>>> +done
>>>  if [ -L "$linux/source" ]; then
>>>      cp "$linux/source/COPYING" "$output/linux-headers"
>>>  else
>>> --
>>> 1.7.5.4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-08-13 14:24     ` Anthony Liguori
@ 2012-08-13 16:05       ` Peter Maydell
  0 siblings, 0 replies; 7+ messages in thread
From: Peter Maydell @ 2012-08-13 16:05 UTC (permalink / raw)
  To: Anthony Liguori
  Cc: patches, Marcelo Tosatti, qemu-devel, Jan Kiszka, Avi Kivity,
	Rusty Russell

On 13 August 2012 15:24, Anthony Liguori <aliguori@us.ibm.com> wrote:
> Peter Maydell <peter.maydell@linaro.org> writes:
>> [re patchwork url: http://patchwork.ozlabs.org/patch/173202/ ]
>> Ping^2 ?
>
> In a previous thread, we all agreed that all changes to linux headers
> would come in through uq/master to ensure that we didn't have a repeat
> scenario of depending on a header that didn't make it to Avi's tree
> unchanged.
>
> Avi/Marcelo, can you guys pick this up through uq/master?

Applies also to http://patchwork.ozlabs.org/patch/171628/
I guess.

thanks
-- PMM

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h
  2012-07-25 15:29 [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h Peter Maydell
  2012-07-28 10:20 ` Jan Kiszka
  2012-08-08 12:34 ` Peter Maydell
@ 2012-08-14 22:23 ` Marcelo Tosatti
  2 siblings, 0 replies; 7+ messages in thread
From: Marcelo Tosatti @ 2012-08-14 22:23 UTC (permalink / raw)
  To: Peter Maydell; +Cc: Jan Kiszka, qemu-devel, Rusty Russell, patches

On Wed, Jul 25, 2012 at 04:29:07PM +0100, Peter Maydell wrote:
> Add asm-generic/kvm_para.h to the set of non-architecture specific
> KVM kernel headers we copy into QEMU. This header may be included
> by an architecture's kvm_para.h header.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  scripts/update-linux-headers.sh |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)

Applied, thanks.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2012-08-14 22:53 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-25 15:29 [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h Peter Maydell
2012-07-28 10:20 ` Jan Kiszka
2012-08-08 12:34 ` Peter Maydell
2012-08-13  9:24   ` Peter Maydell
2012-08-13 14:24     ` Anthony Liguori
2012-08-13 16:05       ` Peter Maydell
2012-08-14 22:23 ` Marcelo Tosatti

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).