[Mesa-dev] [ANNOUNCE] mesa 21.3.0-rc1

2021-10-13 Thread Eric Engestrom
Hello everyone!

Once again, a new release cycle has started. Please test this first
release candidate, and report any issue here:
https://gitlab.freedesktop.org/mesa/mesa/-/issues/new

Issues that should block the release of 21.3.0 should be added to the
corresponding milestone:
https://gitlab.freedesktop.org/mesa/mesa/-/milestones/27

Cheers,
  Eric

---

git tag: mesa-21.3.0-rc1

https://mesa.freedesktop.org/archive/mesa-21.3.0-rc1.tar.xz
SHA256: f1ec269b7db6568347e554fdbe42e09e1632ac5f6a48913e0fb3def654749d95  
mesa-21.3.0-rc1.tar.xz
SHA512: 
346e120e5ac00fbb8ed507765d84d2c54b739595dde24a8c1a9cd19b705ea3e9c3171fbd457e4ace3089eebcf7d293f845568f7b52228ef958e9ef959013
  mesa-21.3.0-rc1.tar.xz
PGP:  https://mesa.freedesktop.org/archive/mesa-21.3.0-rc1.tar.xz.sig



signature.asc
Description: PGP signature


Re: [Mesa-dev] Workflow Proposal

2021-10-13 Thread Jordan Justen
Alyssa Rosenzweig  writes:

>>  Upstream should do what's best for upstream, not for Intel's "unique"
>>  management.
>> 
>>Not sure how from Emma explaining how Rb tags were used by Intel
>>management it came the conclusion that it were used in that way only by
>>Intel management. Spoiler: it is not.
>
> Sorry, I'll make that point more emphatic.
>
> Upstream must do what's best for upstream without zero regard for the
> whims of management. Doubly so for bad management.

If the r-b process ever had any notice from any company's management, I
haven't seen it. (Actually, I think most management would rather have
the short sighted view of skipping code review to more quickly merge
patches.) In terms of who to "track down", that is also a tenuous
connection.

The value of r-b is to give reviewers credit for the hard work that they
do. (Which, I believe is what Matt and apinheiro are also saying.)

Personally I try to make a rework log on patch commit messages to give
reviewers more explicit credit for changes that are made based on their
code review.

I hope Marge Bot doesn't start stripping the r-b tags. But, if Marge can
add Approved-by which allows the review process to flow more quickly for
some types of merge-requests, then I think that's a good thing. I
wouldn't be surprised if this became the most commonly used review
process in Mesa.

-Jordan


Re: [Mesa-dev] Workflow Proposal

2021-10-13 Thread Alyssa Rosenzweig
> > >> I would love to see this be the process across Mesa.  We already don't
> > >> rewrite commit messages for freedreno and i915g, and I only have to do
> > >> the rebase (busy-)work for my projects in other areas of the tree.
> > > Likewise for Panfrost. At least, I don't do the rewriting. Some Panfrost
> > > devs do, which I'm fine with. But it's not a requirement to merging.
> > >
> > > The arguments about "who can help support this years from now?" are moot
> > > at our scale... the team is small enough that the name on the reviewer
> > > is likely the code owner / maintainer, and patches regularly go in
> > > unreviewed for lack of review bandwidth.
> >
> > There is another reason to the Rb tag, that is to measure the quantity
> > of patch review people do.
> >
> > This was well summarized some years ago by Matt Turner, as it was
> > minimized (even suggested to be removed) on a different thread:
> >
> > https://lists.freedesktop.org/archives/mesa-dev/2019-January/213586.html
> 
> I was part of the Intel team when people started doing this r-b
> counting.  I believe that it was being done due to Intel management's
> failure to understand who was doing the work on the team and credit
> them appropriately, and also to encourage those doing less to step up.
> Unfortunately, the problem with Intel management wasn't a lack of
> available information, and I didn't see publishing the counts change
> reviews either.



Upstream should do what's best for upstream, not for Intel's "unique"
management.


Re: [Mesa-dev] Workflow Proposal

2021-10-13 Thread Alyssa Rosenzweig
>  Upstream should do what's best for upstream, not for Intel's "unique"
>  management.
> 
>Not sure how from Emma explaining how Rb tags were used by Intel
>management it came the conclusion that it were used in that way only by
>Intel management. Spoiler: it is not.

Sorry, I'll make that point more emphatic.

Upstream must do what's best for upstream without zero regard for the
whims of management. Doubly so for bad management.