DataTable sottocosto = new DataTable();
        sottocosto = (DataTable) cache["ListaFoto"];


        var results = from Row in sottocosto.AsEnumerable()
                      where Row.Field<string>("DESCRIZIONE").StartsWith("***")
                      select new
                          {
                              COD_ARTICOLO = Row.Field<string>("COD_ARTICOLO"),
                              DESCRIZIONE = Row.Field<string>("DESCRIZIONE"),
                          };


        homeSottocosto.DataSource = results;
        homeSottocosto.DataBind();

EnumerableRowCollection<DataRow> lista = form n in dw.AsEnumerable() where !n.Field<string>("Descrizione").StartWith("***") || !n.Field<string>("Descrizione").StartWith("# ") || !n.Field<string>("Descrizione").StartWith("* ") select n;

Cambierei solo questo pezzetto di codice:

DataTable sottocosto = new DataTable();
sottocosto = (DataTable) cache["ListaFoto"];

in

DataTable sottocosto = cache["ListaFoto"] as DataTable;
if (sottocosto == null){
  //non so come funziona il tuo meccanismo di caching, ma se qui
  //la variabile sottocosto dovesse esser vuota, puoi andare al
  //db a recuperare i dati
}

Sì, secondo me è corretto. Presumo che i "prodotti in sottocosto" siano identici per tutti gli utenti che accedono all'homepage, quindi perché accedere al db per mostrare dati che sono sempre uguali per tutti?...
...e magari metterei in cache anche il "results", così da non doverlo ricalcolare ad ogni richiesta di pagina.