Ok to commit ubd patch ?

[Sven Luther]

Hello,

Since i got no reply to my list of debian patches i sent indirectly earlier, i
will send mail here for each of them, asking if it is ok to commit or not.

Here is the ubd patch :

#! /bin/sh -e
## ubd.dpatch by Matt Zimmerman <address@hidden>
##
## All lines beginning with `## DP:' are a description of the patch.
## DP: Recognize UML UBD devices.
## DP: Closes: #258188

. $(dirname $0)/DPATCH

@DPATCH@
diff -urNad /home/mdz/src/mrs/parted-1.6.9/include/parted/device.h 
parted-1.6.9/include/parted/device.h
--- /home/mdz/src/mrs/parted-1.6.9/include/parted/device.h      2004-07-07 
17:35:53.000000000 -0700
+++ parted-1.6.9/include/parted/device.h        2004-07-07 17:35:54.000000000 
-0700
@@ -34,7 +34,8 @@
        PED_DEVICE_FILE         = 5,
        PED_DEVICE_ATARAID      = 6,
        PED_DEVICE_I2O          = 7,
-       PED_DEVICE_DASD         = 8
+       PED_DEVICE_DASD         = 8,
+       PED_DEVICE_UBD          = 9
 } PedDeviceType;
 
 typedef struct _PedDevice PedDevice;
diff -urNad /home/mdz/src/mrs/parted-1.6.9/libparted/linux.c 
parted-1.6.9/libparted/linux.c
--- /home/mdz/src/mrs/parted-1.6.9/libparted/linux.c    2004-07-07 
17:35:53.000000000 -0700
+++ parted-1.6.9/libparted/linux.c      2004-07-07 17:36:15.000000000 -0700
@@ -217,6 +217,7 @@
 #define I2O_MAJOR7             86
 #define I2O_MAJOR8             87
 #define DASD_MAJOR             94
+#define UBD_MAJOR               98
 
 #define SCSI_BLK_MAJOR(M) (                                            \
                (M) == SCSI_DISK0_MAJOR                                 \
@@ -311,6 +312,8 @@
                dev->type = PED_DEVICE_CPQARRAY;
        } else if (dev_major == DASD_MAJOR && (dev_minor % 0x4 == 0)) {
                dev->type = PED_DEVICE_DASD;
+       } else if (dev_major == UBD_MAJOR && (dev_minor % 0x10 == 0)) {
+               dev->type = PED_DEVICE_UBD;
        } else {
                dev->type = PED_DEVICE_FILE;
        }
@@ -814,6 +817,12 @@
                        goto error_free_dev;
                break;
 
+       case PED_DEVICE_UBD:
+               if (!init_generic (dev, _("User-Mode Linux UBD")))
+                       goto error_free_dev;
+               break;
+
+
        case PED_DEVICE_FILE:
                if (!init_file (dev))
                        goto error_free_dev;

Seems straigthforward too, and doesn't impact on other code part of libparted.
Comments ? 

Friendly,

Sven Luther