[prev in list] [next in list] [prev in thread] [next in thread] 

List:       illumos-developer
Subject:    [developer] [REVIEW] loader disk interface and large (2+TB) disk related patch
From:       "Toomas Soome" <tsoome () me ! com>
Date:       2016-11-20 23:17:08
Message-ID: A3F78ACC-47AF-4683-8B7F-2023547E8EE7 () me ! com
[Download RAW message or body]

hi!

Please review 2 issues:

https://www.illumos.org/rb/r/271/ <https://www.illumos.org/rb/r/271/>
https://www.illumos.org/rb/r/272/ <https://www.illumos.org/rb/r/272/>


I did copy&paste the descriptions from fbsd review posts, but few words ahead: first \
one is implementing an instrument to fetch disk media related information from disk \
abstraction layer (block device -> disk -> partition), to have simple mechanism to \
read partition and disk data. This interface is proved to be useful in different \
cases, one of is presented as second link here.

Second issue is direct consumer for first and is about large disk support - \
apparently the current workaround still has holes (there is an case with 8TB disks \
and SATA interface which is reporting disks with size  0xFFFFFFFF). So the better \
solution was needed. The same system does behave ok with this fix.

Aye, both updates are also just posted for review in fbsd, and not yet commited.

thanks,
toomas


-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/25758058-4e9228dc
Modify Your Subscription: \
https://www.listbox.com/member/?member_id=25758058&id_secret=25758058-c19b436a \
Powered by Listbox: http://www.listbox.com


[Attachment #3 (unknown)]

<html><html><html><head><meta content="text/html charset=us-ascii" \
http-equiv="Content-Type" /></head><body class="" style="word-wrap: break-word; \
-webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">hi!<div \
class=""><br class="" /></div><div class="">Please review 2 issues:</div><div \
class=""><br class="" /></div><div class=""><a class="" \
href="https://www.illumos.org/rb/r/271/">https://www.illumos.org/rb/r/271/</a></div><div \
class=""><a class="" \
href="https://www.illumos.org/rb/r/272/">https://www.illumos.org/rb/r/272/</a></div><div \
class=""><br class="" /></div><div class=""><br class="" /></div><div class="">I did \
copy&amp;paste the descriptions from fbsd review posts, but few words ahead: first \
one is implementing an instrument to fetch disk media related information from disk \
abstraction layer (block device -&gt; disk -&gt; partition), to have simple mechanism \
to read partition and disk data. This interface is proved to be useful in different \
cases, one of is presented as second link here.</div><div class=""><br class="" \
/></div><div class="">Second issue is direct consumer for first and is about large \
disk support - apparently the current workaround still has holes (there is an case \
with 8TB disks and SATA interface which is reporting disks with size \
&nbsp;0xFFFFFFFF). So the better solution was needed. The same system does behave ok \
with this fix.</div><div class=""><br class="" /></div><div class="">Aye, both \
updates are also just posted for review in fbsd, and not yet commited.</div><div \
class=""><br class="" /></div><div class="">thanks,</div><div \
class="">toomas</div><div bgcolor="#ffffff" id="listbox-footer" \
style="width:auto;margin:0;padding:5px;background-color:#fff;clear:both;border-top: \
1px solid #ccc;"><table bgcolor="#ffffff" border="0" cellpadding="0" cellspacing="0" \
style="background-color:#fff" width="100%"><tr><td padding="4px"><font \
color="#333333" size="1" style="font-family:helvetica, sans-serif;">  \
<strong>illumos-developer</strong> | <a \
href="https://www.listbox.com/member/archive/182179/=now" \
style="text-decoration:none;color:#669933;border-bottom: 1px solid #444444" title="Go \
to archives for illumos-developer">Archives</a> <a border="0" \
href="https://www.listbox.com/member/archive/rss/182179/25758058-4e9228dc" \
style="text-decoration:none;color:#669933" title="RSS feed for \
illumos-developer"><img border="0" \
src="http://postlink.www.listbox.com/2252120/833487e62783d55fe81f119fb93ef644/25758058 \
/bb3fe179.jpg?uri=aHR0cHM6Ly93d3cubGlzdGJveC5jb20vaW1hZ2VzL2ZlZWQtaWNvbi0xMHgxMC5qcGc" \
/></a>  | <a href="https://www.listbox.com/member/?member_id=25758058&id_secret=25758058-c19b436a" \
style="text-decoration:none;color:#669933;border-bottom: 1px solid #444444" \
title="">Modify</a>  Your Subscription<td align="right" valign="top"><a \
href="http://www.listbox.com" style="border-bottom:none;"> <img border="0" \
src="http://postlink.www.listbox.com/2252121/3379085af0f1cf7fc3708f04b4471ae2/25758058 \
/bb3fe179.png?uri=aHR0cHM6Ly93d3cubGlzdGJveC5jb20vaW1hZ2VzL2xpc3Rib3gtbG9nby1zbWFsbC5wbmc" \
title="Powered by Listbox" \
/></a></td></font></td></tr></table></div></body></html></html></html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic