--- /dev/null 2008-03-30 12:15:48.586669308 +0200 +++ linux-2.6.24logfs/fs/logfs/compr.c 2008-03-25 16:56:40.399236016 +0100 @@ -0,0 +1,95 @@ +/* + * fs/logfs/compr.c - compression routines + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + */ +#include "logfs.h" +#include <linux/vmalloc.h> +#include <linux/zlib.h> + +#define COMPR_LEVEL 3 + +static DEFINE_MUTEX(compr_mutex); +static struct z...

--- /dev/null 2008-03-30 12:15:48.586669308 +0200 +++ linux-2.6.24logfs/fs/logfs/dev_bdev.c 2008-03-28 20:04:13.961033452 +0100 @@ -0,0 +1,146 @@ +/* + * fs/logfs/dev_bdev.c - Device access methods for block devices + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + */ +#include "logfs.h" +#include <linux/buffer_head.h> + +#define PAGE_OFS(ofs) ((ofs) & (PAGE_SIZE-1)) + +static int bdev_read(stru...

--- /dev/null 2008-03-30 12:15:48.586669308 +0200 +++ linux-2.6.24logfs/fs/logfs/segment.c 2008-03-28 18:40:52.161518316 +0100 @@ -0,0 +1,595 @@ +/* + * fs/logfs/segment.c - Handling the Object Store + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + * + * Object store or ostore makes up the complete device with exception of + * the superblock and journal areas. Apart from its own metadata it stores + * thre...

--- /dev/null 2008-04-02 16:29:12.813336657 +0200 +++ linux-2.6.24logfs/fs/logfs/super.c 2008-04-01 22:58:50.362878974 +0200 @@ -0,0 +1,374 @@ +/* + * fs/logfs/super.c + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + * + * Generally contains mount/umount code and also serves as a dump area for + * any functions that don't fit elsewhere and neither justify a file of their + * own. + */ +#include "logfs.h" ...

--- /dev/null 2008-03-30 12:15:48.586669308 +0200 +++ linux-2.6.24logfs/fs/logfs/inode.c 2008-04-01 19:42:55.146055510 +0200 @@ -0,0 +1,575 @@ +/* + * fs/logfs/inode.c - inode handling code + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + */ +#include "logfs.h" +#include <linux/writeback.h> +#include <linux/backing-dev.h> + +/* + * We need to include <linux/writeback.h> - a header we no...

--- /dev/null 2008-03-30 12:15:48.586669308 +0200 +++ linux-2.6.24logfs/fs/logfs/file.c 2008-03-28 18:26:27.421722776 +0100 @@ -0,0 +1,236 @@ +/* + * fs/logfs/file.c - prepare_write, commit_write and friends + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + */ +#include "logfs.h" +#include <linux/sched.h> +#include <linux/writeback.h> + +static int logfs_prepare_write(struct file *file, struct ...

--- /dev/null 2008-04-02 16:29:12.813336657 +0200 +++ linux-2.6.24logfs/fs/logfs/logfs.h 2008-04-01 21:00:59.600247674 +0200 @@ -0,0 +1,555 @@ +/* + * fs/logfs/logfs.h + * + * As should be obvious for Linux kernel code, license is GPLv2 + * + * Copyright (c) 2005-2007 Joern Engel <joern@logfs.org> + * + * Private header for logfs. + */ +#ifndef fs_logfs_logfs_h +#define fs_logfs_logfs_h + +#define __CHECK_ENDIAN__ + +#include <linux/crc32.h> +#include <linux/fs.h> +#incl...

--- linux-2.6.24logfs/fs/Kconfig~logfs 2008-03-24 12:39:10.190939342 +0100 +++ linux-2.6.24logfs/fs/Kconfig 2008-04-01 19:48:18.517351063 +0200 @@ -1385,6 +1385,23 @@ config JFFS2_CMODE_FAVOURLZO endchoice +config LOGFS + bool "Log Filesystem (EXPERIMENTAL)" + depends on (MTD || BLOCK) && EXPERIMENTAL + select ZLIB_INFLATE + select ZLIB_DEFLATE + select CRC32 + help + Flash filesystem aimed to scale efficiently to large devices. + In comparison to JFFS2 it offers significantly...

