Re: UFS_GJOURNAL and ufs.ko



On Mon, Dec 18, 2006 at 03:39:06PM +0000, Coleman Kane wrote:
Hello,

I have noticed (since adding UFS journal support) that the
UFS_GJOURNAL option hasn't been added to the ufs.ko kernel
module. I've been using this (out of an effort to test as
much as possible as KLDs) and every time I upgrade my kernel
I must manually add -DUFS_GJOURNAL to the CFLAGS line in
its Makefile.

Is there any specific reason why this can't be committed?
So far, my experience with GJOURNAL has been great testing
it both with journal+fs on the same device as well as on
two seperate devices.

It looks like an oversight to me, but I don't want to jump
the gun on it if there is some reasonable argument to keep
it out "by default".

If I understand you right, you run a custom kernel w/o UFS and load
the latter as a module, don't you? If so, just add "options
UFS_GJOURNAL" to your kernel config file, and the module should
pick up the setting from the kernel build directory then, granted
you build modules with the kernel, not with the world.

IMHO CFLAGS should go away from ufs.ko's Makefile because the module
should get SOFTUPDATES and UFS_DIRHASH from the kernel configuration,
too.

--
Yar
_______________________________________________
freebsd-current@xxxxxxxxxxx mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@xxxxxxxxxxx"



Relevant Pages

  • Re: UFS_GJOURNAL and ufs.ko
    ... I have noticed (since adding UFS journal support) that the ... UFS_GJOURNAL option hasn't been added to the ufs.ko kernel ... two seperate devices. ... At this time more review is needed before it can be ...
    (freebsd-current)
  • Re: UFS_GJOURNAL and ufs.ko
    ... I have noticed (since adding UFS journal support) that the ... UFS_GJOURNAL option hasn't been added to the ufs.ko kernel ... two seperate devices. ... At this time more review is needed before it can be ...
    (freebsd-current)
  • UFS_GJOURNAL and ufs.ko
    ... I have noticed (since adding UFS journal support) that the ... UFS_GJOURNAL option hasn't been added to the ufs.ko kernel ... Is there any specific reason why this can't be committed? ... two seperate devices. ...
    (freebsd-current)
  • Re: UFS_GJOURNAL and ufs.ko
    ... UFS_GJOURNAL option hasn't been added to the ufs.ko kernel ... Is there any specific reason why this can't be committed? ... two seperate devices. ... UFS_GJOURNAL" to your kernel config file, ...
    (freebsd-current)
  • Re: UFS_GJOURNAL and ufs.ko
    ... UFS_GJOURNAL option hasn't been added to the ufs.ko kernel ... Is there any specific reason why this can't be committed? ... two seperate devices. ... UFS_GJOURNAL" to your kernel config file, ...
    (freebsd-current)