linux.git/block/blk-map.c, branch v2.6.35

block: Use accessor functions for queue limits

2009-05-22T21:22:54+00:00

Convert all external users of queue limits to using wrapper functions
instead of poking the request queue variables directly.

Signed-off-by: Martin K. Petersen 
Signed-off-by: Jens Axboe

block: Un-export blk_rq_append_bio

2009-05-19T10:14:56+00:00

OSD was the last in-tree user of blk_rq_append_bio(). Now
that it is fixed blk_rq_append_bio is un-exported and
is only used internally by block layer.

Signed-off-by: Boaz Harrosh 
Signed-off-by: Jens Axboe

block: allow blk_rq_map_kern to append to requests

2009-05-19T10:14:55+00:00

Use blk_rq_append_bio() internally instead of blk_rq_bio_prep()
so blk_rq_map_kern can be called multiple times, to map multiple
buffers.

This is in the effort to un-export blk_rq_append_bio()

Signed-off-by: James Bottomley 
Signed-off-by: Boaz Harrosh 
Signed-off-by: Jens Axboe

block: hide request sector and data_len

2009-05-11T07:50:55+00:00

Block low level drivers for some reason have been pretty good at
abusing block layer API.  Especially struct request's fields tend to
get violated in all possible ways.  Make it clear that low level
drivers MUST NOT access or manipulate rq->sector and rq->data_len
directly by prefixing them with double underscores.

This change is also necessary to break build of out-of-tree codes
which assume the previous block API where internal fields can be
manipulated and rq->data_len carries residual count on completion.

[ Impact: hide internal fields, block API change ]

Signed-off-by: Tejun Heo 
Signed-off-by: Jens Axboe

block: kill rq->data

2009-04-28T05:37:36+00:00

Now that all block request data transfer is done via bio, rq->data
isn't used.  Kill it.

While at it, make the roles of rq->special and buffer clear.

[ Impact: drop now unncessary field from struct request ]

Signed-off-by: Tejun Heo 
Cc: Boaz Harrosh

[SCSI] block: make blk_rq_map_user take a NULL user-space buffer for WRITE

2009-01-02T17:10:35+00:00

The commit 818827669d85b84241696ffef2de485db46b0b5e (block: make
blk_rq_map_user take a NULL user-space buffer) extended
blk_rq_map_user to accept a NULL user-space buffer with a READ
command. It was necessary to convert sg to use the block layer mapping
API.

This patch extends blk_rq_map_user again for a WRITE command. It is
necessary to convert st and osst drivers to use the block layer
apping API.

Signed-off-by: FUJITA Tomonori 
Acked-by: Jens Axboe 
Signed-off-by: James Bottomley

[SCSI] block: fix the partial mappings with struct rq_map_data

2009-01-02T17:10:08+00:00

This fixes bio_copy_user_iov to properly handle the partial mappings
with struct rq_map_data (which only sg uses for now but st and osst
will shortly). It adds the offset member to struct rq_map_data and
changes blk_rq_map_user to update it so that bio_copy_user_iov can add
an appropriate page frame via bio_add_pc_page().

Signed-off-by: FUJITA Tomonori 
Acked-by: Jens Axboe 
Signed-off-by: James Bottomley

When block layer fails to map iov, it calls bio_unmap_user to undo

2008-12-03T11:41:20+00:00

mapping.  Which is good if pages were mapped - but if they were provided
by someone else and just copied then bad things happen - pages are
released once here, and once by caller, leading to user triggerable BUG
at include/linux/mm.h:246.

Signed-off-by: Petr Vandrovec 
Signed-off-by: Jens Axboe

block: hold extra reference to bio in blk_rq_map_user_iov()

2008-11-18T14:08:56+00:00

If the size passed in is OK but we end up mapping too many segments,
we call the unmap path directly like from IO completion. But from IO
completion we have an extra reference to the bio, so this error case
goes OOPS when it attempts to free and already free bio.

Fix it by getting an extra reference to the bio before calling the
unmap failure case.

Reported-by: Petr Vandrovec 

Signed-off-by: Jens Axboe

block: make blk_rq_map_user take a NULL user-space buffer

2008-10-09T06:56:11+00:00

This patch changes blk_rq_map_user to accept a NULL user-space buffer
with a READ command if rq_map_data is not NULL. Thus a caller can pass
page frames to lk_rq_map_user to just set up a request and bios with
page frames propely. bio_uncopy_user (called via blk_rq_unmap_user)
doesn't copy data to user space with such request.

Signed-off-by: FUJITA Tomonori 
Signed-off-by: Jens Axboe