Messages in this thread | | | Date | Wed, 20 Apr 2005 22:23:34 +0200 (CEST) | From | Jesper Juhl <> | Subject | Re: [PATCH] rename rw_verify_area() to rw_access_ok() |
| |
On Wed, 20 Apr 2005, Marcelo Tosatti wrote:
> On Sun, Apr 17, 2005 at 11:50:35PM +0200, Jesper Juhl wrote: > > verify_area() will soon be dead and gone, replaced by access_ok(), thus > > the function named rw_verify_area() is badly named and should be renamed. > > This patch renames rw_verify_area to rw_access_ok which seems more > > appropriate (it also updates all callers of the functions as well as > > references to it in comments). > > Not that I care too much, but, rw_verify_area() has nothing to do with > verify_area/access_ok functions. > right, access_ok deals with memory, rw_verify_area deals with files, but both serve a similar purpose - validating access to a region. That's why I thought it would make sense to have them named similarly (as they used to be).
> I dont see real need to rename this function. > Perhaps I went a tad too far, or perhaps I misunderstood the point of rw_verify_area(), that's certainly a possibility. In any case, it's no big deal, I just thought it was the logical thing to do - I'll leave it in Andrews capable hands to decide.
Thank you for commenting.
-- Jesper Juhl
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |