Re: YAFFS in the kernel tree?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 28, 2008 at 4:59 PM, Charles Manning
<manningc2@xxxxxxxxxxxxx> wrote:
> Hi
>
> I'm the author of YAFFS. This is not in the kernel tree, but is fairly easy to
> integrate by just pulling a tarball and running  patch-in script.
>
> I am curious as to whether people consider the current mechanism "good enough"
> or whether it is worth the effort trying to get YAFFS into the kernel tree.
>
> Pros I can see:
> * In tree means better testing (maybe).
> * Keeping current with kernel API changes.

 * exposes your code to folks who otherwise might not be
aware it even exists.

>
> Cons:
> * More effort for YAFFS maintainers (me mostly).

It may look like this is the case initially, but really it becomes a case of
less effort for maintainers (you), once you get over the initial blip/hump of
knocking things into shape and getting it merged.  If you are out of tree,
you have to be aware of all the API changes and subtle dependencies that
may lurk from a side effect of some other changeset.  If you are in-tree and
Al Viro discovers some wide-sweeping bug, he will most likely fix your stuff
for you before you were even aware there was a problem....   :-)

> * Effort getting code into kernel coding style (unless I can get a waiver on
> this).

No get out of jail free card here, for anyone.  Sorry.

Paul.


>
> Thoughts??
>
> -- CHarles
> --
> To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Gstreamer Embedded]     [Linux MMC Devel]     [U-Boot V2]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux ARM Kernel]     [Linux OMAP]     [Linux SCSI]

  Powered by Linux