Re: [GIT PULL] tomoyo update for v6.12
From: Casey Schaufler
Date: Tue Oct 08 2024 - 14:36:04 EST
On 10/8/2024 4:14 AM, Dr. Greg wrote:
> ...
>
> Which we also believe justifies more attention than what it has been
> able to receive in 20 months.
You're right. You're also not alone. There are things that you can do
that will help get the review you're looking for. Developers who attend
to the needs and preferences of reviewers get a whole lot more attention
than those who fuss and fume about not getting what they "deserve". My
hopefully constructive recommendations are:
1. Lead with code. Save the documentation for later.
2. Incremental implementation. Don't drop the whole mess on the
reviewers at once. A patch set should be a story, with each patch
introducing one new element.
3. Emphasize the similarities with existing implementations. No one
wants to deal with novel or clever code. If it is familiar, it is
easy to understand.
4. Thank your reviewers. Complaints about review latency typically
increase it.
5. Do some reviews yourself. That will get in the good graces of other
reviewers.
6. Be brief. The biggest single problem with reviewing TSEM has been that
doing anything takes so long. Multiple paragraph responses to an issue
don't help. Say it, say it once, say it in small words, and use as
few of those as possible.