public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: agk@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 ./WHATS_NEW lib/cache/lvmcache.c Date: Tue, 17 Feb 2009 18:56:00 -0000 [thread overview] Message-ID: <20090217185641.8200.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2009-02-17 18:56:41 Modified files: . : WHATS_NEW lib/cache : lvmcache.c Log message: Index cached vgmetadata by vgid not vgname to cope with duplicate vgnames. (dwyso) Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1043&r2=1.1044 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/cache/lvmcache.c.diff?cvsroot=lvm2&r1=1.63&r2=1.64 --- LVM2/WHATS_NEW 2009/02/12 20:42:07 1.1043 +++ LVM2/WHATS_NEW 2009/02/17 18:56:41 1.1044 @@ -1,5 +1,6 @@ Version 2.02.45 - =================================== + Index cached vgmetadata by vgid not vgname to cope with duplicate vgnames. No longer require kernel and metadata major numbers to match. Add a fully-functional get_cluster_name() to clvmd corosync interface. Remove duplicate cpg_initialize from clvmd startup. --- LVM2/lib/cache/lvmcache.c 2008/11/28 15:51:40 1.63 +++ LVM2/lib/cache/lvmcache.c 2009/02/17 18:56:41 1.64 @@ -71,11 +71,20 @@ log_debug("Metadata cache: VG %s wiped.", vginfo->vgname); } -static void _store_metadata(struct lvmcache_vginfo *vginfo, - struct volume_group *vg, unsigned precommitted) +/* + * Cache VG metadata against the vginfo with matching vgid. + */ +static void _store_metadata(struct volume_group *vg, unsigned precommitted) { + char uuid[64] __attribute((aligned(8))); + struct lvmcache_vginfo *vginfo; int size; + if (!(vginfo = vginfo_from_vgid((const char *)&vg->id))) { + stack; + return; + } + if (vginfo->vgmetadata) _free_cached_vgmetadata(vginfo); @@ -86,8 +95,14 @@ vginfo->precommitted = precommitted; - log_debug("Metadata cache: VG %s stored (%d bytes%s).", vginfo->vgname, - size, precommitted ? ", precommitted" : ""); + if (!id_write_format((const struct id *)vginfo->vgid, uuid, sizeof(uuid))) { + stack; + return; + } + + log_debug("Metadata cache: VG %s (%s) stored (%d bytes%s).", + vginfo->vgname, uuid, size, + precommitted ? ", precommitted" : ""); } static void _update_cache_info_lock_state(struct lvmcache_info *info, @@ -1051,7 +1066,6 @@ { struct pv_list *pvl; struct lvmcache_info *info; - struct lvmcache_vginfo *vginfo; char pvid_s[ID_LEN + 1] __attribute((aligned(8))); pvid_s[sizeof(pvid_s) - 1] = '\0'; @@ -1067,9 +1081,8 @@ } /* store text representation of vg to cache */ - if (vg->cmd->current_settings.cache_vgmetadata && - (vginfo = vginfo_from_vgname(vg->name, NULL))) - _store_metadata(vginfo, vg, precommitted); + if (vg->cmd->current_settings.cache_vgmetadata) + _store_metadata(vg, precommitted); return 1; }
next reply other threads:[~2009-02-17 18:56 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-02-17 18:56 agk [this message] -- strict thread matches above, loose matches on Subject: below -- 2011-03-29 21:34 zkabelac 2011-03-10 3:03 agk 2010-12-01 10:39 zkabelac 2010-03-31 17:20 mbroz 2009-10-22 17:33 agk 2008-11-27 18:13 agk 2008-06-11 11:02 agk 2008-06-09 16:22 agk 2008-05-08 18:28 agk 2006-06-14 20:11 agk
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20090217185641.8200.qmail@sourceware.org \ --to=agk@sourceware.org \ --cc=lvm-devel@redhat.com \ --cc=lvm2-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).