On 9/30/18 2:46 PM, Phillip Potter wrote: > Ah nope, it was me being silly, I was only looking in /sys/module/md_mod/parameters. Still, > I am guessing this needs expanding upon as --detail still uses md_get_array_info which calls > ioctl? As I said, the ioctl is going away in favor of expanding sysctl. There will be no additions to the ioctl at this point, please work on improving sysfs. Thanks, Jes > Regards, > Phil Potter > > On Sun, Sep 30, 2018 at 07:07:18PM +0100, Phillip Potter wrote: >> Sorry, I incorrectly assumed it was ok as Detail.c uses md_get_array_info which still uses the >> ioctl at present. I'm certainly interested in having a pop at exposing this information via >> sysfs, if noone else is already working on it? The sysfs interface seems fairly sparse at >> present as far as reporting details on an array goes. >> >> Regards, >> Phil Potter >> On Sun, Sep 30, 2018 at 01:46:14PM -0400, Jes Sorensen wrote: >>> Sorry for top posting, unfortunately mobile clients are broken :( >>> >>> Please do not mess with the old ioctl interface. No new features should be added there. Use sysfs. >>> >>> Thanks >>> Jes >>> >>> Sent from MailDroid >>> >>> -----Original Message----- >>> From: Phillip Potter <phil@xxxxxxxxxxxxxxxx> >>> To: shli@xxxxxxxxxx, jes.sorensen@xxxxxxxxx >>> Cc: linux-raid@xxxxxxxxxxxxxxx >>> Sent: Sun, 30 Sep 2018 13:42 >>> Subject: [PATCH 0/2] Two patches for RAID-5/6 journal information >>> >>> Dear Jes and Shaohua, >>> >>> I've written the following two patches to expose RAID-5/6 journaling >>> information via the GET_ARRAY_INFO ioctl, as per the TODO in drivers/md/md.c. >>> The first patch is the kernel patch and the second one is for the mdadm >>> tree. I hope these are ok. Many thanks. >>> >>> Regards, >>> Phil Potter