Re: [PATCH -mm 2/2] Introduce freezer flags

[Pavel Machek]

Hi!

> > > > +#ifdef CONFIG_FREEZER
> > > > +	/* Used by the process freezer, defined in freezer.h */
> > > > +	unsigned int freezer_flags;
> > > 
> > > unsigned long freezer_flags; ??
> > 
> > I guess so.
> 
> Still, on x86_64, for example, we'll waste 4 bytes by using 'unsigned long'
> here, which I wouldn't like to do.
> 
> Is it acceptable to use explicit type casting in set_bit() and friends?

I'm afraid set_bit & friends are only defined on unsigned long. You
can cast, but you may run into some nasty surprise... like damaging
surrounding bytes on 64-bit architectures != x86_64.

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/