From a7c729efb12705bf53cf08af0c4fa98d33391637 Mon Sep 17 00:00:00 2001 From: nikita Date: Fri, 30 Sep 2005 10:51:05 +0000 Subject: [PATCH] remove assertion that does not buy us much, while potentially hampering interoperability --- lustre/ost/ost_handler.c | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/lustre/ost/ost_handler.c b/lustre/ost/ost_handler.c index 4053468..f96dec0 100644 --- a/lustre/ost/ost_handler.c +++ b/lustre/ost/ost_handler.c @@ -479,18 +479,6 @@ static int ost_brw_lock_get(int mode, struct obd_export *exp, if (nrbufs == 0 || !(nb[0].flags & OBD_BRW_SRVLOCK)) RETURN(0); - /* EXPENSIVE ASSERTION */ - for (i = 1; i < nrbufs; i ++) - /* - * check that niobufs are contiguous ->offset-wise. Strictly - * speaking, this is not required by the code below. What we - * are trying to assert here is that RPC we are handling was - * sent by a liblustre-style cache-less client rather than by - * usual llite OSC layer than can arbitrarily mix pages from - * different write(2) calls. - */ - LASSERT(nb[i].offset == nb[i - 1].offset + nb[i - 1].len); - policy.l_extent.start = nb[0].offset & CFS_PAGE_MASK; policy.l_extent.end = (nb[nrbufs - 1].offset + nb[nrbufs - 1].len - 1) | ~CFS_PAGE_MASK; -- 1.8.3.1