From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adam Kwolek Subject: [PATCH] FIX: Add spare throws exception (v2) Date: Fri, 18 Mar 2011 10:08:04 +0100 Message-ID: <20110318090804.8068.28272.stgit@gklab-128-013.igk.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Return-path: Sender: linux-raid-owner@vger.kernel.org To: neilb@suse.de Cc: linux-raid@vger.kernel.org, dan.j.williams@intel.com, ed.ciechanowski@intel.com, wojciech.neubauer@intel.com List-Id: linux-raid.ids sync_metadata() requires st->sb to be loaded, otherwise exception is generated. This fails expansion, because spares cannot be added. metadata update uses tst instead st pointer, it is better than loading anchor for st as I proposed previously. Signed-off-by: Adam Kwolek --- Manage.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Manage.c b/Manage.c index 5808557..217c6b6 100644 --- a/Manage.c +++ b/Manage.c @@ -914,10 +914,10 @@ int Manage_subdevs(char *devname, int fd, close(container_fd); return 1; } - if (st->update_tail) - flush_metadata_updates(st); + if (tst->update_tail) + flush_metadata_updates(tst); else - tst->ss->sync_metadata(st); + tst->ss->sync_metadata(tst); sra = sysfs_read(container_fd, -1, 0); if (!sra) {