avg-I commented on this pull request.


>       ARCSTAT_INCR(arcstat_l2_size, write_sz);
-       ARCSTAT_INCR(arcstat_l2_asize, write_asize);
-       vdev_space_update(dev->l2ad_vdev, write_asize, 0, 0);
+       ARCSTAT_INCR(arcstat_l2_asize, write_psize);

Actually, I must withdraw my previous comment. There still will be a 
distinction between psize and asize.. And thus far we seem to account 
arcstat_l2_asize, the vdev space and l2ad_alloc refcount interms of pszie. So, 
I am going to follow your advice and rename arcstat_l2_asize to 
arcstat_l2_psize. Maybe later on we will change how the accounting is done.  
E.g. I think that the vdev space should be accounted in terms of asize.  But 
that's an issue for another day (I hope).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/310#discussion_r121226686
------------------------------------------
openzfs-developer
Archives: 
https://openzfs.topicbox.com/groups/developer/discussions/Td414270217188b5f-Mc5341f5e427dbf75a73fdc69
Powered by Topicbox: https://topicbox.com

Reply via email to