On Tue, Feb 14, 2006 at 01:54:11PM -0600, Dave Kleikamp wrote:
>...
> I wasn't able to trace it to an exact line of code, but I think I see
> what the problem is.
>
> This patch moves the copy_to_user from smb_read_data after the test
> whether smb_read_data is null. It's good not to dereference a pointer
> if you have a reason to test it for null afterward.
>
> This patch has not been compiled or tested.
>...
Thanks, this patch works fine for me and after disassembling it seems
you've found exactly the place where the kernel Oops'ed for me in the
forcedirectio case.
But even with this patch applied, I had a freeze in the
non-forcedirectio case, although much later than usual.
There seems to be more than one bug. :-(
I'm currently trying whether there are still any problems in the
forcedirectio case, and I'll report back as soon as I am able to provide
any additional information.
> David Kleikamp
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
