All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>,
	Marek Vasut <marek.vasut@gmail.com>,
	Otavio Salvador <otavio.salvador@ossystems.com.br>,
	OpenEmbedded Core Mailing List
	<openembedded-core@lists.openembedded.org>
Subject: Re: [U-Boot] [PATCH] u-boot: Upgrade to 2017.09
Date: Mon, 18 Sep 2017 12:06:10 -0400	[thread overview]
Message-ID: <20170918160610.GD17137@bill-the-cat> (raw)
In-Reply-To: <CAJTo0LbFHX_Wm_W2MteS=6XbJgqzZh-jtGZ_Bhu95kO8Nue+dw@mail.gmail.com>

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

On Mon, Sep 18, 2017 at 04:51:31PM +0100, Burton, Ross wrote:
> On 18 September 2017 at 16:46, Otavio Salvador <
> otavio.salvador@ossystems.com.br> wrote:
> 
> > What is the policy on doing u-boot version upgrades this late in the
> >
> > release cycle? SHouldn't this wait until after the release?
> >
> >
> > Why?
> >
> > It is just another recipe and we are upgrading to the final release.
> >
> > As Martin said, it was already broken.
> >
> > I'll take the responsibility to fix it. But as other packages, upgrades
> > has risk and we have more than enough time to proper fix it.
> >
> 
> Why?  Because it was merged to master after the freeze.
> 
> Personally I'm of the opinion that u-boot is one of those special recipes
> that if an upgrade appears just after freeze, we consider it.  If we don't
> keep it up to date BSPs won't use the recipe, and we'll be back to every
> BSP layer containing its own special copy of the u-boot recipe.

Also please note that U-Boot has a rather regular and public release
schedule: http://www.denx.de/wiki/U-Boot/ReleaseCycle so you can have a
good idea beforehand if you want to grab something for a release or not.

Personally, I would like to see this in.  But the security issue that's
been disclosed now is "resolved" just by not enabling functionality that
no one was enabling in mainline, and will be removed in the next
release, so don't feel you need to pull it in on those grounds.

-- 
Tom

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [OE-core]  [PATCH] u-boot: Upgrade to 2017.09
Date: Mon, 18 Sep 2017 12:06:10 -0400	[thread overview]
Message-ID: <20170918160610.GD17137@bill-the-cat> (raw)
In-Reply-To: <CAJTo0LbFHX_Wm_W2MteS=6XbJgqzZh-jtGZ_Bhu95kO8Nue+dw@mail.gmail.com>

On Mon, Sep 18, 2017 at 04:51:31PM +0100, Burton, Ross wrote:
> On 18 September 2017 at 16:46, Otavio Salvador <
> otavio.salvador at ossystems.com.br> wrote:
> 
> > What is the policy on doing u-boot version upgrades this late in the
> >
> > release cycle? SHouldn't this wait until after the release?
> >
> >
> > Why?
> >
> > It is just another recipe and we are upgrading to the final release.
> >
> > As Martin said, it was already broken.
> >
> > I'll take the responsibility to fix it. But as other packages, upgrades
> > has risk and we have more than enough time to proper fix it.
> >
> 
> Why?  Because it was merged to master after the freeze.
> 
> Personally I'm of the opinion that u-boot is one of those special recipes
> that if an upgrade appears just after freeze, we consider it.  If we don't
> keep it up to date BSPs won't use the recipe, and we'll be back to every
> BSP layer containing its own special copy of the u-boot recipe.

Also please note that U-Boot has a rather regular and public release
schedule: http://www.denx.de/wiki/U-Boot/ReleaseCycle so you can have a
good idea beforehand if you want to grab something for a release or not.

Personally, I would like to see this in.  But the security issue that's
been disclosed now is "resolved" just by not enabling functionality that
no one was enabling in mainline, and will be removed in the next
release, so don't feel you need to pull it in on those grounds.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170918/e77d5853/attachment.sig>

  reply	other threads:[~2017-09-18 16:06 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-11 20:30 [PATCH] u-boot: Upgrade to 2017.09 Otavio Salvador
2017-09-11 20:30 ` [U-Boot] " Otavio Salvador
2017-09-13 20:05 ` Marek Vasut
2017-09-13 20:11   ` Otavio Salvador
2017-09-13 20:11     ` Otavio Salvador
2017-09-13 20:20     ` Marek Vasut
2017-09-13 21:02       ` Otavio Salvador
2017-09-13 21:02         ` Otavio Salvador
2017-09-13 21:23         ` Marek Vasut
2017-09-17  9:18 ` Martin Jansa
2017-09-17  9:18   ` [U-Boot] [OE-core] " Martin Jansa
2017-09-17 11:06   ` Martin Jansa
2017-09-17 11:06     ` [U-Boot] [OE-core] " Martin Jansa
2017-09-18  5:32     ` Marek Vasut
2017-09-18  7:54       ` [U-Boot] " Martin Jansa
2017-09-18  7:54         ` [U-Boot] [OE-core] " Martin Jansa
2017-09-18  8:31         ` Marek Vasut
2017-09-18  8:39           ` [U-Boot] " Martin Jansa
2017-09-18  8:39             ` [U-Boot] [OE-core] " Martin Jansa
2017-09-18 15:20             ` [U-Boot] " Philip Balister
2017-09-18 15:20               ` [U-Boot] [OE-core] " Philip Balister
2017-09-18 15:46               ` [U-Boot] " Otavio Salvador
2017-09-18 15:46                 ` [U-Boot] [OE-core] " Otavio Salvador
2017-09-18 15:51                 ` [U-Boot] " Burton, Ross
2017-09-18 15:51                   ` [U-Boot] [OE-core] " Burton, Ross
2017-09-18 16:06                   ` Tom Rini [this message]
2017-09-18 16:06                     ` Tom Rini
2017-09-19  8:15                     ` [U-Boot] " Marek Vasut
2017-09-19  8:15                       ` [U-Boot] [OE-core] " Marek Vasut
2017-09-21 21:35                       ` [U-Boot] " Philip Balister
2017-09-21 21:35                         ` [U-Boot] [OE-core] " Philip Balister
2017-09-22 22:19                         ` [U-Boot] " Marek Vasut
2017-09-22 22:19                           ` [U-Boot] [OE-core] " Marek Vasut
2017-09-22 22:22                           ` [U-Boot] " Moritz Fischer
2017-09-22 22:22                             ` [U-Boot] [OE-core] " Moritz Fischer
2017-09-22 23:36                           ` [U-Boot] " Martin Jansa
2017-09-22 23:36                             ` [U-Boot] [OE-core] " Martin Jansa
2017-09-22 23:44                             ` [U-Boot] " Marek Vasut
2017-09-22 23:44                               ` [U-Boot] [OE-core] " Marek Vasut
2017-09-17 15:35   ` Khem Raj
2017-09-17 15:35     ` [U-Boot] [OE-core] " Khem Raj
2017-09-18  8:39     ` Łukasz Majewski
2017-09-19  2:11     ` Randy MacLeod
2017-09-19  2:11       ` [U-Boot] [OE-core] " Randy MacLeod

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170918160610.GD17137@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=marek.vasut@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio.salvador@ossystems.com.br \
    --cc=ross.burton@intel.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.