2011-03-30 03:27:11

by Peter Pan(潘卫平)

Subject: [PATCH] delete a warning in function xfs_buf_readahead

/home/pwp/mysvn/linux/linux-2.6/fs/xfs/linux-2.6/xfs_buf.c: In function
‘xfs_buf_readahead’:
/home/pwp/mysvn/linux/linux-2.6/fs/xfs/linux-2.6/xfs_buf.c:660:27: warning:
unused variable ‘bdi’
Signed-off-by: Pan Weiping <[email protected]>
---
fs/xfs/linux-2.6/xfs_buf.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/fs/xfs/linux-2.6/xfs_buf.c b/fs/xfs/linux-2.6/xfs_buf.c
index 596bb2c..d917146 100644
--- a/fs/xfs/linux-2.6/xfs_buf.c
+++ b/fs/xfs/linux-2.6/xfs_buf.c
@@ -657,8 +657,6 @@ xfs_buf_readahead(
xfs_off_t ioff,
size_t isize)
{
- struct backing_dev_info *bdi;
-
if (bdi_read_congested(target->bt_bdi))
return;

--
1.7.4

2011-03-30 03:47:36

by Alex Elder

Subject: Re: [PATCH] delete a warning in function xfs_buf_readahead

On Wed, 2011-03-30 at 11:28 +0800, Pan Weiping wrote:
> /home/pwp/mysvn/linux/linux-2.6/fs/xfs/linux-2.6/xfs_buf.c: In function
> ‘xfs_buf_readahead’:
> /home/pwp/mysvn/linux/linux-2.6/fs/xfs/linux-2.6/xfs_buf.c:660:27: warning:
> unused variable ‘bdi’
> Signed-off-by: Pan Weiping <[email protected]>
> ---
> fs/xfs/linux-2.6/xfs_buf.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)

Thanks for sending this. I already have a fix
in place. I have not yet sent it to Linus yet
but will soon.

-Alex