On Tue, 2013-11-19 at 15:42 -0800, Nicholas A. Bellinger wrote: > Hey Hannes! > > On Tue, 2013-11-19 at 09:07 +0100, Hannes Reinecke wrote: > > Hi Nic, > > > > here's the second version of my ALUA update & Referrals support patches. > > As per request I've split up the supported states into individual > > attributes, and while there I've also renamed the rather confusing > > 'alua_access_type' and 'alua_access_status' into 'alua_management_type' > > and 'alua_status_modification', respectively. > > > > The other features are: > > > > - Make supported states configurable: > > We should make the list of supported ALUA states configurable, > > as some setups would possibly like to support a small subset > > of ALUA states only. > > - Asynchronous transitioning: I've switched 'transitioning' > > handling to use a workqueue, that should allow us to simulate > > asynchronous transitioning modes. IE TCM should now be capable > > of handling requests while in transitioning, and properly terminate > > these with the correct sense code. > > - Include target device descriptor in VPD page 83 > > For the ALUA device handler we'd need to identify the target device > > where a given target port belongs to. So include the respective > > values in the VPD page. > > > > And, of course, referrals support. > > > > Ok, even though it is late, I've applied this series to for-next. > > So the one main concern here is that the user visible changes to > existing ALUA configfs attributes end up breaking the lio-utils code > related to ALUA that depend upon them, which will need to be resolved > separately. > > However, this breakage is pretty minor and having more sensible + > consistent attribute naming is worth the inconvenience. > OK, I'm having second thoughts about the user-visible changes to existing configfs attributes.. I'm fine with these types of changes, but need some more time to sort out how it's going to effect existing userspace dependencies. The one that immediately made do a double take is changing 'alua_access_state' to change input/output, but keep the same attribute name which makes backwards compatibility alot more tricky. So all that said, let's defer to v3.14 for this particular series, and sort out the user visable changes first, and make adjustments as necessary from there. WDYT..? --nab -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
