Re: [PATCH 00/11] removing the on-stack struct request

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Tue, Apr 22, 2008 at 2:26 AM, FUJITA Tomonori
<fujita.tomonori@xxxxxxxxxxxxx> wrote:
> This patchset converts some of the users of struct request on the
>  stack to use blk_get_request (as discussed in the large command
>  support thread [*1]).
>
>  IDE has the most users of the on-stack struct request. The on-stack
>  struct request has the benefit that it'll always work but I think that
>  using blk_get_request properly make the code clearer.
>
>  I've not got rid of all the on-stack struct request yet, but at least
>  this patchset removes the users of struct request that calls
>  blk_put_request. That is, we can remove the following hack in
>  blk_put_request:
>
>  /*
>   * Gee, IDE calls in w/ NULL q.  Fix IDE and remove the
>   * following if (q) test.
>   */
>  if (q) {
>         spin_lock_irqsave(q->queue_lock, flags);
>         __blk_put_request(q, req);
>         spin_unlock_irqrestore(q->queue_lock, flags);
>  }
>
>  This patchset is against Bartlomiej's ide tree. #1-9 patches are for
>  the IDE subsystem and #10-11 are for the block layer. #10-11 depends
>  on #1-9 so probably it would be easier to push all the patch via a
>  single tree.
>
>  [*1]
>  http://marc.info/?l=linux-scsi&m=120817161219068&w=2

Looks promising but probably I'll not be able to review it properly
this week (I'm busy with my real job) and the next week (I'll be busy
with pushing overdue IDE updates).  OTOH I'm hoping that Borislav
would be able to take a look in the meantime.

Thanks,
Bart
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux