From: Laurent Riffard <laurent.riffard@free.fr>
To: Alasdair G Kergon <agk@redhat.com>
Cc: Andrew Morton <akpm@osdl.org>,
linux-kernel@vger.kernel.org,
Felipe Alfaro Solana <felipe.alfaro@gmail.com>
Subject: Re: [PATCH] device-mapper: Fix target suspension oops.
Date: Tue, 12 Jul 2005 23:48:19 +0200 [thread overview]
Message-ID: <42D43AA3.9040000@free.fr> (raw)
In-Reply-To: <20050712212449.GK12337@agk.surrey.redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1852 bytes --]
Le 12.07.2005 23:24, Alasdair G Kergon wrote:
> This section got lost while refactoring the 'Fix deadlocks in core' set
> of patches I sent yesterday. Without it, you'll have problems
> activating any device-mapper devices.
>
> The NULL detection is moved inside the functions instead of every
> caller having to do it.
>
> --- drivers/md/dm-table.c 2005-07-12 22:10:20.000000000 +0100
> +++ /data/kernels/pm-2612udm1/source/drivers/md/dm-table.c 2005-07-06 18:52:18.000000000 +0100
> @@ -869,11 +869,17 @@
>
> void dm_table_presuspend_targets(struct dm_table *t)
> {
> + if (!t)
> + return;
> +
> return suspend_targets(t, 0);
> }
>
> void dm_table_postsuspend_targets(struct dm_table *t)
> {
> + if (!t)
> + return;
> +
> return suspend_targets(t, 1);
> }
>
Ok, it seems that it works well now.
[root@antares ~]# cat /proc/version
Linux version 2.6.13-rc2-mm2 (laurent@antares.localdomain) (gcc version 3.4.3 (Mandrakelinux 10.2 3.4.3-7mdk)) #104 Tue Jul 12 19:16:28 CEST 2005
[root@antares ~]# dmsetup table
vglinux1-lvhome: 0 7340032 linear 3:4 4604288
vglinux1-lvhome: 7340032 1048576 linear 3:4 14655872
vglinux1-lvtmp: 0 106496 linear 3:4 13238656
vglinux1-lvusr: 0 4194304 linear 3:4 409984
vglinux1-lvusr: 4194304 212992 linear 3:4 11944320
vglinux1-lvusr: 4407296 835584 linear 3:4 12403072
vglinux1-lvvar: 0 409600 linear 3:4 384
vglinux1-lvvar: 409600 245760 linear 3:4 12157312
crypt-swap2: 0 1590434 crypt aes-cbc-plain ad79bd09a98d3bf601bf15bb7cedd656ecd42fc1eb5c48822148de23ab724aad 0 3:71 0
crypt-swap1: 0 128457 crypt aes-cbc-plain 800d0c169e6e1b9dece5bfc645dfe08d61fd2d8a18f578c14501988cbf31590d 0 3:5 0
vglinux1-test: 0 1310720 linear 3:4 13345152
crypt-tmp: 0 106496 crypt aes-cbc-plain d3de9d10862e89107374542c0cedb569c16d637262f5537f2c96bfc20eb2c1c1 0 254:3 0
Thank you for your quickness, Alasdair.
--
laurent
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
next prev parent reply other threads:[~2005-07-12 21:48 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-07-12 9:17 2.6.13-rc2-mm2 Andrew Morton
2005-07-12 11:05 ` 2.6.13-rc2-mm2 Adrian Bunk
2005-07-12 12:01 ` 2.6.13-rc2-mm2 David Woodhouse
2005-07-12 12:12 ` 2.6.13-rc2-mm2 Adrian Bunk
2005-07-12 18:16 ` 2.6.13-rc2-mm2 Felipe Alfaro Solana
2005-07-12 20:24 ` 2.6.13-rc2-mm2 Alasdair G Kergon
2005-07-13 9:29 ` 2.6.13-rc2-mm2 Felipe Alfaro Solana
2005-07-12 18:52 ` 2.6.13-rc2-mm2 : oops in dm_mod Laurent Riffard
2005-07-12 20:45 ` Mike Anderson
2005-07-12 20:47 ` Alasdair G Kergon
2005-07-12 21:02 ` Andrew Morton
2005-07-12 21:24 ` [PATCH] device-mapper: Fix target suspension oops Alasdair G Kergon
2005-07-12 21:48 ` Laurent Riffard [this message]
2005-07-14 3:04 ` 2.6.13-rc2-mm2 Matthias Urlichs
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=42D43AA3.9040000@free.fr \
--to=laurent.riffard@free.fr \
--cc=agk@redhat.com \
--cc=akpm@osdl.org \
--cc=felipe.alfaro@gmail.com \
--cc=linux-kernel@vger.kernel.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.