lkml.org 
[lkml]   [2002]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Kill obsolete and unused suspend/resume code from IDE
Hi!

> > This code in ide is obsolete and unused. I have followup patch to
> > integrate IDE into sysfs. Please apply,
>
> The code is not obsolete, it is not unused.

Can you point out what uses it? I certainly could not find it and
killing standby/etc produced no compilation errors.

As of obsolete... It is doing power managment outside of sysfs
framework... Which is not how it should be done.

> > + do_idedisk_standby(drive);
> > if ((drive->id->cfs_enable_2 & 0x3000) && drive->wcache)
> > if (do_idedisk_flushcache(drive))
> > printk (KERN_INFO "%s: Write Cache FAILED Flushing!\n",
>
> What locking rules are you using here ?

I did not change the code, it works exactly as it did before (ide disk
is never ATAPI device). If it is broken then sorry (but it was broken
before).

> Linus please reject this patch. Its just getting in the way of actually
> fixing the IDE code properly.

Linus actually asked me for the patch, and discussion about it was
public, then he silently droped it. This was "only" retransmit.

Pavel
--
Casualities in World Trade Center: ~3k dead inside the building,
cryptography in U.S.A. and free speech in Czech Republic.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:31    [W:0.078 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site