Btrfs devel - Oct 2008 - Problem with building btrfs into the kernel instead of module

Hi

I have compiled btrfs into the kernel instead of building as module.
Then I create a btrfs filesystem on a partition and try to mount it and
I get the following error.

# mkfs.btrfs /dev/sda1
fs created label (null) on /dev/sda1
        nodesize 4096 leafsize 4096 sectorsize 4096 size 4.88GB
Btrfs v0.16-8-g09ab036

# mount -t btrfs /dev/sda1 /mnt/sda1/
mount: unknown filesystem type ''btrfs''

If I perform the above steps with btrfs built as a module I don''t get
any error.

Is this a known issue or am I doing something wrong?

-Arun
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

On Wed, 8 Oct 2008, Arun Bhanu wrote:
> Hi
> 
> I have compiled btrfs into the kernel instead of building as module.
> Then I create a btrfs filesystem on a partition and try to mount it and
> I get the following error.
fs/btrfs/Makefile is a bit wonky.  This at least makes it work both ways:


diff --git a/fs/btrfs/Makefile b/fs/btrfs/Makefile
index 48b7909..7125716 100644
--- a/fs/btrfs/Makefile
+++ b/fs/btrfs/Makefile
@@ -1,7 +1,7 @@
 ifneq ($(KERNELRELEASE),)
 # kbuild part of makefile
 
-obj-m  := btrfs.o
+obj-$(CONFIG_BTRFS_FS) := btrfs.o
 btrfs-y := super.o ctree.o extent-tree.o print-tree.o root-tree.o 
dir-item.o \
           file-item.o inode-item.o inode-map.o disk-io.o \
           transaction.o inode.o file.o tree-defrag.o \
@@ -14,7 +14,7 @@ else
 
 KERNELDIR := /lib/modules/`uname -r`/build
 all:
-       $(MAKE) -C $(KERNELDIR) M=`pwd` modules
+       $(MAKE) -C $(KERNELDIR) M=`pwd` CONFIG_BTRFS_FS=m modules
 
 modules_install:
        $(MAKE) -C $(KERNELDIR) M=`pwd` modules_install

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

* Sage Weil (sage@newdream.net) wrote:
> On Wed, 8 Oct 2008, Arun Bhanu wrote:
> > Hi
> > 
> > I have compiled btrfs into the kernel instead of building as module.
> > Then I create a btrfs filesystem on a partition and try to mount it
and
> > I get the following error.
> 
> fs/btrfs/Makefile is a bit wonky.  This at least makes it work both ways:
Thanks. With your patch applied, it is working as expected in both
cases.
> diff --git a/fs/btrfs/Makefile b/fs/btrfs/Makefile
> index 48b7909..7125716 100644
> --- a/fs/btrfs/Makefile
> +++ b/fs/btrfs/Makefile
> @@ -1,7 +1,7 @@
>  ifneq ($(KERNELRELEASE),)
>  # kbuild part of makefile
>  
> -obj-m  := btrfs.o
> +obj-$(CONFIG_BTRFS_FS) := btrfs.o
>  btrfs-y := super.o ctree.o extent-tree.o print-tree.o root-tree.o 
> dir-item.o \
>            file-item.o inode-item.o inode-map.o disk-io.o \
>            transaction.o inode.o file.o tree-defrag.o \
> @@ -14,7 +14,7 @@ else
>  
>  KERNELDIR := /lib/modules/`uname -r`/build
>  all:
> -       $(MAKE) -C $(KERNELDIR) M=`pwd` modules
> +       $(MAKE) -C $(KERNELDIR) M=`pwd` CONFIG_BTRFS_FS=m modules
>  
>  modules_install:
>         $(MAKE) -C $(KERNELDIR) M=`pwd` modules_install
> 
-Arun
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html