[Differential] [Updated] D3121: Don't allow malloc() to wait for resource while holding a lock in ITS

Zbigniew Bodek zbb at semihalf.com
Mon Jul 20 18:38:15 UTC 2015


That is why I was asking whether you don't want to see any messages at all.
Before Phabricator we used to send patch to the mailing list for review.
Now the e-mail is sent automatically by Phabricator but unfortunately
all other stuff related to that diff is also being sent.
If Phabricator cannot limit its messages to the "announcement" that
there is a patch for review we can basically get back to sending
"heads up" on the mailing list "hey there is a patch for review here:
<link to phabricator>" without adding mailing list to the subscribers
on Phabric.
(Or separate mailing list if that is what you prefer).

I always add mailing list to review since one can get much better
feedback if someone who is interested in that patch is not added to
reviewers on Phabricator.

BTW. Actually on Code Review Wiki
(https://wiki.freebsd.org/CodeReview) we can see an example of adding
mailing list to the subscribers, so you may want to remove that if it
is not "kosher".

Best regards
zbb

2015-07-20 18:33 GMT+02:00 Ian Lepore <ian at freebsd.org>:
> On Mon, 2015-07-20 at 09:08 -0700, John-Mark Gurney wrote:
>> Zbigniew Bodek wrote this message on Mon, Jul 20, 2015 at 12:27 +0200:
>> > Just to clarify. You don't want to receive any e-mails from
>> > Phabricator on freebsd-arm@ or just those redundant like "(...)
>> > created a dependency"?
>>
>> Mailing lists should never received subscriber added messages..
>> They are a complete waste of traffic...
>>
>> IMO, the only updates the mailing list should receive are created
>> and diff uploaded...
>>
>> Some could argue that the comments should be too, but IMO, if you care
>> about the review, you'll subscribe yourself, and the review has all
>> the comments tied together...
>>
>> Anything else should be not set... and this applies to ALL mailing
>> lists...
>
> I'm not sure ANY review emails should get sent to this list.  But if
> they are, IMO, it should only be "new review created" emails, nothing
> more, ever, even if someone specifically adds the list as a subscriber.
> A single "announcement" per review (and likewise for bugs in bugzilla,
> which I think we've got right now) allows interested parties to
> subscribe themselves.
>
> -- Ian
>
>


More information about the freebsd-arm mailing list