Re: [PATCH 15/17] block: kill end_request()

From: Tejun Heo
Date: Fri Mar 20 2009 - 23:00:46 EST


Tejun Heo wrote:
> Impact: kill obsolete interface function
>
> end_request() has been kept around for backward compatibility;
> however, it seems to be about time for it to go away.
>
> * There aren't too many users left.
>
> * Its use of @updtodate is pretty confusing.
>
> * In some cases, newer code ends up using mixture of end_request() and
> [__]blk_end_request[_all](), which is way too confusing.
>
> So, kill it.
>
> Most conversions are straightforward. Noteworthy ones are...
>
> * paride/pcd: next_request() updated to take 0/-errno instead of 1/0.
>
> * paride/pf: pf_end_request() and next_request() updated to take
> 0/-errno instead of 1/0.
>
> * xd: xd_readwrite() updated to return 0/-errno instead of 1/0.
>
> * mtd/mtd_blkdevs: blktrans_discard_request() updated to return
> 0/-errno instead of 1/0. Unnecessary local variable res
> initialization removed from mtd_blktrans_thread().

This patch isn't correct. Will post updated version later.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/