diff mbox series

config: update Mini-OS commit

Message ID 20240207133106.6185-1-jgross@suse.com (mailing list archive)
State Superseded
Headers show
Series config: update Mini-OS commit | expand

Commit Message

Jürgen Groß Feb. 7, 2024, 1:31 p.m. UTC 
Update the Mini-OS upstream revision.

Signed-off-by: Juergen Gross <jgross@suse.com>
---
 Config.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Julien Grall Feb. 7, 2024, 1:41 p.m. UTC | #1 
Hi Juergen,

On 07/02/2024 13:31, Juergen Gross wrote:
> Update the Mini-OS upstream revision.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>
> ---
>   Config.mk | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Config.mk b/Config.mk
> index f7d6d84847..077d841bb7 100644
> --- a/Config.mk
> +++ b/Config.mk
> @@ -224,7 +224,7 @@ QEMU_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/qemu-xen.git
>   QEMU_UPSTREAM_REVISION ?= master
>   
>   MINIOS_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/mini-os.git
> -MINIOS_UPSTREAM_REVISION ?= 090eeeb1631f00a9a41ebf66d9b4aacb97eb51e7
> +MINIOS_UPSTREAM_REVISION ?= b119f0178fd86876d0678007dfcf435ab8bb7568

I looked at the changes and I am puzzled by one. In p9_stat(), I see a 
call with:

   free_stat(&stat);

'stat' is defined as 'struct p9_stat *stat' and 'free_stat' expects a 
'struct p9_stat'.

Wouldn't this resuilt to a build error?

Cheers,
Jan Beulich Feb. 7, 2024, 1:43 p.m. UTC | #2 
On 07.02.2024 14:31, Juergen Gross wrote:
> Update the Mini-OS upstream revision.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>

Acked-by: Jan Beulich <jbeulich@suse.com>
Jürgen Groß Feb. 7, 2024, 1:45 p.m. UTC | #3 
On 07.02.24 14:41, Julien Grall wrote:
> Hi Juergen,
> 
> On 07/02/2024 13:31, Juergen Gross wrote:
>> Update the Mini-OS upstream revision.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
>> ---
>>   Config.mk | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/Config.mk b/Config.mk
>> index f7d6d84847..077d841bb7 100644
>> --- a/Config.mk
>> +++ b/Config.mk
>> @@ -224,7 +224,7 @@ QEMU_UPSTREAM_URL ?= 
>> https://xenbits.xen.org/git-http/qemu-xen.git
>>   QEMU_UPSTREAM_REVISION ?= master
>>   MINIOS_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/mini-os.git
>> -MINIOS_UPSTREAM_REVISION ?= 090eeeb1631f00a9a41ebf66d9b4aacb97eb51e7
>> +MINIOS_UPSTREAM_REVISION ?= b119f0178fd86876d0678007dfcf435ab8bb7568
> 
> I looked at the changes and I am puzzled by one. In p9_stat(), I see a call with:
> 
>    free_stat(&stat);
> 
> 'stat' is defined as 'struct p9_stat *stat' and 'free_stat' expects a 'struct 
> p9_stat'.
> 
> Wouldn't this resuilt to a build error?

Hmm, I'm puzzled, too. I didn't see a build error when doing "make testbuild".

But I agree that this looks very suspicious.

Oh, in test builds we don't have HAVE_LIBC defined. I just verified that
9pfront.c gets compiled, while I didn't pay attention to the parts which were
compiled during the build. Shame on me!

Fixup patch coming soon...

Thanks for noticing!


Juergen
diff mbox series

Patch

diff --git a/Config.mk b/Config.mk
index f7d6d84847..077d841bb7 100644
--- a/Config.mk
+++ b/Config.mk
@@ -224,7 +224,7 @@  QEMU_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/qemu-xen.git
 QEMU_UPSTREAM_REVISION ?= master
 
 MINIOS_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/mini-os.git
-MINIOS_UPSTREAM_REVISION ?= 090eeeb1631f00a9a41ebf66d9b4aacb97eb51e7
+MINIOS_UPSTREAM_REVISION ?= b119f0178fd86876d0678007dfcf435ab8bb7568
 
 SEABIOS_UPSTREAM_URL ?= https://xenbits.xen.org/git-http/seabios.git
 SEABIOS_UPSTREAM_REVISION ?= rel-1.16.3