<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <title></title>

</head>

<body bgcolor="#ffffff" text="#000099">

Roland,<br>

<blockquote cite="mid52zmpvhll8.fsf@cisco.com" type="cite">

  <pre wrap="">    Vu> Since all the tuned parameter are target-centralized (passing

    Vu> in when add new target) I think about moving FMR resources

    Vu> (size, max_page...) ie. fmr_pool into srp_target_port

    Vu> struct. Each newly added target will have their own customized

    Vu> FMR pool.

That makes some sense.  An issue is that FMRs are a fairly limited

resource, and a system with many SRP targets where each target doesn't

get much traffic could tie up a lot of FMRs.

  </pre>

</blockquote>

You're right. For the same reason of unused port (ie. srp_host), I

create fmr resource per device and keep it in srp_device_data struct<br>

<br>

I put back fmr + your patch and it works well with my setup.<br>

<br>

Signed-off-by: Vu Pham <a href="mailto:sean.hefty@intel.com"><vu@mellanox.com></a><br>

<br>

 <br>

</body>

</html>