On Wed, Dec 09, 2015 at 12:47:55AM +0200, Or Gerlitz wrote: > On Wed, Dec 9, 2015 at 12:04 AM, Doug Ledford <[email protected]> wrote: > > Makes sense. > > thanks. > > > Show me what you are talking about (either a link to Ira's > > patch you are referring to or your own patch). > > The patch is three liner to add the cached attrs -- > http://marc.info/?l=linux-rdma&m=142309296813985&w=2 -- if you are OK > with that, I will add a 2nd patch that ports all ULPs to use the > cached copy instead of their code which does the query. > > Actually, why not start with this approach and later decide if we need > to go further of this is enough?
Or, can we please stop this bikeshedding. Christoph's patch is done, well tested and does a lot more clean up that this hacky three liner. It is a good patch, and although patchworks doesn't have my remarks from an earlier revision I still think it should go forward. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
