<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>On 01/04/2020 14:09, Christian Mauderer wrote:<br>
</p>
<blockquote type="cite"
cite="mid:01613f71-99fe-de6b-ac0f-3638f9a41dbf@embedded-brains.de">
<pre class="moz-quote-pre" wrap="">On 01/04/2020 11:55, Sebastian Huber wrote:
</pre>
<blockquote type="cite" style="color: #000000;">
<pre class="moz-quote-pre" wrap="">On 01/04/2020 11:45, Christian Mauderer wrote:
</pre>
<blockquote type="cite" style="color: #000000;">
<pre class="moz-quote-pre" wrap="">The rtems_bsd_mmcsd_attach_worker acquired the bus without releasing it.
If a MMC device has multiple hardware partitions (like eMMC which
typically has at least one boot partition) the
rtems_bsd_mmcsd_attach_worker would try to acquire the bus multiple
times. This doesn't work.
Doing it right would mean to acquire and release the bus for each
access which would have an performance impact. Beneath that it would
mean that partition switching has to be supported by the RTEMS code too.
There is currently no known use case where the access would be
necessary. Therefore this patch prefers the performance and just avoids
all further hardware partitions.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">Looks good.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">Should we apply it only to master or to 5-freebsd-12 too? The problem
affects both so I would suggest to apply it to both branches.</pre>
</blockquote>
Both branches please.<br>
</body>
</html>