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-9185 Lustre <p>Current layout add replay is flawed, because layout add replies whole file LOVEA to client, and when replay, client sends the whole LOVEA back to server, server can't know which components were newly created by this layout add, and the replay would have to calls lod_sub_object_declare_create()/lod_sub_object_create() against all components, even for the pre-existing components.</p> <p>A proposal fix is to add one more 'buffer' parameter to dt_xattr_set(), and it's passed down to lod_xattr_set() -&gt; lod_striping_create(), in lod_striping_create(), the 'buffer' will be filled with the LOVEA for the newly created components, at the end, the 'buffer' will be copied to reply buffer and returned to client, so that on replay, client will send back the partial LOVEA which being newly created, and server knows which components needs be re-processed.</p> <p>In a word, layout add creates only partial layout, so server should return the partial layout to client but not the whole LOVEA.</p> LU-9185

flawed layout add replay

Technical task LU-8998 Major Resolved Fixed Zhenyu Xu Niu Yawei Mon, 6 Mar 2017 14:27:41 +0000 Mon, 27 Mar 2017 01:28:56 +0000 Mon, 27 Mar 2017 01:28:56 +0000 0 4 <p>Bobi Jam (bobijam@hotmail.com) uploaded a new patch: <a href="https://review.whamcloud.com/25824" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/25824</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-9185" title="flawed layout add replay" class="issue-link" data-issue-key="LU-9185"><del>LU-9185</del></a> pfl: fix replay of component add<br/> Project: fs/lustre-release<br/> Branch: pfl<br/> Current Patch Set: 1<br/> Commit: 35d06135c9b7a127668f4705c2857a0beeb8a52e</p> <p>Why couldn't it filter out the existing components at replay time?</p> <blockquote> <p>Why couldn't it filter out the existing components at replay time?</p></blockquote> <p>Because server can't tell which are pre-existing components, unless client tells more information.</p> <p>With dynamic layout introduced, layout add won't instantiate objects anymore (I think the semantic of layout add would be changed a little bit), so this problem can be fixed in a much easier way: remove all the code of instantiate objects, replay with same instantiated objects. Layout add now becomes a simple xattr set.</p> <p>I may miss something - but isn't newly added components deduced as (replay components)^(existing components)?</p> <blockquote> <p>I may miss something - but isn't newly added components deduced as (replay components)^(existing components)?</p></blockquote> <p>I think you are right, that fix requires much less code changes than my original proposal. Anyway, since new layout add won't instantiate objects, now we need only to remove all that replay handling code.</p> Development Rank 1|hzz5wv: Rank (Obsolete) 9223372036854775807