On Wed, Oct 24 2007, Olivier Galibert wrote: > On Wed, Oct 24, 2007 at 11:12:42AM +0200, Jens Axboe wrote: > > sg_set_buf() also sets length and offset, sg_set_page() is just a mirror > > of that. So I'd prefer to keep the naming. > > Hmmm, sg_set_phys/sg_set_virt to be more symmetrical to > sg_phys/sg_virt? (please don't drop cc lists) That doesn't make any sense. Both sg_set_buf() and sg_set_page() set the same thing in the sg entry, the input is just different. It has nothing to do with setting the physical value, for instance. -- Jens Axboe - 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/
- Follow-Ups:
- Re: [PATCH 09/10] Change table chaining layout
- From: Olivier Galibert <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- References:
- Re: [PATCH 09/10] Change table chaining layout
- From: Boaz Harrosh <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Jens Axboe <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Boaz Harrosh <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Jens Axboe <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Boaz Harrosh <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Linus Torvalds <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Jens Axboe <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Geert Uytterhoeven <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Jens Axboe <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- From: Olivier Galibert <[email protected]>
- Re: [PATCH 09/10] Change table chaining layout
- Prev by Date: kernel processes - are they really needed?
- Next by Date: Re: [RFC PATCH 4/5] Remove CPU_DEAD/CPU_UP_CANCELLED handling from workqueue.c
- Previous by thread: Re: [PATCH 09/10] Change table chaining layout
- Next by thread: Re: [PATCH 09/10] Change table chaining layout
- Index(es):
 |