On Wed, Jul 21, 2010 at 10:53:59AM +0800, KAMEZAWA Hiroyuki wrote: > On Wed, 21 Jul 2010 12:54:37 +1000 > Nick Piggin <npiggin@xxxxxxx> wrote: > > > On Tue, Jul 20, 2010 at 03:29:48PM -0700, Andrew Morton wrote: > > > From: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> > > > > > > Now, rw_verify_area() checsk f_pos is negative or not. And if negative, > > > returns -EINVAL. > > > > > > But, some special files as /dev/(k)mem and /proc/<pid>/mem etc.. has > > > negative offsets. And we can't do any access via read/write to the > > > file(device). > > > > > > So introduce FMODE_NEG_OFFSET to allow negative file offsets. > > > > Minor nitpick but I don't understand why this is called NEG_OFFSET. It's > > a large positive offset into the file so FMODE_UNSIGNED_OFFSET seems > > like it would be better. > > > Ah, agreed. Agreed too. > I'll prepare a macro-renaming patch later (if this patch goes now.) > Thank you for review. Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
