Re: [PATCH 03/13] block: Move queue limits to an embedded struct

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

 



On Fri, May 15 2009 at 12:40am -0400,
Martin K. Petersen <martin.petersen@xxxxxxxxxx> wrote:

> From: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
> 
> To accommodate stacking drivers that do not have an associated request
> queue we're moving the limits to a separate, embedded structure.

Given DM motivated this change, any reason you held off including the
associated DM changes (in a separate patch in the series)?  Your initial
DM patch looked reasonable.

On some level leaving that patch out of the series prevents reviewers
the opportunity to really see why a separate 'queue_limits' structure is
even needed.  The follow-up question being: is the topology
infrastructure adequately equipped to support DM?  I think so
but having others' review would be nice.

Are you looking for such DM changes to be submitted through Alasdair's
tree _after_ your changes go upstream through Jens' tree?

Regards,
Mike
--
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