Remove unused variable olm1
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 229 Build 229: arc lint + arc unit
Event Timeline
Just because the change needs to be submitted upstream, do not give up.
Matt Ahrens can help with changes that need to be pushed upstream to Illumos. @mahrens , can you help with this?
You can use the instructions here: https://github.com/openzfs/openzfs to submit a pull request and get regression tests. Then solicit code reviews to developer@lists.illumos.org.
@araujo : the instructions that @mahrens gave are pretty straightforward. I looked at: https://github.com/openzfs/openzfs/pulls
but did not see you open a pull request. Do you plan to do this?
This patch doesn't apply for Illumos, as their merge.c seems more updated than the one we have.
From Illumos:
292 /*
293 * Don't do the recursive equivalency checking more than
294 * we have to.
295 */
296 if (olm1 == NULL || olm1->ml_type->t_id != ml1->ml_type->t_id) {
297 if (ml1->ml_size != ml2->ml_size ||
298 !equiv_node(ml1->ml_type, ml2->ml_type, ed))
299 return (0);
300 }
I do believe soom we will import these changes from vendor.