On Tue, 2006-02-14 at 15:30 -0800, Daniel Phillips wrote:
> Badari Pulavarty wrote:
> > + for (i = 0; ; i++) {
> > + if (i == nblocks) {
> > + *map_valid = 0;
> > + break;
> > + } else if (page_block == blocks_per_page)
> > + break;
> > + blocks[page_block] = map_bh->b_blocknr + i;
> > + page_block++;
> > + block_in_file++;
> > + }
>
> Hi Badari, a tiny nit:
>
> for (i = 0; ; i++) {
> if (i == nblocks) {
> *map_valid = 0;
> break;
> }
> if (page_block == blocks_per_page)
> break;
> blocks[page_block] = map_bh->b_blocknr + i;
> page_block++;
> block_in_file++;
> }
>
Fixed. Wow !! People really do read others patches. Just kidding :)
Thanks,
Badari
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]