[parted-devel] [PATCH v2 0/2] libparted: enhanced s390 DASD probing

Viktor Mihajlovski mihajlov at linux.vnet.ibm.com
Fri Oct 23 12:24:25 UTC 2015


On 29.09.2015 23:00, Brian C. Lane wrote:
> On Thu, Sep 17, 2015 at 03:33:27PM +0200, Hendrik Brueckner wrote:
>> The current fdasd code present in parted prevents DASDs attached
>> non-natively, for example, through virtblk, from being recognized
>> as such and, thus, prevent proper partitioning.
>>
>> The following changes are implemented in this series:
>>
>> 1. Update parted with fdasd.c code from s390-tools to not rely on
>>     the DASD specific ioctl's anymore.
>>
>> 2. Adapt the parted code to be able to handle disks in DASD format
>>     irrespective of the used block device driver.
>>
>> 3. Add protection against messing up non-DASD block devices through
>>     better error checking.
>>
>>
>> Changes in the v2 series:
>>    - Use translatable strings and enhanced error handling.
>>    - Avoid API changes
>>
>>
>> Viktor Mihajlovski (2):
>>    fdasd: geometry handling updated from upstream s390-tools
>>    dasd: enhance device probing
>>
>>   include/parted/fdasd.in.h |    4 +-
>>   libparted/arch/linux.c    |   85 +++++++++++++++++++----
>>   libparted/device.c        |   14 +---
>>   libparted/labels/dasd.c   |   18 ++---
>>   libparted/labels/fdasd.c  |  166 +++++++++++++++++++++++++++++++--------------
>>   5 files changed, 201 insertions(+), 86 deletions(-)
>
> Thanks for the changes, these look good to me.
>
as there seem to be no further concerns (or relevance to non-s390 
usage), any chances to have this pushed any time soon?

-- 

Mit freundlichen Grüßen/Kind Regards
    Viktor Mihajlovski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Martina Köderitz
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294




More information about the parted-devel mailing list