LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Munroe <munroesj@linux.vnet.ibm.com>
To: "Carlos O'Donell" <carlos@redhat.com>
Cc: Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>,
	Adhemerval Zanella <adhemerval.zanella@linaro.org>,
	libc-alpha@sourceware.org, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] Add hwcap2 bits for POWER9
Date: Tue, 12 Jan 2016 10:39:17 -0600	[thread overview]
Message-ID: <1452616757.29488.10.camel@oc7878010663> (raw)
In-Reply-To: <56941532.9030800@redhat.com>

On Mon, 2016-01-11 at 15:48 -0500, Carlos O'Donell wrote:
> On 01/11/2016 02:55 PM, Tulio Magno Quites Machado Filho wrote:
> > "Carlos O'Donell" <carlos@redhat.com> writes:
> > 
> >> On 01/11/2016 10:16 AM, Tulio Magno Quites Machado Filho wrote:
> >>> Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:
> >>>
> >>>> On 08-01-2016 13:36, Peter Bergner wrote:
> >>>>> On Fri, 2016-01-08 at 11:25 -0200, Tulio Magno Quites Machado Filho wrote:
> >>>>>> Peter, this solves the issue you reported previously [1].
> >>>>>>
> >>>>>> [1] https://sourceware.org/ml/libc-alpha/2015-12/msg00522.html
> >>>>>
> >>>>> Agreed, thanks.  I'll also add the POWER9 support to the GCC side
> >>>>> of the patch now that the glibc code is upstream.
> >>>>
> >>>> I do not see these bits being added in kernel side yet and GLIBC usual
> >>>> only sync these kind of bits *after* they are included in kernel side.
> >>>> So I would advise to either get these pieces (kernel support and hwcap
> >>>> advertise) in kernel before 2.23 release, otherwise revert the patches.
> >>>
> >>> Ack.
> >>> It has just been sent to the correspondent Linux mailing list:
> >>> https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-January/137763.html
> >>
> >> Please revert the changes from glibc until you checkin support to linux
> >> kernel mainline.
> >>
> >> Leaving these bits in increases the risk that someone uses to deploy a glibc
> >> that then may have the wrong value.
> > 
> > Could you clarify this statement, please?
> > I fail to see how they could have the wrong value.
> 
> Until it is checked into the mainline kernel it is not canonical.
> 
> That's the rule. There are no other discussions to be had.
> 
Well is was posted to to powerpc next:
https://git.kernel.org/powerpc/c/e708c24cd01ce80b1609d8bacc

We have agreement between the kernel and GLIBC (and the ABI). 

The issue is just coordination across communities and individuals that
may not being paying attention to other communities dead lines.

Have you ever tried to push a string, up hill. That is open source
development in nutshell. ;)

So it is in flight and glibc is soft/slush freeze. I would hate to
revert this one day just to add it back to the next. Especially if those
days straddle the hard freeze ...

So can we let this ride a day or too?

> The single rule avoids discussions like "it can never be wrong because that's
> what our ABI says it is."
> 

  reply	other threads:[~2016-01-12 16:39 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <568C272D.6000705@linux.vnet.ibm.com>
     [not found] ` <87egdsi4om.fsf@totoro.br.ibm.com>
     [not found]   ` <1452267366.5201.12.camel@vnet.ibm.com>
     [not found]     ` <568FE3D0.7080008@linaro.org>
2016-01-11 15:16       ` [PATCH] Add hwcap2 bits for POWER9 Tulio Magno Quites Machado Filho
2016-01-11 15:21         ` Carlos O'Donell
2016-01-11 19:55           ` Tulio Magno Quites Machado Filho
2016-01-11 20:48             ` Carlos O'Donell
2016-01-12 16:39               ` Steven Munroe [this message]
2016-01-12 17:45                 ` Carlos O'Donell
2016-01-15 23:09                 ` Michael Ellerman
2016-01-15 22:30             ` Carlos Eduardo Seo

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=1452616757.29488.10.camel@oc7878010663 \
    --to=munroesj@linux.vnet.ibm.com \
    --cc=adhemerval.zanella@linaro.org \
    --cc=carlos@redhat.com \
    --cc=libc-alpha@sourceware.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=tuliom@linux.vnet.ibm.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox