Should be Cc: dm-devel@xxxxxxxxxx Maybe there should be a Cc: line in the top of each file?? Thanks, NeilBrown On Thursday May 3, akpm@xxxxxxxxxxxxxxxxxxxx wrote: > > The patch titled > Kcopyd: use mutex instead of semaphore > has been added to the -mm tree. Its filename is > kcopyd-use-mutex-instead-of-semaphore.patch > > *** Remember to use Documentation/SubmitChecklist when testing your code *** > > See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find > out what to do about this > > ------------------------------------------------------ > Subject: Kcopyd: use mutex instead of semaphore > From: Matthias Kaehlcke <matthias.kaehlcke@xxxxxxxxx> > > Kcopyd uses a semaphore as mutex. Use the mutex API instead of the (binary) > semaphore, > > Signed-off-by: Matthias Kaehlcke <matthias.kaehlcke@xxxxxxxxx> > Cc: Neil Brown <neilb@xxxxxxx> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > --- > > drivers/md/kcopyd.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff -puN drivers/md/kcopyd.c~kcopyd-use-mutex-instead-of-semaphore drivers/md/kcopyd.c > --- a/drivers/md/kcopyd.c~kcopyd-use-mutex-instead-of-semaphore > +++ a/drivers/md/kcopyd.c > @@ -198,7 +198,7 @@ struct kcopyd_job { > * These fields are only used if the job has been split > * into more manageable parts. > */ > - struct semaphore lock; > + struct mutex lock; > atomic_t sub_jobs; > sector_t progress; > }; > @@ -459,7 +459,7 @@ static void segment_complete(int read_er > sector_t count = 0; > struct kcopyd_job *job = (struct kcopyd_job *) context; > > - down(&job->lock); > + mutex_lock(&job->lock); > > /* update the error */ > if (read_err) > @@ -483,7 +483,7 @@ static void segment_complete(int read_er > job->progress += count; > } > } > - up(&job->lock); > + mutex_unlock(&job->lock); > > if (count) { > int i; > @@ -565,7 +565,7 @@ int kcopyd_copy(struct kcopyd_client *kc > dispatch_job(job); > > else { > - init_MUTEX(&job->lock); > + mutex_init(&job->lock); > job->progress = 0; > split_job(job); > } > _ > > Patches currently in -mm which might be from matthias.kaehlcke@xxxxxxxxx are > > origin.patch > include-kern_-constant-in-printk-calls-in-mm-slabc.patch > srmcons-fix-kmallocgfp_kernel-inside-spinlock.patch > power-management-use-mutexes-instead-of-semaphores.patch > sysdev-use-mutex-instead-of-semaphore.patch > git-dvb.patch > use-mutex-instead-of-binary-semaphore-in-idt77252-driver.patch > qla1280-use-dma_64bit_mask-instead-of-0ull.patch > use-mutex-instead-of-binary-semaphore-in-cdu-31a-driver.patch > use-mutex-instead-of-semaphore-in-sbpcd-driver.patch > use-mutex-instead-of-semaphore-in-berkshire-usb-pc-watchdog-driver.patch > use-mutex-instead-of-semaphore-in-rocketport-driver.patch > use-mutex-instead-of-semaphore-in-tpm-driver.patch > use-mutex-instead-of-semaphore-in-hdaps-driver.patch > use-mutex-instead-of-semaphore-for-misc-char-devices.patch > fix-spinlock-usage-in-hysdn_log_close.patch > use-mutex-instead-of-semaphore-in-capi-20-interface.patch > use-mutex-instead-of-semaphore-in-virtual-console-driver.patch > kcopyd-use-mutex-instead-of-semaphore.patch > pvrusb2-use-mutex-instead-of-semaphore.patch > scx200-use-mutex-instead-of-semaphore.patch -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
