From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030208AbVIIQgp (ORCPT ); Fri, 9 Sep 2005 12:36:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751429AbVIIQgo (ORCPT ); Fri, 9 Sep 2005 12:36:44 -0400 Received: from zeniv.linux.org.uk ([195.92.253.2]:6345 "EHLO ZenIV.linux.org.uk") by vger.kernel.org with ESMTP id S1750831AbVIIQgo (ORCPT ); Fri, 9 Sep 2005 12:36:44 -0400 Date: Fri, 9 Sep 2005 17:36:43 +0100 From: viro@ZenIV.linux.org.uk To: Andreas Schwab Cc: Linus Torvalds , linux-kernel@vger.kernel.org Subject: Re: [PATCH] bogus cast in bio.c Message-ID: <20050909163643.GO9623@ZenIV.linux.org.uk> References: <20050909155356.GF9623@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 09, 2005 at 06:18:14PM +0200, Andreas Schwab wrote: > viro@ZenIV.linux.org.uk writes: > > > void * is not the same as void *... > > IMHO it should. ... and indeed it should. My apologies - that's a combination of sparse bug and me being very low on coffee... FWIW, looks like we are losing address_space in the first form: with current sparse we get fs/bio.c:686:15: warning: incorrect type in assignment (different address spaces) fs/bio.c:686:15: expected void [noderef] *iov_base fs/bio.c:686:15: got void [noderef] * from the first form (cast to __user void *). Lovely... OK, I think I know what's going on there, will fix.