https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-2667 Lustre <p>This ticket is for tracking the cleanup of the NRS structures from the lustre_net.h header into a separate header lustre_nrs.h. There are quite a number of NRS-specific structures that are not generally useful in the rest of the RPC code, and would be better in a separate header.</p> <p>It might also make sense to have separate policy-specific private headers instead of putting them into lustre_nrs.h. This avoids inter-connecting the policies unnecessarily, and also ensures that the headers are properly structured to allow use by future NRS policies more easily.</p> LU-2667

move NRS structures/definitions from lustre_net.h to new lustre_nrs.h header

Technical task LU-398 Minor Resolved Fixed WC Triage Andreas Dilger patch Wed, 23 Jan 2013 01:45:06 +0000 Tue, 9 Jun 2015 21:01:28 +0000 Tue, 9 Jun 2015 21:01:28 +0000 Lustre 2.4.0 Lustre 2.8.0 0 7 <p>Nikitas, it seems I'm not able to assign this ticket to you, but I filed it to track the cleanup discussed in the NRS base patch.</p> <p>Thanks Andreas, as discussed in the Gerrit change, I will address the requirements in this ticket once more urgent coding tasks are completed.</p> <p>Hi Nikitas, now that master is open again for landings, it would be great to see your patches to clean up the NRS code.</p> <p>Hi Andreas, i'd like to start writing and submitting some NRS patches again, but i have been assigned to a high-priority task in my employer's roadmap, which leaves we with little time to do much else atm unfortunately; depending on how things go, i'll do my best to put some work towards NRS (i need to at least start with addressing <a href="https://jira.whamcloud.com/browse/LU-3430" title="SWL failure: (hash.c:546:cfs_hash_bd_del_locked()) ASSERTION( bd-&gt;bd_bucket-&gt;hsb_count &gt; 0 ) failed:" class="issue-link" data-issue-key="LU-3430"><del>LU-3430</del></a> fairly soon). </p> <p>Nikitas, I appreciate that you are busy (as we all are). However, if there is technical debt in patches that is not being addressed promptly after the patch is landed, then it makes it much more difficult for us to accept patches in the future under similar circumstances.</p> <p>I prefer to be flexible when there are some minor issues with a patch, and fix the non-critical problems afterward, but we already got flak from other parties about accepting NRS policies after the feature freeze, so in the future we may be forced to deny patches from landing until all of the issues are addressed, if we aren't confident that the remaining issues will be addressed promptly. It seems that management doesn't understand that fixing technical debt is an important part of the development process, so blocking the whole feature from landing would keep their focus on fixing remaining issues in order to get the patch landed rather than dragging you off to work on something else. Hopefully you can convince your management that fixing these issues is not optional, and we aren't forced into an inflexible patch acceptance policy.</p> <p>I agree that that is a fair stance, and to the benefit of the project Andreas; thanks for putting that forward. We are looking at ways to address this work, I will come back soon with an update.</p> <p>Chris Horn (hornc@cray.com) uploaded a new patch: <a href="http://review.whamcloud.com/13966" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13966</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-2667" title="move NRS structures/definitions from lustre_net.h to new lustre_nrs.h header" class="issue-link" data-issue-key="LU-2667"><del>LU-2667</del></a> ptlrpc: Move NRS structures out of lustre_net.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 535e33b36784637dd1cb321f54dac53b147ea6ac</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13966/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13966/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-2667" title="move NRS structures/definitions from lustre_net.h to new lustre_nrs.h header" class="issue-link" data-issue-key="LU-2667"><del>LU-2667</del></a> ptlrpc: Move NRS structures out of lustre_net.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d867368cdfa1138909a64d13ab11ed3c0212a0b2</p> <p>Patch landed to master for 2.8.0.</p> Development Rank 1|hzvfw7: Rank (Obsolete) 6221