LinuxTV

News

2011-09-18   Linuxtv adds support for Patchwork at patchwork.linuxtv.org

patchwork.linuxtv.org will be used to store all patches sent to the linux-media@vger.kernel.org mailing list.

As several of you noticed, we were having several troubles with patchwork.kernel.org, resulting on patches that got lost and several other troubles. Still, it is the main tool I use here to collect random patches that people send through the mailing list and that the driver maintainer is bbusy enough to not care picking and sending it to me via a git pull request.

At the end of August, kernel.org infrastructure were put down for maintainance, and there's no ETA for it to return. So, I had to set an alternate way for handling patches. So, starting from now, all new patches sent to the linux-media ML will now be sent also to patchwork.linuxtv.org where they'll be stored if patchwork euristics detect the email as a patch.

However, with the patchwork.kernel.org servers down, I could not recover the patches that were there. Due to that, I've sent to patchwork filter the emails from my linux-media archives (that has a few holes, but it is better than nothing).

About 7500+ patches were added there, since 2009. As most of the patches were already applied, I've scripted some logic to detect if a patch were (partially) applied, or if the patch were rejected. I'm now reviewing manually the patches that still applies and pinging people about them, if the patch is too old and I didn't notice it being applied. This gives us a second chance of reviewing the patches that older versions of Patchwork (or kernel.org instance) didn't catch.

The patchwork instance at linuxtv is capable of automatically sending notifications to people if a patch changes its state. I'll likely enable this feature on a near future, after finishing the review of the old stuff. I'll also likely try to use the delegate function of patchwork. I'll keep you updated about new features as we'll be able to activate them, via the mailing list.

For those that are submitting patches, I'd like to remember that patchwork has some rules to accept a patch. It basically will get your patch if:

    • the patch sent inlined;
      the patch weren't mangled by the emailer (some emailers love to break long lines into smaller ones, damaging the patches);
      are encoded with ASCII and doesn't contain any character upper than 0x80 (no accented characters).
  • Patchwork might also get patches uuencoded, with different charsets (in particular, if your name has accents, I recommend you to use UTF-8) and/or with the patch attached, but I recommend you to double check at patchwork.linuxtv.org to see if the patch were caught.

    Also, although patchwork does accept attached patches, you should not send patches on this way, as people can't easily review it (as several developers use plain text emailers that can't reply to the inlined patch text).

    If you discover any problems there, or notice a patch that were badly tagged (and aren't marked there as "New"), please drop me an email.

    Ah, you should notice that, by default, patchwork will only show patches tagged as New or Under review (e. g., patches with "Action required"). So, if you want to search for a patch that might be wrongly tagged, you'll need to click into patchwork "Filters:" field and change it to show all patch status, or click on the URL bellow:

    All media patches at patchwork.linuxtv.org

    I hope you enjoy patchwork.linuxtv.org!

    Thanks,
    Mauro

    mchehab

    Privacy Policy