Hi Ralf, A few exported symbols in MIPS-specific code can be unexported, and this series of patches does so. This series was tested using a custom config that was produced by running `make allmodconfig', after which the CONFIG_VORTEX option was manually turned off. The patches I am submitting do not touch the code affected by this option, however. Setting this options on leads to unbuildable kernel independently of whether the patches are applied. Additionally, the entire patchset was tested ...

No users for the rtc_mips_set_time() routine exist outside of the core kernel code. Therefore, EXPORT_SYMBOL(rtc_mips_set_time) is useless, and this patch removes it. Signed-off-by: Dmitri Vorobiev <dmitri.vorobiev@gmail.com> --- arch/mips/kernel/time.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/mips/kernel/time.c b/arch/mips/kernel/time.c index b45a709..d70ce5c 100644 --- a/arch/mips/kernel/time.c +++ b/arch/mips/kernel/time.c @@ -38,7 +38,6 @@ int __...

The copy_to_user_page() function is called only in the core kernel code. Therefore, there is no need to export it. This patch removes EXPORT_SYMBOL(copy_to_user_page). Signed-off-by: Dmitri Vorobiev <dmitri.vorobiev@gmail.com> --- arch/mips/mm/init.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/arch/mips/mm/init.c b/arch/mips/mm/init.c index 71e5962..6b9d54b 100644 --- a/arch/mips/mm/init.c +++ b/arch/mips/mm/init.c @@ -247,8 +247,6 @@ void copy_to_user_page...

I found a few ways to cause pages and pages of spew to dmesg of the following form.. rhythmbox: page allocation failure. order:3, mode:0x4020 Pid: 4299, comm: rhythmbox Not tainted 2.6.25-0.172.rc7.git4.fc9.x86_64 #1 Call Trace: <IRQ> [<ffffffff810862dc>] __alloc_pages+0x3a3/0x3c3 [<ffffffff812a58df>] ? trace_hardirqs_on_thunk+0x35/0x3a [<ffffffff8109fd94>] alloc_pages_current+0x100/0x109 [<ffffffff810a6fd5>] new_slab+0x4a/0x249 [<ffffffff810a776a>] __s...

So the problem is that SH3 is allowed to enable HUGETLB_PAGE/HUGETLBFS, but only SH4/5 define the HPAGE_SHIFT macro, due to some #ifdeffery on the hugepage size. So either the choice needs to be extended to include CPU_SH3 (which builds here, when changed) or the HUGETLBFS conditional needs to depend more specifically on SH4/5 and not just SUPERH. I think the arch maintainer has to make that call, as I don't know the hardware to say if SH3 actually supports multiple hugepage sizes. Thanks, Nish...

Use cpuset_scnprintf to print cpus on a leaf instead of requiring a new "cpumask_scnprintf_len" function to determine the size of the temporary buffer. cpuset_scnprintf can be used to print directly to the output buffer, eliminating the need for the temporary buffer. The format that cpuset_scnprintf uses is dependent on the sysctl variable kernel.compat_cpuset_printf. The default is '1' and results in this printout when NR_CPUS = 4096: # cat /sys/devices/system/cpu/cpu3/cache/index0/shared_...

* Add a new cpuset_scnprintf and a sysctl flag to control how cpumask sets are printed. The default is to use the current cpumask_scnprintf. If kernel.compat_cpuset_printf is '0' (default 1), then cpulist_scnprintf is used. In addition, a nodeset_scnprintf is provided for compatibilty. This is introduced with a CONFIG_KERN_COMPAT_CPUSET_PRINTF flag which currently is only defined for X86_64_SMP architecture. For all other architectures the current cpumask_scnprintf() is used...

Found this in my FC6-based bittorrent box (K7-800 running a 2.6.25-rc6-git2 kernel) this evening. The kernel was upgraded two weeks ago to fix the bug in which an USB VIA driver hammered the PCI bus causing ATA disk performance to drop, and has been running since then (it still is). So it's actually 2.6.25-rc6-git2 plus patch as in here http://www.gossamer-threads.com/lists/linux/kernel/895506#895506 If there's anything useful I can do, just ask. Thanks ! [1164423.381894] WARNING: at ne...

but, but, the 'k' means kernel. Is that so bad? --- ~Randy --

From: Leonardo Potenza <lpotenza@inwind.it> Removed the warning messages: drivers/atm/iphase.c:961: warning: 'tcnter' defined but not used drivers/atm/iphase.c:963: warning: 'xdump' defined but not used tcnter and xdump() are used only in debug build Signed-off-by: Leonardo Potenza <lpotenza@inwind.it> --- iphase.c | 2 ++ 1 file changed, 2 insertions(+) --- linux-2.6.orig/drivers/atm/iphase.c +++ linux-2.6/drivers/atm/iphase.c @@ -958,6 +958,7 @@ static void ia_suni_pm734...

the problem: doing malloc() in userspace on no-mmu systems can take up an inordinate amount of time merely due to kernel forcing anonymously mapped memory to zero. this is because the only way to implement userspace malloc() on no-mmu systems is via mmap(MAP_ANONYMOUS), and the kernel has an explicit memset() in mm/nommu.c:do_mmap_private() when MAP_ANONYMOUS is in the mmap flags. however, POSIX states that memory returned from malloc() may be uninitialized, so this zeroing is not wanted in that ca...

[Empty message]

Hi, This appeared in my logs: kmemcheck: Caught 32-bit read from freed memory (f7042348) Pid: 1374, comm: bash Not tainted (2.6.25-rc7 #92) EIP: 0060:[<c0502f0d>] EFLAGS: 00210202 CPU: 0 EIP is at call_for_each_cic+0x2d/0x44 EAX: 00200286 EBX: 00000001 ECX: c200e908 EDX: f7042348 ESI: f6c26c60 EDI: c0503310 EBP: f70fff38 ESP: c082ec88 DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0068 CR0: 8005003b CR2: f7826904 CR3: 36cd7000 CR4: 000006c0 DR0: 00000000 DR1: 00000000 DR2: 00000000 ...

Why the heck is cic_free_func() immediately doing a kmem_cache_free() on the cfq_io_context structure??? Shouldn't we have a call_rcu() or a synchronize_rcu() in there somewhere??? Given the way this is written, wouldn't readers on other code paths get dumped onto the freelist? This would not be a good thing... Thanx, Paul --

As per recommendations made at LSF'08, a stackable file system which does not change the data across layers, should try to use vm_operations instead of address_space_operations. This means we now have to implement out own ->read and ->write methods because we cannot rely on VFS helpers which require us to have a ->readpage method. Either way there are two caveats: (1) It's not possible currently to set inode->i_mapping->a_ops to NULL, because too many code paths expect i_mapping to ...

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu> --- fs/unionfs/super.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/unionfs/super.c b/fs/unionfs/super.c index e5cb235..4cddc83 100644 --- a/fs/unionfs/super.c +++ b/fs/unionfs/super.c @@ -755,7 +755,7 @@ out_no_change: /* grab new lower super references; release old ones */ for (i = 0; i < new_branches; i++) atomic_inc(&new_data[i].sb->s_active); - for (i = 0; i < new_branches; i++) + for ...

Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu> --- fs/unionfs/commonfops.c | 32 ++++++++++----------------- fs/unionfs/dirfops.c | 18 ++++++++++----- fs/unionfs/file.c | 55 +++++++++++++++++++++++++++++------------------ fs/unionfs/union.h | 1 - 4 files changed, 58 insertions(+), 48 deletions(-) diff --git a/fs/unionfs/commonfops.c b/fs/unionfs/commonfops.c index 2add167..50f4eda 100644 --- a/fs/unionfs/commonfops.c +++ b/fs/unionfs/commonfops.c @@ -302,7 +302,7...