Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262113AbVDRPrb (ORCPT ); Mon, 18 Apr 2005 11:47:31 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262109AbVDRPra (ORCPT ); Mon, 18 Apr 2005 11:47:30 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:19671 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S262113AbVDRPrN (ORCPT ); Mon, 18 Apr 2005 11:47:13 -0400 Subject: Re: [RFC 1 of 9] patches to add diskdump functionality to block layer From: Arjan van de Ven To: mike.miller@hp.com Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, marcelo.tosatti@cyclades.com In-Reply-To: <20050418153644.GA25409@beardog.cca.cpqcorp.net> References: <20050418153644.GA25409@beardog.cca.cpqcorp.net> Content-Type: text/plain Date: Mon, 18 Apr 2005 17:47:07 +0200 Message-Id: <1113839227.6274.80.camel@laptopd505.fenrus.org> Mime-Version: 1.0 X-Mailer: Evolution 2.0.4 (2.0.4-2) Content-Transfer-Encoding: 7bit X-Spam-Score: 3.7 (+++) X-Spam-Report: SpamAssassin version 2.63 on pentafluge.infradead.org summary: Content analysis details: (3.7 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 1.1 RCVD_IN_DSBL RBL: Received via a relay in list.dsbl.org [] 2.5 RCVD_IN_DYNABLOCK RBL: Sent directly from dynamic IP address [80.57.133.107 listed in dnsbl.sorbs.net] 0.1 RCVD_IN_SORBS RBL: SORBS: sender is listed in SORBS [80.57.133.107 listed in dnsbl.sorbs.net] X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 850 Lines: 22 On Mon, 2005-04-18 at 10:36 -0500, mike.miller@hp.com wrote: > + > +/* > + * Extended block operations for dump for preserving binary compatibility. > + */ > +struct block_dump_ops { > + int (*sanity_check)(void *device); > + int (*rw_block)(void *device, int rw, unsigned long dump_block_nr, void *buf, int len, unsigned long start_sect, unsigned long nr_sects); > + int (*quiesce)(void *device); > + int (*shutdown)(void *device); > +}; this looks wrong. In linux we don't care about module ABI, and just go for the clean solution instead! (eg in this case, just put the methods in the block dev ops) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/