Re: [PATCH 5/7] block: Find bio sector offset given idx and offset

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: Jens Axboe

Subject: Re: [PATCH 5/7] block: Find bio sector offset given idx and offset

Date: Thursday, October 2, 2008 - 10:07 am

On Wed, Oct 01 2008, Martin K. Petersen wrote:
quoted text> >>>>> "Jens" == Jens Axboe <jens.axboe@oracle.com> writes:
> 
> Jens> Too may BUG's there, me thinks, and the interface looks
> Jens> fragile. What if the bio is alreday done (bi_idx == bi_vcnt)?
> 
> Good point!  Obviously I've only used the function for bio splitting
> and not at completion time.  Updated patch below.
> 
> 
> block: Find bio sector offset given idx and offset
> 
> Helper function to find the sector offset in a bio given bvec index
> and page offset.

Looks good, I've applied patches 1-5.

-- 
Jens Axboe

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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

Block integrity patches for 2.6.28, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 1/7] block: Introduce integrity data ownership flag, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 2/7] block: Fix double put in blk_integrity_unregister, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 3/7] block: Switch blk_integrity_compare from bdev ..., Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 4/7] block: gendisk integrity wrapper, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 5/7] block: Find bio sector offset given idx and offset, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 6/7] dm: Add support for data integrity to DM, Martin K. Petersen, (Wed Oct 1, 12:38 am)

[PATCH 7/7] md: Add support for data integrity to MD, Martin K. Petersen, (Wed Oct 1, 12:38 am)

Re: [PATCH 5/7] block: Find bio sector offset given idx an ..., Jens Axboe, (Wed Oct 1, 1:10 am)

Re: [PATCH 5/7] block: Find bio sector offset given idx an ..., Martin K. Petersen, (Wed Oct 1, 7:42 pm)

Re: Block integrity patches for 2.6.28, Jens Axboe, (Thu Oct 2, 3:56 am)

Re: Block integrity patches for 2.6.28, Martin K. Petersen, (Thu Oct 2, 6:54 am)

Re: Block integrity patches for 2.6.28, Jens Axboe, (Thu Oct 2, 9:29 am)

Re: [PATCH 5/7] block: Find bio sector offset given idx an ..., Jens Axboe, (Thu Oct 2, 10:07 am)


linux-kernel:
Greg KH	Og dreams of kernels
Jens Axboe	[PATCH 31/33] Fusion: sg chaining support
Arnd Bergmann	Re: finding your own dead "CONFIG_" variables
Mark Brown	[PATCH 2/2] Subject: natsemi: Allow users to disable workaround for DspCfg reset
Tony Breeds	[LGUEST] Look in object dir for .config
git:
Brian Downing	Re: Git in a Nutshell guide
John Benes	Re: master has some toys
Matthias Lederhofer	[PATCH 4/7] introduce GIT_WORK_TREE to specify the work tree
Alexander Sulfrian	[RFC/PATCH] RE: git calls SSH_ASKPASS even if DISPLAY is not set
Junio C Hamano	Re: Rss produced by git is not valid xml?
git-commits-head:
Linux Kernel Mailing List	iSeries: fix section mismatch in iseries_veth
Linux Kernel Mailing List	ixbge: remove TX lock and redo TX accounting.
Linux Kernel Mailing List	ixgbe: fix several counter register errata
Linux Kernel Mailing List	b43: fix build with CONFIG_SSB_PCIHOST=n
Linux Kernel Mailing List	9p: block-based virtio client
linux-netdev:
Michael Breuer	Re: [PATCH] af_packet: Don't use skb after dev_queue_xmit()
Michael Breuer	Re: [PATCH] af_packet: Don't use skb after dev_queue_xmit()
David Daney	[PATCH 5/7] Staging: Octeon Ethernet: Convert to NAPI.
Wolfgang Grandegger	[PATCH net-next v4 1/3] can: mscan: fix improper return if dlc < 8 in start_xmi...
Amit Kumar Salecha	[PATCHv3 NEXT 2/2] NET: Add Qlogic ethernet driver for CNA devices
openbsd-misc:
Theo de Raadt	Re: Old IPSEC bug
Tomáš Bodžár	Problem with vpnc connection - check group password !
Insan Praja SW	Mandoc Compiling Error
Carl Roberso	Re: Cannot change MTU of carp interface?
Richard Daemon	Re: booting openbsd on eee without cd-rom