Re: [PATCH v3 00/44] Meta Linux Kernel Port

[James Hogan]

Hi Vineet

On 28/01/13 07:10, Vineet Gupta wrote:
>> You don't need Acked-by statements on every single patch, but having
>> more of those is certainly benefitial. When it comes to the merge
>> window, please send a pull request to Linus, and keep me on Cc,
>> so I can weigh in with an additional Ack to the series.
> 
> While the first pull request can go directly from github, I presume the logistics
> for setting up accounts on kernel.org will only kick start after the first batch
> of code has been accepted. I can't seem to find any discussions on lists to that
> effect.

this may be relevant:

https://korg.wiki.kernel.org/index.php/Userdoc:getting_account

> BTW I went back to hexagon submission patches in 2011 and it seems every new arch
> makes the exact same mistakes:
> -idle sleep race
> -faltering in TIF_WORK_RESUME and in fixing that breaking the syscall restarting
> -redundant symbols in module.c
> ...
> 
> Will it make sense to have a checklist-for-new-arches with concrete examples of
> broken and fixed code so that you, Al and other reviewers don't have to repeat the
> same thing over and over again.

I suspect some of the problem is that the code will have been copied
from other architectures which were subsequently fixed, but the fixes
weren't being tracked while the out of tree port was being updated.

I don't think it's a bad idea to have a checklist like that, but I think
this is always going to continue to happen, and be dependent on when the
arch port was first written, so such a checklist may well need updating
lots based on what mistakes are most commonly made.

I'd suggest the following generic things:
* Work with upstream in the first place to minimise the period the arch
is out of tree (tends not to happen as much as we'd all like).
* Read linux-arch ML so that common per-arch problems are noticed as
soon as they're apparent, and cc it when those sorts of issues are raised.
* Look at comments for other/previous arch submissions (I definitely
made a fair few fixes/changes after looking at c6x, tile, and arc comments).
* Documenting these arch specific issues (like Al's signal brain dump
which was very valuable to understand the subtleties).

Cheers
James

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/