diff mbox series

[kirkstone] meta-ti-extras: recipes-ti: Update jailhouse SRC_URI to point to ti-jailhouse repo

Message ID 20231017114107.3791996-1-p-bhagat@ti.com
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [kirkstone] meta-ti-extras: recipes-ti: Update jailhouse SRC_URI to point to ti-jailhouse repo | expand

Commit Message

Paresh Bhagat Oct. 17, 2023, 11:41 a.m. UTC
The ti-jailhouse repo is synced with the Siemens/jailhouse
at https://github.com/siemens/jailhouse. Use this for
development and build for TI devices.

Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
---
 meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ryan Eatmon Oct. 17, 2023, 1:46 p.m. UTC | #1
On 10/17/2023 6:41 AM, Paresh Bhagat wrote:
> The ti-jailhouse repo is synced with the Siemens/jailhouse
> at https://github.com/siemens/jailhouse. Use this for
> development and build for TI devices.

Is this TI repo a perfect clone of the Siemens' repo?  Or is a loose 
copy with patches from TI on top of it?


> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
> ---
>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> index 1c63b400..a96a7831 100644
> --- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> +++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> @@ -19,7 +19,7 @@ SRCREV = "e57d1eff6d55aeed5f977fe4e2acfb6ccbdd7560"
>   BRANCH = "master"
>   
>   SRC_URI = " \
> -    git://github.com/siemens/jailhouse.git;protocol=https;branch=${BRANCH} \
> +    git://git.ti.com/git/jailhouse/ti-jailhouse.git;protocol=https;branch=${BRANCH} \
>       file://0001-configs-arm64-k3-am625-sk-Add-crypto-memory-region.patch \
>       file://0002-configs-arm64-k3-am625-sk-Switch-inmate-boot-console.patch \
>       file://0003-configs-arm64-k3-am625-sk-Add-VTM-memory-region.patch \
Denys Dmytriyenko Oct. 17, 2023, 6:07 p.m. UTC | #2
On Tue, Oct 17, 2023 at 08:46:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> 
> 
> On 10/17/2023 6:41 AM, Paresh Bhagat wrote:
> >The ti-jailhouse repo is synced with the Siemens/jailhouse
> >at https://github.com/siemens/jailhouse. Use this for
> >development and build for TI devices.
> 
> Is this TI repo a perfect clone of the Siemens' repo?  Or is a loose
> copy with patches from TI on top of it?

I don't see SRCREV being changed, which means it's a perfect clone as of now. 
In this case it doesn't make sense to deviate from upstream (yet?).


> >Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
> >---
> >  meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> >index 1c63b400..a96a7831 100644
> >--- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> >+++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
> >@@ -19,7 +19,7 @@ SRCREV = "e57d1eff6d55aeed5f977fe4e2acfb6ccbdd7560"
> >  BRANCH = "master"
> >  SRC_URI = " \
> >-    git://github.com/siemens/jailhouse.git;protocol=https;branch=${BRANCH} \
> >+    git://git.ti.com/git/jailhouse/ti-jailhouse.git;protocol=https;branch=${BRANCH} \
> >      file://0001-configs-arm64-k3-am625-sk-Add-crypto-memory-region.patch \
> >      file://0002-configs-arm64-k3-am625-sk-Switch-inmate-boot-console.patch \
> >      file://0003-configs-arm64-k3-am625-sk-Add-VTM-memory-region.patch \
Ryan Eatmon Oct. 17, 2023, 7:06 p.m. UTC | #3
On 10/17/2023 1:07 PM, Denys Dmytriyenko wrote:
> On Tue, Oct 17, 2023 at 08:46:15AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>>
>> On 10/17/2023 6:41 AM, Paresh Bhagat wrote:
>>> The ti-jailhouse repo is synced with the Siemens/jailhouse
>>> at https://github.com/siemens/jailhouse. Use this for
>>> development and build for TI devices.
>>
>> Is this TI repo a perfect clone of the Siemens' repo?  Or is a loose
>> copy with patches from TI on top of it?
> 
> I don't see SRCREV being changed, which means it's a perfect clone as of now.
> In this case it doesn't make sense to deviate from upstream (yet?).

Good point.  But the question is in the long run.  If we want this repo 
to deviate with some of our custom patches then the three patches we are 
carrying should move to that repo.  That's why I was asking.


> 
>>> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
>>> ---
>>>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>> index 1c63b400..a96a7831 100644
>>> --- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>> +++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>> @@ -19,7 +19,7 @@ SRCREV = "e57d1eff6d55aeed5f977fe4e2acfb6ccbdd7560"
>>>   BRANCH = "master"
>>>   SRC_URI = " \
>>> -    git://github.com/siemens/jailhouse.git;protocol=https;branch=${BRANCH} \
>>> +    git://git.ti.com/git/jailhouse/ti-jailhouse.git;protocol=https;branch=${BRANCH} \
>>>       file://0001-configs-arm64-k3-am625-sk-Add-crypto-memory-region.patch \
>>>       file://0002-configs-arm64-k3-am625-sk-Switch-inmate-boot-console.patch \
>>>       file://0003-configs-arm64-k3-am625-sk-Add-VTM-memory-region.patch \
Paresh Bhagat Oct. 18, 2023, 6:51 a.m. UTC | #4
Hi Ryan and Denys,

On 18/10/23 00:36, Ryan Eatmon wrote:
>
>
> On 10/17/2023 1:07 PM, Denys Dmytriyenko wrote:
>> On Tue, Oct 17, 2023 at 08:46:15AM -0500, Ryan Eatmon via 
>> lists.yoctoproject.org wrote:
>>>
>>>
>>> On 10/17/2023 6:41 AM, Paresh Bhagat wrote:
>>>> The ti-jailhouse repo is synced with the Siemens/jailhouse
>>>> at https://github.com/siemens/jailhouse. Use this for
>>>> development and build for TI devices.
>>>
>>> Is this TI repo a perfect clone of the Siemens' repo?  Or is a loose
>>> copy with patches from TI on top of it?
>>
>> I don't see SRCREV being changed, which means it's a perfect clone as 
>> of now.
>> In this case it doesn't make sense to deviate from upstream (yet?).
>
> Good point.  But the question is in the long run.  If we want this 
> repo to deviate with some of our custom patches then the three patches 
> we are carrying should move to that repo.  That's why I was asking.
Agreed the patches will be moved to the TI repo. As of now its a perfect 
clone of the Siemens repo which does not make sense but we will also 
have custom patches in the TI repo for upcoming releases.
>
>
>>
>>>> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
>>>> ---
>>>>   meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb 
>>>> b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>>> index 1c63b400..a96a7831 100644
>>>> --- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>>> +++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
>>>> @@ -19,7 +19,7 @@ SRCREV = "e57d1eff6d55aeed5f977fe4e2acfb6ccbdd7560"
>>>>   BRANCH = "master"
>>>>   SRC_URI = " \
>>>> - 
>>>> git://github.com/siemens/jailhouse.git;protocol=https;branch=${BRANCH} 
>>>> \
>>>> + 
>>>> git://git.ti.com/git/jailhouse/ti-jailhouse.git;protocol=https;branch=${BRANCH} 
>>>> \
>>>> file://0001-configs-arm64-k3-am625-sk-Add-crypto-memory-region.patch \
>>>> file://0002-configs-arm64-k3-am625-sk-Switch-inmate-boot-console.patch 
>>>> \
>>>> file://0003-configs-arm64-k3-am625-sk-Add-VTM-memory-region.patch \
>
diff mbox series

Patch

diff --git a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
index 1c63b400..a96a7831 100644
--- a/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
+++ b/meta-ti-extras/recipes-ti/jailhouse/jailhouse_git.bb
@@ -19,7 +19,7 @@  SRCREV = "e57d1eff6d55aeed5f977fe4e2acfb6ccbdd7560"
 BRANCH = "master"
 
 SRC_URI = " \
-    git://github.com/siemens/jailhouse.git;protocol=https;branch=${BRANCH} \
+    git://git.ti.com/git/jailhouse/ti-jailhouse.git;protocol=https;branch=${BRANCH} \
     file://0001-configs-arm64-k3-am625-sk-Add-crypto-memory-region.patch \
     file://0002-configs-arm64-k3-am625-sk-Switch-inmate-boot-console.patch \
     file://0003-configs-arm64-k3-am625-sk-Add-VTM-memory-region.patch \