Re: [PATCH?] make __bio_add_page check q->max_hw_sectors

[Adam J. Richter]

On Sun, 10 Oct 2004 10:14:16 +0200, Jens Axboe wrote:
>On Sun, Oct 10 2004, Adam J. Richter wrote:
[...]
>> 	I do not understand the intended difference between
>> the new max_hw_sectors field and max_sectors, so it is unclear
>> to me if it is a bug that my dm-crypt request_queue has
>> q->max_hw_sectors < q->max_sectors.  If q->max_hw_sectors
>> is supposed to be guaranteed to be greater than or equal
>> to q->max_sectors, then the real bug is elsewhere and my
>> patch is unnecessary.

>That's exactly correct, ->max_sectors must never be bigger than
>max_hw_sectors, that is the real bug.

	OK.  Please disregard my previous patch.  Thanks for
your clarification.

	The problem I saw was due to my mistake in merging the
2.6.9-rc3 change that added request_queue->max_sectors with one
of my local changes (replacing some fields in struct request_queue
with struct io_restrictions, a patch which I should repost one of
these days).

                    __     ______________
Adam J. Richter        \ /
adam@xxxxxxxxxxxxx      | g g d r a s i l
-
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/