Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c

From	Paul Bolle <pebolle@tiscali.nl>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c
Date	2016-11-08 10:20 +0100
Message-ID	<sBaJs-7rN-35@gated-at.bofh.it> (permalink)
References	(1 earlier) <sAmD0-8Y-11@gated-at.bofh.it> <sAMe5-893-7@gated-at.bofh.it> <sAOSB-1m1-5@gated-at.bofh.it> <sAS0b-3Aq-67@gated-at.bofh.it> <sASjw-3Kx-35@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, 2016-11-07 at 14:38 +0100, Heiko Carstens wrote:
> On Mon, Nov 07, 2016 at 02:13:06PM +0100, Paul Bolle wrote:
> > --- /dev/null
> > +++ b/arch/s390/include/asm/facilities.h
> > @@ -0,0 +1,43 @@
> > +#ifndef __ASM_FACILITIES_H
> > +#define __ASM_FACILITIES_H
> > +
> > +#define FACILITIES_ALS \
> > +	_AC(IS_BUILTIN(CONFIG_HAVE_MARCH_Z900_FEATURES), UL) << 0 |	/* N3 instructions */ \
> 
> So, this is wrong. It should be " << 63" to match the existing code.

Bother.

> > +#define FACILITIES_KVM \
> > +	_BITUL(0)  |	/* N3 instructions */ \
> > +	_BITUL(1)  |	/* z/Arch mode installed */ \
> > +	_BITUL(2)  |	/* z/Arch mode active */ \
> > +	_BITUL(3)  |	/* DAT-enhancement */ \
> > +	_BITUL(4)  |	/* idte segment table */ \
> > +	_BITUL(5)  |	/* idte region table */ \
> > +	_BITUL(6)  |	/* ASN-and-LX reuse */ \
> > +	_BITUL(7)  |	/* stfle */ \
> > +	_BITUL(8)  |	/* enhanced-DAT 1 */ \
> > +	_BITUL(9)  |	/* sense-running-status */ \
> > +	_BITUL(10) |	/* conditional sske */ \
> > +	_BITUL(13) |	/* ipte-range */ \
> > +	_BITUL(14)  	/* nonquiescing key-setting */ \
> > +	, \
> > +	_BITUL(9)  |	/* transactional execution */ \
> > +	_BITUL(11) |	/* access-exception-fetch/store indication */ \
> 
> And this is exactly what I want to avoid: start counting from zero again if
> we cross a double word. It _must_ read 73, 75, otherwise this becomes the
> unmaintainable and error prone mess we had before.
> 
> I just want a list with bit numbers and the rest must be created
> automatically.

This sounds like a can of worms. Better keep it closed.

Thanks,


Paul Bolle

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Masahiro Yamada <yamada.masahiro@socionext.com> - 2016-11-06 04:50 +0100
  Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Heiko Carstens <heiko.carstens@de.ibm.com> - 2016-11-07 08:10 +0100
    Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Martin Schwidefsky <schwidefsky@de.ibm.com> - 2016-11-07 11:00 +0100
      Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Paul Bolle <pebolle@tiscali.nl> - 2016-11-07 14:20 +0100
        Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Heiko Carstens <heiko.carstens@de.ibm.com> - 2016-11-07 14:40 +0100
          Re: [PATCH 2/2] s390: squash facilities_src.h into gen_facilities.c Paul Bolle <pebolle@tiscali.nl> - 2016-11-08 10:20 +0100

csiph-web