Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 15 Nov 2017 12:22:14 +0100
From:   Krzysztof Mazur <krzysiek@podlesie.net>
To:     Richard Weinberger <richard@nod.at>
Cc:     Jeff Dike <jdike@addtoit.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>,
        "H. Peter Anvin" <hpa@zytor.com>, x86@kernel.org,
        user-mode-linux-devel@lists.sourceforge.net,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] um: use POSIX ucontext_t instead of struct ucontext
Message-ID: <20171115112214.GA12121@shrek.podlesie.net>
References: <20171115101239.4882-1-krzysiek@podlesie.net>
 <1518438.KRCSCszYuI@blindfold>
 <20171115110416.GA29538@shrek.podlesie.net>
 <1828554.ETQJSmL9mu@blindfold>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1828554.ETQJSmL9mu@blindfold>
User-Agent: Mutt/1.6.2 (2016-07-01)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Nov 15, 2017 at 12:15:00PM +0100, Richard Weinberger wrote:
> Am Mittwoch, 15. November 2017, 12:04:16 CET schrieb Krzysztof Mazur:
> > On Wed, Nov 15, 2017 at 11:19:41AM +0100, Richard Weinberger wrote:
> > > Am Mittwoch, 15. November 2017, 11:12:39 CET schrieb Krzysztof Mazur:
> > > > glibc 2.26 removed the 'struct ucontext' to "improve" POSIX compliance
> > > > and break programs, including User Mode Linux. Fix User Mode Linux
> > > > by using POSIX ucontext_t.
> > > > 
> > > > This fixes:
> > > > 
> > > > arch/um/os-Linux/signal.c: In function 'hard_handler':
> > > > arch/um/os-Linux/signal.c:163:22: error: dereferencing pointer to
> > > > incomplete type 'struct ucontext' mcontext_t *mc = &uc->uc_mcontext;
> > > > arch/x86/um/stub_segv.c: In function 'stub_segv_handler':
> > > > arch/x86/um/stub_segv.c:16:13: error: dereferencing pointer to
> > > > incomplete
> > > > type 'struct ucontext' &uc->uc_mcontext);
> > > 
> > > Do all older glibcs have ucontext_t?
> > > Otherwise this patch will break other stuff.
> > 
> > Yes, ucontext_t typedef was always available. They changed:
> > 
> > typedef struct ucontext { ... } ucontex_t;
> > 
> > to
> > 
> > typedef struct ucontext_t { ... } ucontex_t;
> > 
> > https://sourceware.org/glibc/wiki/Release/2.26#Removal_of_.27struct_ucontext
> > .27
> 
> Okay, then we can mark your patch as stable and hope for the best. ;-)

Thanks

Krzysiek

From 1584130919663052874@xxx Wed Nov 15 11:22:51 +0000 2017
X-GM-THRID: 1584126952248990726
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread