| To: | Dave Chinner <david@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 4/8] xfs: clean up buffer get/read call API |
| From: | Mark Tinguely <tinguely@xxxxxxx> |
| Date: | Fri, 30 Mar 2012 14:12:10 -0500 |
| Cc: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1333023835-12856-5-git-send-email-david@xxxxxxxxxxxxx> |
| References: | <1333023835-12856-1-git-send-email-david@xxxxxxxxxxxxx> <1333023835-12856-5-git-send-email-david@xxxxxxxxxxxxx> |
| User-agent: | Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0 |
On 03/29/12 07:23, Dave Chinner wrote: From: Dave Chinner<dchinner@xxxxxxxxxx> The xfs_buf_get/read API is not consistent in the units it uses, and does not use appropriate or consistent units/types for the variables. Convert the API to use disk addresses and block counts for all buffer get and read calls. Use consistent naming for all the functions and their declarations, and convert the internal functions to use disk addresses and block counts to avoid need to convert them from one type to another and back again. Fix all the callers to use disk addresses and block counts. In many cases, this removes an additional conversion from the function call as the callers already have a block count. Signed-off-by: Dave Chinner<dchinner@xxxxxxxxxx> Reviewed-by: Christoph Hellwig<hch@xxxxxx> --- Looks good. Reviewed-by: Mark Tinguely <tinguely@xxxxxxx> |
| Previous by Date: | Re: [PATCH 5/7] xfs: honor the O_SYNC flag for aysnchronous direct I/O requests, Eric Sandeen |
|---|---|
| Next by Date: | Re: [PATCH 5/8] xfs: kill b_file_offset, Mark Tinguely |
| Previous by Thread: | [PATCH 4/8] xfs: clean up buffer get/read call API, Dave Chinner |
| Next by Thread: | [PATCH 8/8] xfs: kill xfs_buf_btoc, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |