[Qemu-devel] QEMU dtc submodule

[Qemu-devel] QEMU dtc submodule

[Leon Alrae]

Hi,

What's the procedure to update / who can update QEMU's dtc mirror
git://git.qemu.org/dtc.git which is used as submodule?

There's a patch series relying on the dtc v1.4.2 tag:
https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg01815.html
but the dtc mirror is outdated and that tag is not there.

Thanks,
Leon

Re: [Qemu-devel] QEMU dtc submodule

[Thomas Huth]

On Wed, 21 Sep 2016 12:44:59 +0100
Leon Alrae <leon.alrae@imgtec.com> wrote:

> Hi,
> 
> What's the procedure to update / who can update QEMU's dtc mirror
> git://git.qemu.org/dtc.git which is used as submodule?
> 
> There's a patch series relying on the dtc v1.4.2 tag:
> https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg01815.html
> but the dtc mirror is outdated and that tag is not there.

Last time, Alex Graf was taking care of the update request from me:

https://lists.gnu.org/archive/html/qemu-ppc/2015-05/msg00232.html

... so maybe he could help here? If not, try to ask in #qemu IRC
channel, I'm pretty sure that Stefan, Peter or somebody else can help
here.

 Thomas

Re: [Qemu-devel] QEMU dtc submodule

[Alexander Graf]

On 09/22/2016 10:11 AM, Thomas Huth wrote:
> On Wed, 21 Sep 2016 12:44:59 +0100
> Leon Alrae <leon.alrae@imgtec.com> wrote:
>
>> Hi,
>>
>> What's the procedure to update / who can update QEMU's dtc mirror
>> git://git.qemu.org/dtc.git which is used as submodule?
>>
>> There's a patch series relying on the dtc v1.4.2 tag:
>> https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg01815.html
>> but the dtc mirror is outdated and that tag is not there.
> Last time, Alex Graf was taking care of the update request from me:
>
> https://lists.gnu.org/archive/html/qemu-ppc/2015-05/msg00232.html
>
> ... so maybe he could help here? If not, try to ask in #qemu IRC
> channel, I'm pretty sure that Stefan, Peter or somebody else can help
> here.

Yes, poke on IRC. Stefan used to handle the git sync, but IIUC he passed 
that on. Once the sync is there, just send a patch that changes the 
submodule reference.


Alex

Re: [Qemu-devel] QEMU dtc submodule

[Stefan Hajnoczi]

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

On Thu, Sep 22, 2016 at 10:42:40AM +0200, Alexander Graf wrote:
> On 09/22/2016 10:11 AM, Thomas Huth wrote:
> > On Wed, 21 Sep 2016 12:44:59 +0100
> > Leon Alrae <leon.alrae@imgtec.com> wrote:
> > 
> > > Hi,
> > > 
> > > What's the procedure to update / who can update QEMU's dtc mirror
> > > git://git.qemu.org/dtc.git which is used as submodule?
> > > 
> > > There's a patch series relying on the dtc v1.4.2 tag:
> > > https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg01815.html
> > > but the dtc mirror is outdated and that tag is not there.
> > Last time, Alex Graf was taking care of the update request from me:
> > 
> > https://lists.gnu.org/archive/html/qemu-ppc/2015-05/msg00232.html
> > 
> > ... so maybe he could help here? If not, try to ask in #qemu IRC
> > channel, I'm pretty sure that Stefan, Peter or somebody else can help
> > here.
> 
> Yes, poke on IRC. Stefan used to handle the git sync, but IIUC he passed
> that on. Once the sync is there, just send a patch that changes the
> submodule reference.

Hi Guys,
Jeff Cody is the QEMU systems administrator.  He can adjust git mirrors
for qit.qemu-project.org.

Leon: Please give the git URL and branch that should be mirrored.  It
cannot be a tag since that is immutable.  Instead it should be the
dtc development/release tree that will be updated in the future.

Stefan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]

Re: [Qemu-devel] QEMU dtc submodule

[Jeff Cody]

On Fri, Sep 23, 2016 at 02:04:47PM +0100, Stefan Hajnoczi wrote:
> On Thu, Sep 22, 2016 at 10:42:40AM +0200, Alexander Graf wrote:
> > On 09/22/2016 10:11 AM, Thomas Huth wrote:
> > > On Wed, 21 Sep 2016 12:44:59 +0100
> > > Leon Alrae <leon.alrae@imgtec.com> wrote:
> > > 
> > > > Hi,
> > > > 
> > > > What's the procedure to update / who can update QEMU's dtc mirror
> > > > git://git.qemu.org/dtc.git which is used as submodule?
> > > > 
> > > > There's a patch series relying on the dtc v1.4.2 tag:
> > > > https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg01815.html
> > > > but the dtc mirror is outdated and that tag is not there.
> > > Last time, Alex Graf was taking care of the update request from me:
> > > 
> > > https://lists.gnu.org/archive/html/qemu-ppc/2015-05/msg00232.html
> > > 
> > > ... so maybe he could help here? If not, try to ask in #qemu IRC
> > > channel, I'm pretty sure that Stefan, Peter or somebody else can help
> > > here.
> > 
> > Yes, poke on IRC. Stefan used to handle the git sync, but IIUC he passed
> > that on. Once the sync is there, just send a patch that changes the
> > submodule reference.
> 
> Hi Guys,
> Jeff Cody is the QEMU systems administrator.  He can adjust git mirrors
> for qit.qemu-project.org.
> 
> Leon: Please give the git URL and branch that should be mirrored.  It
> cannot be a tag since that is immutable.  Instead it should be the
> dtc development/release tree that will be updated in the future.
>

Yes, once I get the git URL I'll update it on the server.  I presume it is
'master' on git://git.kernel.org/pub/scm/utils/dtc/dtc.git, but I will wait
for confirmation before I do anything.

Thanks,
Jeff

Re: [Qemu-devel] QEMU dtc submodule

[Paul Burton]

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

On Friday, 23 September 2016 09:13:51 BST Jeff Cody wrote:
> > Leon: Please give the git URL and branch that should be mirrored.  It
> > cannot be a tag since that is immutable.  Instead it should be the
> > dtc development/release tree that will be updated in the future.
> 
> Yes, once I get the git URL I'll update it on the server.  I presume it is
> 'master' on git://git.kernel.org/pub/scm/utils/dtc/dtc.git, but I will wait
> for confirmation before I do anything.
> 
> Thanks,
> Jeff

Hi Jeff,

That would be the right branch for DTC, but sadly the master branch doesn't contain the 
actual commit that was tagged as the v1.4.2 release. v1.4.2 tags this commit:

https://git.kernel.org/cgit/utils/dtc/dtc.git/commit/?
h=v1.4.2&id=ec02b34c05be04f249ffaaca4b666f5246877dea[1] 

Its parent commit is in the master branch, but it isn't. So simply mirroring the master 
branch wouldn't be enough, you'd need to include the v1.4.2 tag specifically. I've CC'd David 
Gibson who tagged DTC v1.4.2 in case he has input or can rectify this (which would 
probably be either a rebase & force push of the last 2 commits on the master branch atop 
v1.4.2, or a new release).

Thanks,
    Paul

--------
[1] https://git.kernel.org/cgit/utils/dtc/dtc.git/commit/?
h=v1.4.2&id=ec02b34c05be04f249ffaaca4b666f5246877dea

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [Qemu-devel] QEMU dtc submodule

[David Gibson]

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

On Fri, Sep 23, 2016 at 03:23:26PM +0100, Paul Burton wrote:
> On Friday, 23 September 2016 09:13:51 BST Jeff Cody wrote:
> > > Leon: Please give the git URL and branch that should be mirrored.  It
> > > cannot be a tag since that is immutable.  Instead it should be the
> > > dtc development/release tree that will be updated in the future.
> > 
> > Yes, once I get the git URL I'll update it on the server.  I presume it is
> > 'master' on git://git.kernel.org/pub/scm/utils/dtc/dtc.git, but I will wait
> > for confirmation before I do anything.
> > 
> > Thanks,
> > Jeff
> 
> Hi Jeff,
> 
> That would be the right branch for DTC, but sadly the master branch doesn't contain the 
> actual commit that was tagged as the v1.4.2 release. v1.4.2 tags this commit:
> 
> https://git.kernel.org/cgit/utils/dtc/dtc.git/commit/?
> h=v1.4.2&id=ec02b34c05be04f249ffaaca4b666f5246877dea[1] 
> 
> Its parent commit is in the master branch, but it isn't. So simply mirroring the master 
> branch wouldn't be enough, you'd need to include the v1.4.2 tag specifically. I've CC'd David 
> Gibson who tagged DTC v1.4.2 in case he has input or can rectify this (which would 
> probably be either a rebase & force push of the last 2 commits on the master branch atop 
> v1.4.2, or a new release).

Oops.. that's my mistake (with my upstream dtc maintainer hat on).

Not sure quite how I managed that, but yes, master branches off just
before the v1.4.2 release, instead of just after.  I've merged them
back together now, so that master should include v1.4.2 now.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [Qemu-devel] QEMU dtc submodule

[Jeff Cody]

On Sat, Sep 24, 2016 at 11:21:18AM +1000, David Gibson wrote:
> On Fri, Sep 23, 2016 at 03:23:26PM +0100, Paul Burton wrote:
> > On Friday, 23 September 2016 09:13:51 BST Jeff Cody wrote:
> > > > Leon: Please give the git URL and branch that should be mirrored.  It
> > > > cannot be a tag since that is immutable.  Instead it should be the
> > > > dtc development/release tree that will be updated in the future.
> > > 
> > > Yes, once I get the git URL I'll update it on the server.  I presume it is
> > > 'master' on git://git.kernel.org/pub/scm/utils/dtc/dtc.git, but I will wait
> > > for confirmation before I do anything.
> > > 
> > > Thanks,
> > > Jeff
> > 
> > Hi Jeff,
> > 
> > That would be the right branch for DTC, but sadly the master branch doesn't contain the 
> > actual commit that was tagged as the v1.4.2 release. v1.4.2 tags this commit:
> > 
> > https://git.kernel.org/cgit/utils/dtc/dtc.git/commit/?
> > h=v1.4.2&id=ec02b34c05be04f249ffaaca4b666f5246877dea[1] 
> > 
> > Its parent commit is in the master branch, but it isn't. So simply mirroring the master 
> > branch wouldn't be enough, you'd need to include the v1.4.2 tag specifically. I've CC'd David 
> > Gibson who tagged DTC v1.4.2 in case he has input or can rectify this (which would 
> > probably be either a rebase & force push of the last 2 commits on the master branch atop 
> > v1.4.2, or a new release).
> 
> Oops.. that's my mistake (with my upstream dtc maintainer hat on).
> 
> Not sure quite how I managed that, but yes, master branches off just
> before the v1.4.2 release, instead of just after.  I've merged them
> back together now, so that master should include v1.4.2 now.
> 

Thanks,

I've updated the git server, and submitted a patch to move the QEMU dtc
submodule version to 1.4.2.

Thanks,
Jeff