On 8/25/2017 3:37 PM, Konstantin Belousov wrote:
> On Fri, Aug 25, 2017 at 02:29:28PM -0700, Bryan Drewery wrote:
>> https://people.freebsd.org/~bdrewery/patches/vdrop-global-list.diff
> I do not object against his handling.
> 
> There is really very small amount of code shared between mp != NULL
> and mp == NULL cases, you may consider making them use separate paths
> instead of adding more than one check for mp != NULL.
> 

Sure, and I incorrectly was adding VMP_TMPMNTFREELIST flag for mp==NULL
case.  I'll open a review.

> VNASSERT() statement has wrong indent on the continuation line.
> 


-- 
Regards,
Bryan Drewery

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to