Re: [RFC][PATCH 1/3] PM: Introduce new top level suspend and hibernation callbacks (rev. 4)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thursday, 27 of March 2008, Benjamin Herrenschmidt wrote:
> 
> > > There is absolutely no point getting a second struct anymore.
> > 
> > I obviously disagree with that opinion, so please elaborate.
> 
> Well, what does it bring you ? Why can't it be one struct ? To save
> space in the data area ?

Mostly, but not only that.

There are users of 'struct pm_ops' that aren't even supposed to define the
_noirq callbacks (device types and device classes), so I thought it would be
better to introduce a separate _noirq struct after all.

> I don't think it makes things much cleaner. But I won't fight a war for it,
> now that they are clearly named differently and things like prepare/complete
> are no longer in "noirq", it's semantically the same thing as having the
> fields in one structure, so it's mostly cosmetic.

Well, I'm not going to fight for having the two separate stuctures either.
Also, it wouldn't be difficult to rearrange thigs so that all of the callbacks
are in one structure, so if other people think it's better to do it this way,
I'll go for it.

Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux